[email protected] breaks build 🚨 (#30)

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

* Use lodash.omit which creates a new object for data

package.json

@@ -45,9 +45,9 @@
     "npm": ">=1.4.0"
   },
   "dependencies": {
-    "babel-polyfill": "^6.7.4",
     "feathers-errors": "^2.0.1",
     "feathers-query-filters": "^1.5.1",
+    "lodash.omit": "^4.3.0",
     "uberproto": "^1.2.0"
   },
   "devDependencies": {
@@ -60,7 +60,7 @@
     "chai": "^3.0.0",
     "feathers": "^2.0.0-pre.4",
     "feathers-rest": "^1.3.0",
-    "feathers-service-tests": "^0.6.0",
+    "feathers-service-tests": "^0.6.1",
     "feathers-socketio": "^1.3.3",
     "jshint": "^2.8.0",
     "mocha": "^2.5.0",

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';
@@ -110,13 +109,10 @@ class Service {
   patch(id, data, params) {
     let { query, options } = multiOptions(id, this.id, params);
 
-    // We can not update the id
-    delete data[this.id];
-    delete data._id;
-
     // Run the query
-    return nfcall(this.Model, 'update', query, { $set: data }, options)
-      .then(() => this._findOrGet(id, params));
+    return nfcall(this.Model, 'update', query, {
+      $set: omit(data, this.id, '_id')
+    }, options).then(() => this._findOrGet(id, params));
   }
 
   update(id, data, params) {
@@ -125,12 +121,8 @@ class Service {
     }
 
     let { query, options } = multiOptions(id, this.id, params);
-
-    // We can not update the id
-    data[this.id] = id;
-    delete data._id;
-
-    return nfcall(this.Model, 'update', query, data, options)
+
+    return nfcall(this.Model, 'update', query, omit(data, this.id, '_id'), options)
       .then(() => this._findOrGet(id));
   }