Test and fix for request to serialize errors properly

src/client/request.js

@@ -11,7 +11,11 @@ export default class Service extends Base {
         }
 
         if(!error && res.statusCode >= 400) {
-          return reject(new Error(data));
+          if(typeof data === 'string') {
+            return reject(new Error(data));
+          }
+
+          return reject(Object.assign(new Error(data.message), data));
         }
 
         resolve(data);

test/client/request.test.js

@@ -33,11 +33,11 @@ describe('node-request REST connector', function() {
         query: {}
       })).then(done).catch(done);
   });
-  
+
   it('can initialize a client instance', done => {
     const init = rest(url).request(request);
     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,11 @@ describe('node-request REST connector', function() {
       }
     ])).then(() => done()).catch(done);
   });
+
+  it('converts errors properly', done => {
+    service.get(1, { query: { error: true } }).catch(e => {
+      assert.equal(e.message, 'Something went wrong');
+      done();
+    }).catch(done);
+  });
 });

test/client/server.js

@@ -3,6 +3,20 @@ import bodyParser from 'body-parser';
 import memory from 'feathers-memory';
 import rest from '../../src';
 
+Object.defineProperty(Error.prototype, 'toJSON', {
+    value: function () {
+        var alt = {};
+
+        Object.getOwnPropertyNames(this).forEach(function (key) {
+            alt[key] = this[key];
+        }, this);
+
+        return alt;
+    },
+    configurable: true,
+    writable: true
+});
+
 /*jshint unused: false*/
 let errorHandler = function(error, req, res, next) {
   const code = !isNaN( parseInt(error.code, 10) ) ? parseInt(error.code, 10) : 500;