Support patch, update and remove many (#41)

src/client/base.js

@@ -13,7 +13,7 @@ export default class Base {
     params = params || {};
     let url = this.base;
 
-    if (typeof id !== 'undefined') {
+    if (typeof id !== 'undefined' && id !== null) {
       url += `/${id}`;
     }
 

test/client/fetch.test.js

@@ -40,11 +40,11 @@ describe('fetch REST connector', function() {
       done();
     });
   });
-  
+
   it('can initialize a client instance', done => {
     const init = rest(url).fetch(fetch);
     const todos = init.service('todos');
-    
+
     assert.ok(todos instanceof init.Service, 'Returned service is a client');
     todos.find({}).then(todos => assert.deepEqual(todos, [
       {
@@ -54,4 +54,12 @@ describe('fetch REST connector', function() {
       }
     ])).then(() => done()).catch(done);
   });
+
+  it('remove many', done => {
+    service.remove(null).then(todo => {
+      assert.equal(todo.id, null);
+      assert.equal(todo.text, 'deleted many');
+      done();
+    });
+  });
 });

test/client/jquery.test.js

@@ -45,11 +45,11 @@ describe('jQuery REST connector', function() {
         query: {}
       })).then(done).catch(done);
   });
-  
+
   it('can initialize a client instance', done => {
     const init = rest(url).jquery(service.connection);
     const todos = init.service('todos');
-    
+
     assert.ok(todos instanceof init.Service, 'Returned service is a client');
     todos.find({}).then(todos => assert.deepEqual(todos, [
       {
@@ -59,4 +59,12 @@ describe('jQuery REST connector', function() {
       }
     ])).then(() => done()).catch(done);
   });
+
+  it('remove many', done => {
+    service.remove(null).then(todo => {
+      assert.equal(todo.id, null);
+      assert.equal(todo.text, 'deleted many');
+      done();
+    });
+  });
 });

test/client/request.test.js

@@ -54,4 +54,12 @@ describe('node-request REST connector', function() {
       done();
     }).catch(done);
   });
+
+  it('remove many', done => {
+    service.remove(null).then(todo => {
+      assert.equal(todo.id, null);
+      assert.equal(todo.text, 'deleted many');
+      done();
+    });
+  });
 });

test/client/server.js

@@ -33,13 +33,23 @@ let errorHandler = function(error, req, res, next) {
 module.exports = function(configurer) {
   // Create an in-memory CRUD service for our Todos
   var todoService = memory().extend({
-    get: function(id, params) {
+    get(id, params) {
       if(params.query.error) {
         throw new Error('Something went wrong');
       }
 
       return this._super(id, params)
         .then(data => Object.assign({ query: params.query }, data));
+    },
+
+    remove(id) {
+      if(id === null) {
+        return Promise.resolve({
+          id, text: 'deleted many'
+        });
+      }
+
+      return this._super.apply(this, arguments);
     }
   });
 

test/client/superagent.test.js

@@ -33,11 +33,11 @@ describe('Superagent REST connector', function() {
         query: {}
       })).then(done).catch(done);
   });
-  
+
   it('can initialize a client instance', done => {
     const init = rest(url).superagent(superagent);
     const todos = init.service('todos');
-    
+
     assert.ok(todos instanceof init.Service, 'Returned service is a client');
     todos.find({}).then(todos => assert.deepEqual(todos, [
       {
@@ -47,4 +47,12 @@ describe('Superagent REST connector', function() {
       }
     ])).then(() => done()).catch(done);
   });
+
+  it('remove many', done => {
+    service.remove(null).then(todo => {
+      assert.equal(todo.id, null);
+      assert.equal(todo.text, 'deleted many');
+      done();
+    });
+  });
 });