refactor: rename Team.getRepos to Team.listRepos

clayreimann · clayreimann · commit 7d9afd04f94c · 2016-05-25T17:10:03.000-05:00
diff --git a/lib/Team.js b/lib/Team.js
@@ -41,7 +41,7 @@ class Team extends Requestable {
     * @param {Requestable.callback} [cb] - will receive the list of repositories
     * @return {Promise} - the promise for the http request
     */
-   getRepos(cb) {
+   listRepos(cb) {
       log(`Fetching repositories for Team ${this.__teamId}`);
       return this._requestAllPages(`/teams/${this.__teamId}/repos`, undefined, cb);
    }
diff --git a/test/team.spec.js b/test/team.spec.js
@@ -58,45 +58,47 @@ describe('Team', function() { // Isolate tests that are based on a fixed team
 
             let hasTestUser = members.reduce(
                (found, member) => member.login === testUser.USERNAME || found,
-               false);
+               false
+            );
 
             expect(hasTestUser).to.be.true();
          });
    });
 
    it('should get team repos', function() {
-      return team.getRepos()
-        .then(({data}) => {
-           const hasRepo = data.reduce(
-              (found, repo) => repo.name === 'fixed-test-repo-1' || found,
-              false);
-
-           expect(hasRepo).to.be.true();
-        });
+      return team.listRepos()
+         .then(({data}) => {
+            const hasRepo = data.reduce(
+               (found, repo) => repo.name === 'fixed-test-repo-1' || found,
+               false
+            );
+
+            expect(hasRepo).to.be.true();
+         });
    });
 
    it('should get team', function() {
       return team.getTeam()
-        .then(({data}) => {
-           expect(data.name).to.equal('Fixed Test Team 1');
-        });
+         .then(({data}) => {
+            expect(data.name).to.equal('Fixed Test Team 1');
+         });
    });
 
    it('should check if team manages repo', function() {
       return team.isManagedRepo(testUser.ORGANIZATION, 'fixed-test-repo-1')
-        .then((result) => {
-           expect(result).to.be.true();
-        });
+         .then((result) => {
+            expect(result).to.be.true();
+         });
    });
 });
 
 describe('Team', function() { // Isolate tests that need a new team per test
    beforeEach(function() {
       return createTestTeam()
-        .then((x) => {
-           team = x.team;
-           name = x.name;
-        });
+         .then((x) => {
+            team = x.team;
+            name = x.name;
+         });
    });
 
    // Test for Team deletion
@@ -108,9 +110,9 @@ describe('Team', function() { // Isolate tests that need a new team per test
    it('should update team', function() {
       const newName = `${name}-updated`;
       return team.editTeam({name: newName})
-        .then(function({data}) {
-           expect(data.name).to.equal(newName);
-        });
+         .then(function({data}) {
+            expect(data.name).to.equal(newName);
+         });
    });
 
    it('should add membership for a given user', function() {

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ class Team extends Requestable {`
`41`	`41`	`* @param {Requestable.callback} [cb] - will receive the list of repositories`
`42`	`42`	`* @return {Promise} - the promise for the http request`
`43`	`43`	`*/`
`44`		`- getRepos(cb) {`
	`44`	`+ listRepos(cb) {`
`45`	`45`	log(`Fetching repositories for Team ${this.__teamId}`);
`46`	`46`	return this._requestAllPages(`/teams/${this.__teamId}/repos`, undefined, cb);
`47`	`47`	`}`