{"action":"created","issue":{"url":"https://api.github.com/repos/oppia/oppia/issues/2153","repository_url":"https://api.github.com/repos/oppia/oppia","labels_url":"https://api.github.com/repos/oppia/oppia/issues/2153/labels{/name}","comments_url":"https://api.github.com/repos/oppia/oppia/issues/2153/comments","events_url":"https://api.github.com/repos/oppia/oppia/issues/2153/events","html_url":"https://github.com/oppia/oppia/pull/2153","id":161640424,"number":2153,"title":"Test collection linearizer service","user":{"login":"amgowano","id":2756900,"avatar_url":"https://avatars.githubusercontent.com/u/2756900?v=3","gravatar_id":"","url":"https://api.github.com/users/amgowano","html_url":"https://github.com/amgowano","followers_url":"https://api.github.com/users/amgowano/followers","following_url":"https://api.github.com/users/amgowano/following{/other_user}","gists_url":"https://api.github.com/users/amgowano/gists{/gist_id}","starred_url":"https://api.github.com/users/amgowano/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amgowano/subscriptions","organizations_url":"https://api.github.com/users/amgowano/orgs","repos_url":"https://api.github.com/users/amgowano/repos","events_url":"https://api.github.com/users/amgowano/events{/privacy}","received_events_url":"https://api.github.com/users/amgowano/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/oppia/oppia/labels/PR:%20don't%20merge%20-%20IN%20REVIEW","name":"PR: don't merge - IN REVIEW","color":"e11d21"}],"state":"open","locked":false,"assignee":{"login":"kevinlee12","id":5758256,"avatar_url":"https://avatars.githubusercontent.com/u/5758256?v=3","gravatar_id":"","url":"https://api.github.com/users/kevinlee12","html_url":"https://github.com/kevinlee12","followers_url":"https://api.github.com/users/kevinlee12/followers","following_url":"https://api.github.com/users/kevinlee12/following{/other_user}","gists_url":"https://api.github.com/users/kevinlee12/gists{/gist_id}","starred_url":"https://api.github.com/users/kevinlee12/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kevinlee12/subscriptions","organizations_url":"https://api.github.com/users/kevinlee12/orgs","repos_url":"https://api.github.com/users/kevinlee12/repos","events_url":"https://api.github.com/users/kevinlee12/events{/privacy}","received_events_url":"https://api.github.com/users/kevinlee12/received_events","type":"User","site_admin":false},"assignees":[{"login":"kevinlee12","id":5758256,"avatar_url":"https://avatars.githubusercontent.com/u/5758256?v=3","gravatar_id":"","url":"https://api.github.com/users/kevinlee12","html_url":"https://github.com/kevinlee12","followers_url":"https://api.github.com/users/kevinlee12/followers","following_url":"https://api.github.com/users/kevinlee12/following{/other_user}","gists_url":"https://api.github.com/users/kevinlee12/gists{/gist_id}","starred_url":"https://api.github.com/users/kevinlee12/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kevinlee12/subscriptions","organizations_url":"https://api.github.com/users/kevinlee12/orgs","repos_url":"https://api.github.com/users/kevinlee12/repos","events_url":"https://api.github.com/users/kevinlee12/events{/privacy}","received_events_url":"https://api.github.com/users/kevinlee12/received_events","type":"User","site_admin":false},{"login":"BenHenning","id":12983742,"avatar_url":"https://avatars.githubusercontent.com/u/12983742?v=3","gravatar_id":"","url":"https://api.github.com/users/BenHenning","html_url":"https://github.com/BenHenning","followers_url":"https://api.github.com/users/BenHenning/followers","following_url":"https://api.github.com/users/BenHenning/following{/other_user}","gists_url":"https://api.github.com/users/BenHenning/gists{/gist_id}","starred_url":"https://api.github.com/users/BenHenning/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BenHenning/subscriptions","organizations_url":"https://api.github.com/users/BenHenning/orgs","repos_url":"https://api.github.com/users/BenHenning/repos","events_url":"https://api.github.com/users/BenHenning/events{/privacy}","received_events_url":"https://api.github.com/users/BenHenning/received_events","type":"User","site_admin":false}],"milestone":null,"comments":9,"created_at":"2016-06-22T09:59:33Z","updated_at":"2016-07-12T20:38:46Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/oppia/oppia/pulls/2153","html_url":"https://github.com/oppia/oppia/pull/2153","diff_url":"https://github.com/oppia/oppia/pull/2153.diff","patch_url":"https://github.com/oppia/oppia/pull/2153.patch"},"body":"Fix #1998: implement tests for collection linearizer service.\r\n"},"comment":{"url":"https://api.github.com/repos/oppia/oppia/issues/comments/232173100","html_url":"https://github.com/oppia/oppia/pull/2153#issuecomment-232173100","issue_url":"https://api.github.com/repos/oppia/oppia/issues/2153","id":232173100,"user":{"login":"amgowano","id":2756900,"avatar_url":"https://avatars.githubusercontent.com/u/2756900?v=3","gravatar_id":"","url":"https://api.github.com/users/amgowano","html_url":"https://github.com/amgowano","followers_url":"https://api.github.com/users/amgowano/followers","following_url":"https://api.github.com/users/amgowano/following{/other_user}","gists_url":"https://api.github.com/users/amgowano/gists{/gist_id}","starred_url":"https://api.github.com/users/amgowano/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amgowano/subscriptions","organizations_url":"https://api.github.com/users/amgowano/orgs","repos_url":"https://api.github.com/users/amgowano/repos","events_url":"https://api.github.com/users/amgowano/events{/privacy}","received_events_url":"https://api.github.com/users/amgowano/received_events","type":"User","site_admin":false},"created_at":"2016-07-12T20:38:45Z","updated_at":"2016-07-12T20:38:45Z","body":"@kevinlee12: Thanks for the link to the doc. Actually my concern for the distinction were based on changes to`CollectionLinearizerService.js` file which is not a testing file. With the changes suggested, code that used to look like:\r\n\r\n```\r\ngetNextExplorationIds: function(collection, completedExpIds) {\r\n    return _getNextExplorationIds(collection, completedExpIds);\r\n}\r\n```\r\n\r\nWill now look like this:\r\n\r\n```\r\ngetNextExplorationIds: function(collection, completedExpIds) {\r\n    return getNextExplorationIds(collection, completedExpIds);\r\n}\r\n```\r\n\r\nNow there are two functions with the same name. "}}
{"actor":{"display_login":"amgowano"}}