[email protected] breaks build 🚨 (#50)

* chore(package): update feathers-service-tests to version 0.6.1

* Use lodash.omit to make sure the original data object does not get modified

package.json

@@ -49,9 +49,9 @@
     "lib": "lib"
   },
   "dependencies": {
-    "babel-polyfill": "^6.3.14",
     "feathers-errors": "^2.0.1",
     "feathers-query-filters": "^1.5.1",
+    "lodash.omit": "^4.3.0",
     "uberproto": "^1.1.2"
   },
   "devDependencies": {
@@ -62,9 +62,9 @@
     "babel-preset-es2015": "^6.1.2",
     "body-parser": "^1.14.1",
     "chai": "^3.4.1",
-    "feathers": "^2.0.0-pre.4",
+    "feathers": "^2.0.0",
     "feathers-rest": "^1.3.0",
-    "feathers-service-tests": "^0.6.0",
+    "feathers-service-tests": "^0.6.1",
     "jshint": "^2.8.0",
     "mocha": "^2.5.0",
     "sequelize": "^3.14.1",

src/index.js

@@ -1,5 +1,4 @@
-if(!global._babelPolyfill) { require('babel-polyfill'); }
-
+import omit from 'lodash.omit';
 import Proto from 'uberproto';
 import filter from 'feathers-query-filters';
 import errors from 'feathers-errors';
@@ -101,9 +100,7 @@ class Service {
 
     const options = Object.assign({}, params.sequelize, { where });
 
-    delete data[this.id];
-
-    return this.Model.update(data, options)
+    return this.Model.update(omit(data, this.id), options)
       .then(() => this._getOrFind(id, params))
       .catch(utils.errorHandler);
   }
@@ -115,8 +112,6 @@ class Service {
       return Promise.reject('Not replacing multiple records. Did you mean `patch`?');
     }
 
-    delete data[this.id];
-
     return this.Model.findById(id).then(instance => {
       if(!instance) {
         throw new errors.NotFound(`No record found for id '${id}'`);