feature(organization): add api to list organization members

clayreimann · clayreimann · commit c35cdbf7713a · 2016-05-11T10:30:07.000-05:00
Closes #265
diff --git a/lib/Organization.js b/lib/Organization.js
@@ -44,6 +44,19 @@ class Organization extends Requestable {
 
       return this._requestAllPages(`/orgs/${this.__name}/repos`, requestOptions, cb);
    }
+
+   /**
+    * List the users who are members of the company
+    * @see https://developer.github.com/v3/orgs/members/#members-list
+    * @param {object} options
+    * @param {string} [options.filter=all] - can be either `2fa_disabled` or `all`
+    * @param {string} [options.role=all] - can be one of: `all`, `admin`, or `member`
+    * @param {Requestable.callback} [cb] - will receive the list of users
+    * @return {Promise} - the promise for the http request
+    */
+   listMembers(options, cb) {
+      return this._request('GET', `/orgs/${this.__name}/members`, options, cb);
+   }
 }
 
 module.exports = Organization;
diff --git a/test/organization.spec.js b/test/organization.spec.js
@@ -5,9 +5,9 @@ import testUser from './fixtures/user.json';
 import {assertSuccessful, assertArray} from './helpers/callbacks';
 import getTestRepoName from './helpers/getTestRepoName';
 
-// jscs:disable requireCamelCaseOrUpperCaseIdentifiers
 describe('Organization', function() {
    let github;
+   const ORG_NAME = 'github-tools';
 
    before(function() {
       github = new Github({
@@ -22,12 +22,24 @@ describe('Organization', function() {
       let organization;
 
       before(function() {
-         organization = github.getOrganization('openaddresses');
+         organization = github.getOrganization(ORG_NAME);
       });
 
       it('should show user\'s organisation repos', function(done) {
          organization.getRepos(assertArray(done));
       });
+
+      it('should list the users in the organization', function(done) {
+         organization.listMembers()
+            .then(function({data: members}) {
+               expect(members).to.be.an.array();
+
+               let hasClayReimann = members.reduce((found, member) => member.login === 'clayreimann' || found, false);
+               expect(hasClayReimann).to.be.true();
+
+               done();
+            }).catch(done);
+      });
    });
 
    describe('creating/updating', function() {
@@ -44,14 +56,14 @@ describe('Organization', function() {
             description: 'test create organization repo',
             homepage: 'https://github.com/',
             private: false,
-            has_issues: true,
-            has_wiki: true,
-            has_downloads: true
+            has_issues: true, // jscs:ignore
+            has_wiki: true, // jscs:ignore
+            has_downloads: true // jscs:ignore
          };
 
          organization.createRepo(options, assertSuccessful(done, function(err, repo) {
             expect(repo.name).to.equal(testRepoName);
-            expect(repo.full_name).to.equal(`${testUser.ORGANIZATION}/${testRepoName}`);
+            expect(repo.full_name).to.equal(`${testUser.ORGANIZATION}/${testRepoName}`); // jscs:ignore
             done();
          }));
       });
