Fix and tests for patch to allow NeDB modifiers (#72)

src/index.js

@@ -154,11 +154,18 @@ class Service {
             [this.id]: { $in: idList }
           }
         });
+        const updateData = { $set: {} };
 
-        return nfcall(this.Model, 'update', query, {
-          $set: omit(data, this.id, '_id')
-        }, options)
-        .then(() => this._findOrGet(id, findParams));
+        Object.keys(data).forEach(key => {
+          if (key.indexOf('$') === 0) {
+            updateData[key] = data[key];
+          } else if (key !== '_id' && key !== this.id) {
+            updateData.$set[key] = data[key];
+          }
+        });
+
+        return nfcall(this.Model, 'update', query, updateData, options)
+          .then(() => this._findOrGet(id, findParams));
       })
       .then(select(params, this.id));
   }

test/index.test.js

@@ -107,6 +107,19 @@ describe('NeDB Service', function () {
         assert.deepEqual(updated.data, ['first', 'second'])
       );
     });
+
+    it('allows NeDB modifiers in patch (#65)', () => {
+      const service = app.service('people');
+
+      return service.create({
+        name: 'Modifier',
+        data: [ 'first' ]
+      }).then(person =>
+        service.patch(person._id, { $push: { data: 'second' } })
+      ).then(updated =>
+        assert.deepEqual(updated.data, ['first', 'second'])
+      );
+    });
   });
 });