Merge pull request #373 from feathers-plus/ajv-1

Revert ajv from 6.1.1 to 5.5.2. Errors testing schemas in existing ajv instances.

package.json

@@ -52,7 +52,7 @@
     "@feathersjs/commons": "^1.4.0",
     "@feathersjs/errors": "^3.3.0",
     "@feathersjs/feathers": "^3.1.3",
-    "ajv": "^6.1.1",
+    "ajv": "^5.5.2",
     "debug": "^3.1.0",
     "process": "0.11.10",
     "traverse": "^0.6.6"

tests/services/act-on-dispatch.test.js → tests-async/act-on-dispatch.test.js

@@ -1,6 +1,6 @@
 
 const { assert } = require('chai');
-const { actOnDefault, actOnDispatch, combine, getItems, replaceItems } = require('../../lib');
+const { actOnDefault, actOnDispatch, combine, getItems, replaceItems } = require('../lib');
 
 let hookBefore;
 

tests/services/act-on-dispatch-stub.test.js

@@ -0,0 +1,7 @@
+
+
+if (process.version.substr(0, 2) >= 'v8') {
+  require('../../tests-async/act-on-dispatch.test');
+} else {
+  console.log('\n...act-on-dispatch test ignored. Node version is prior to v8.');
+}

tests/services/discard.test.js

@@ -152,5 +152,32 @@ describe('services discard', () => {
         { empl: { name: { first: 'John', last: 'Doe' }, status: 'AA' }, dept: 'Acct' }
       );
     });
+
+    it('discards multiple fields', () => {
+      let hook = {
+        type: 'after',
+        method: 'get',
+        result: {
+          roles: [ 'super' ],
+          _id: 'a',
+          email: 'foo',
+          password: 'bar',
+          name: 'Rafael',
+          id: 'b',
+        },
+        query: {},
+      };
+
+      hooks.discard('email', 'password')(hook);
+
+      assert.deepEqual(hook.result, {
+        roles: [ 'super' ],
+        _id: 'a',
+        //email: 'foo',
+        //password: 'bar',
+        name: 'Rafael',
+        id: 'b',
+      });
+    });
   });
 });