Removed non-working code

it is guaranteed to not work as it stands, as we do not get the uri
property in the req object at all (atleast when testing with github)
and with github enterprise - things just dont't get set up correctly
in the first place.

Also, the string /api/v3 will be a part of the uri (which is not
present in the result), however - a part of the endpoint URL and  not
a part of the path (which is present in the result) in any case.

Author: saraf

lib/api.js

@@ -6,6 +6,7 @@ var debug = require('debug')('strider-github:api');
 var Step = require('step');
 var superagent = require('superagent');
 var url = require('url');
+var util = require('util');
 
 var GITHUB_API_ENDPOINT = process.env.PLUGIN_GITHUB_API_ENDPOINT || 'https://api.github.com';
 
@@ -418,16 +419,9 @@ function getRepos(token, username, callback) {
 
       _.each(results, function (result) {
         var team_id = result.req.path.split('/');
-        if (result.req.path.indexOf('/api/v3/') > -1) {
-          // Github Enterprise url
-          team_id = team_id[4]; //TODO: need a github enterprise account for
-                                // this to be tested - does the path
-                                // really contain /api/v3?
-        }
-        else {
-          // Github url
-          team_id = team_id[2];
-        }
+        debug("We get the following repo path: " + util.inspect(result.req.path));
+        team_id = team_id[2];
+
         var team_detail = team_detail_requests[parseInt(team_id, 10)];
         if (result.statusCode === 204) {
           pageinated_api_call('/teams/' + team_id + '/repos', token, group());

test/test_api.js

@@ -1,6 +1,6 @@
 
 var expect = require('expect.js')
-  , api = require('../lib/api')
+  , api = require('../lib/api');
 
 describe('github api', function () {
   describe('getFile', function () {