Merge pull request #52 from saraf/nocktest

Testing:  getting repos - mocking GitHub with nock

Author: Ilya Radchenko

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",