add groupMembers resource

Author: fengmk2

Makefile

@@ -1,8 +1,28 @@
 gitlab
 =======
 
-[![Build Status](https://secure.travis-ci.org/repo-utils/gitlab.png)](http://travis-ci.org/repo-utils/gitlab)
-[![Coverage Status](https://coveralls.io/repos/repo-utils/gitlab/badge.png)](https://coveralls.io/r/repo-utils/gitlab)
+[![NPM version][npm-image]][npm-url]
+[![build status][travis-image]][travis-url]
+[![Test coverage][coveralls-image]][coveralls-url]
+[![Gittip][gittip-image]][gittip-url]
+[![David deps][david-image]][david-url]
+[![node version][node-image]][node-url]
+[![npm download][download-image]][download-url]
+
+[npm-image]: https://img.shields.io/npm/v/node-gitlab.svg?style=flat-square
+[npm-url]: https://npmjs.org/package/node-gitlab
+[travis-image]: https://img.shields.io/travis/repo-utils/gitlab.svg?style=flat-square
+[travis-url]: https://travis-ci.org/repo-utils/gitlab
+[coveralls-image]: https://img.shields.io/coveralls/repo-utils/gitlab.svg?style=flat-square
+[coveralls-url]: https://coveralls.io/r/repo-utils/gitlab?branch=master
+[gittip-image]: https://img.shields.io/gittip/fengmk2.svg?style=flat-square
+[gittip-url]: https://www.gittip.com/fengmk2/
+[david-image]: https://img.shields.io/david/repo-utils/gitlab.svg?style=flat-square
+[david-url]: https://david-dm.org/repo-utils/gitlab
+[node-image]: https://img.shields.io/badge/node.js-%3E=_0.10-green.svg?style=flat-square
+[node-url]: http://nodejs.org/download/
+[download-image]: https://img.shields.io/npm/dm/node-gitlab.svg?style=flat-square
+[download-url]: https://npmjs.org/package/node-gitlab
 
 ![logo](https://raw.github.com/repo-utils/gitlab/master/logo.png)
 
@@ -13,7 +33,7 @@ Gitlab API nodejs client.
 ## Install
 
 ```bash
-$ npm install node-gitlab
+$ npm install node-gitlab --save
 ```
 
 ## Usage

README.md

@@ -16,14 +16,15 @@ var util = require('util');
 var Milestone = require('./milestone');
 var Project = require('./project');
 var Member = require('./member');
-var Hook = require('./hook');
 var GlobalHook = require('./globalHook.js');
 var Issue = require('./issue');
 var User = require('./user');
 var Repository = require('./repository');
 var MergeRequest = require('./merge_request');
 var RepositoryFile = require('./repository_file');
 
+module.exports = Gitlab;
+
 /**
  * Create a gitlab API client.
  *
@@ -34,23 +35,31 @@ var RepositoryFile = require('./repository_file');
 function Gitlab(options) {
   options = options || {};
   options.api = options.api || 'https://gitlab.com/api/v3';
-  this.constructor.super_.call(this, options);
+  restful.RESTFulClient.call(this, options);
 
   this.privateToken = options.privateToken;
 
-  this.milestones = new Milestone(this);
-  this.members = new Member(this);
-  this.hooks = new Hook(this);
-  this.globalHooks = new GlobalHook(this);
-  this.issues = new Issue(this);
-  this.users = new User(this);
-  this.merge_requests = this.mergeRequests = new MergeRequest(this);
-
   this.addResources({
     projects: Project,
     repository: Repository,
     repositoryFiles: RepositoryFile,
+    mergeRequests: MergeRequest,
+    users: User,
+    issues: Issue,
+    globalHooks: GlobalHook,
+    members: Member,
+    milestones: Milestone,
+    hooks: {
+      resourcePath: '/projects/:id/hooks',
+      idName: 'hook_id',
+    },
+    groupMembers: {
+      resourcePath: '/groups/:id/members',
+      idName: 'user_id',
+    },
   });
+
+  this.merge_requests = this.mergeRequests;
 }
 
 util.inherits(Gitlab, restful.RESTFulClient);
@@ -60,6 +69,6 @@ Gitlab.prototype.setAuthentication = function (req) {
   return req;
 };
 
-exports.create = function (options) {
+Gitlab.create = function (options) {
   return new Gitlab(options);
 };

lib/gitlab.js

@@ -4,24 +4,27 @@
   "description": "Gitlab API nodejs client.",
   "main": "index.js",
   "scripts": {
-    "test": "make test-all",
-    "blanket": { "pattern": "gitlab/lib" },
-    "travis-cov": { "threshold": 100 }
+    "test": "mocha -R spec -t 20000 -r should-http test/*.test.js",
+    "test-cov": "node --harmony node_modules/.bin/istanbul cover node_modules/.bin/_mocha -- -t 20000 -r should-http test/*.test.js",
+    "test-travis": "node --harmony node_modules/.bin/istanbul cover node_modules/.bin/_mocha --report lcovonly -- -t 20000 -r should-http test/*.test.js",
+    "jshint": "jshint .",
+    "autod": "autod -w --prefix '~' && npm run cnpm",
+    "cnpm": "npm install --registry=https://registry.npm.taobao.org",
+    "contributors": "contributors -f plain -o AUTHORS"
   },
   "dependencies": {
-    "debug": "~1.0.2",
-    "restful-client": "~0.2.0"
+    "debug": "~1.0.4",
+    "restful-client": "~1.0.0"
   },
   "devDependencies": {
     "autod": "*",
     "blanket": "*",
     "contributors": "*",
     "coveralls": "*",
     "mocha": "*",
-    "mocha-lcov-reporter": "*",
-    "pedding": "~0.0.3",
-    "should": "~3.3.2",
-    "travis-cov": "*"
+    "pedding": "~1.0.0",
+    "should": "~4.0.4",
+    "should-http": "*"
   },
   "homepage": "https://github.com/repo-utils/gitlab",
   "repository": {

lib/hook.js

@@ -82,7 +82,7 @@ describe('issue.test.js', function () {
           state_event: 'close',
         }, function (err, row) {
           should.not.exists(err);
-          row.title.should.include(' update');
+          row.title.should.containEql(' update');
           row.state.should.equal('closed');
           done();
         });

package.json

@@ -216,7 +216,7 @@ describe.skip('repository.test.js', function () {
         should.ok(Buffer.isBuffer(blob));
         blob.should.be.instanceof(Buffer);
         blob.length.should.above(0);
-        blob.toString().should.include('alidata - lib/alidata.js');
+        blob.toString().should.containEql('alidata - lib/alidata.js');
         done();
       });
     });
@@ -230,7 +230,7 @@ describe.skip('repository.test.js', function () {
         should.ok(Buffer.isBuffer(raw));
         raw.should.be.a.Buffer;
         raw.length.should.above(0);
-        raw.toString().should.include('gitlab-client-unittest\n=======\n\n');
+        raw.toString().should.containEql('gitlab-client-unittest\n=======\n\n');
         done();
       });
     });