Testing: getting repos - mocking GitHub with nock

Simulate a case where a user Strider Tester is registered
with github and has admin access to TWO repositories
one which belongs to him (stridertester/proj1) and one that
belongs to a team Strider Testers Union (stridertestersunion/union-proj-1)

getRepos should return an array containing the two repositories.
we are using actual responses received from github.com - as recorded
and mocked by nock to simulate.

This test was failing and we were getting three repositories.
This is because when getting the repos for the user, we also get
all the repos that he has access to from the teams of which he is a
member.

Also this was from the fix I intended to check in for
Strider-CD/strider#846

xdissent has already checked in the fix,
I have changed the comment to explain why a de-dupe is really needed.

Author: saraf

lib/api.js

@@ -420,7 +420,7 @@ function getRepos(token, username, callback) {
       _.each(results, function (result) {
         var team_id = result.req.path.split('/');
         debug("We get the following repo path: " + util.inspect(result.req.path));
-        team_id = team_id[2];
+        team_id = team_id[2]; //TODO: add Enterprise support. This might be different for Github Enterprise 
 
         var team_detail = team_detail_requests[parseInt(team_id, 10)];
         if (result.statusCode === 204) {
@@ -458,7 +458,8 @@ function getRepos(token, username, callback) {
         }
       });
 
-      // Sometimes we can get multiple copies of the same repo, so we uniq it
+      //If the user is a member of a team, we get a repository in repos
+      //as well as team_repos. Thus we need to merge the two and de-dupe 
       repos = _.uniq(repos.concat(team_repos), false, function (item) {
         return item.id;
       });

package.json

@@ -35,9 +35,10 @@
     "inline_icon": "github"
   },
   "devDependencies": {
-    "jshint": "~2.1.11",
     "expect.js": "~0.2.0",
-    "mocha": "~1.13.0"
+    "jshint": "~2.1.11",
+    "mocha": "~1.13.0",
+    "nock": "^2.10.0"
   },
   "dependencies": {
     "async": "~0.2.9",