{"action":"opened","issue":{"url":"https://api.github.com/repos/ColumbiaJS/js-course/issues/32","labels_url":"https://api.github.com/repos/ColumbiaJS/js-course/issues/32/labels{/name}","comments_url":"https://api.github.com/repos/ColumbiaJS/js-course/issues/32/comments","events_url":"https://api.github.com/repos/ColumbiaJS/js-course/issues/32/events","html_url":"https://github.com/ColumbiaJS/js-course/issues/32","id":58257541,"number":32,"title":"Unregistered functions","user":{"login":"levbrie","id":1441307,"avatar_url":"https://avatars.githubusercontent.com/u/1441307?v=3","gravatar_id":"","url":"https://api.github.com/users/levbrie","html_url":"https://github.com/levbrie","followers_url":"https://api.github.com/users/levbrie/followers","following_url":"https://api.github.com/users/levbrie/following{/other_user}","gists_url":"https://api.github.com/users/levbrie/gists{/gist_id}","starred_url":"https://api.github.com/users/levbrie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/levbrie/subscriptions","organizations_url":"https://api.github.com/users/levbrie/orgs","repos_url":"https://api.github.com/users/levbrie/repos","events_url":"https://api.github.com/users/levbrie/events{/privacy}","received_events_url":"https://api.github.com/users/levbrie/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ColumbiaJS/js-course/labels/hw3","name":"hw3","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-02-19T19:37:49Z","updated_at":"2015-02-19T19:37:49Z","closed_at":null,"body":"@zgleicher asked: For the test, \"shouldn\\'t treat the parameters in nested functions as dependencies\" in DI.spec.js simple, parameters 'app' and 'login' have not been registered. 'login' does not even exist. Should it be Auth and User instead?\r\n\r\nMy response:\r\n\r\n@zgleicher Yes, it probably should be Auth and User but your tests should pass regardless. I believe at some point I renamed app and and long to Auth and User, but ended up neglecting this test because it passes anyway, since this particular test is testing how many arguments your inject method has parsed and making sure it doesn't parse nested functions, rather than actually making use of any of those nested injected functions as it does in other tests. Good catch!"}}