Add more tests for a testing guide

.gitignore

@@ -108,5 +108,6 @@ $RECYCLE.BIN/
 *.lnk
 
 # Others
+test/data/
 data/
 tmp/

config/test.json

@@ -0,0 +1,3 @@
+{
+  "nedb": "../test/data"
+}

package.json

@@ -25,7 +25,8 @@
     "test": "npm run eslint && npm run mocha",
     "eslint": "eslint src/. test/. --config .eslintrc.json",
     "start": "node src/",
-    "mocha": "mocha test/ --recursive --exit"
+    "clean": "shx rm -rf test/data/",
+    "mocha": "npm run clean && NODE_ENV=test mocha test/ --recursive --exit"
   },
   "dependencies": {
     "@feathersjs/authentication": "^2.0.1",
@@ -49,6 +50,7 @@
     "feathers-memory": "^2.1.0",
     "mocha": "^4.0.1",
     "request": "^2.83.0",
-    "request-promise": "^4.2.2"
+    "request-promise": "^4.2.2",
+    "shx": "^0.2.2"
   }
 }

src/services/messages/messages.hooks.js

@@ -6,7 +6,7 @@ const populateUser = require('../../hooks/populate-user');
 
 module.exports = {
   before: {
-    all: [ authenticate('jwt') ],
+    all: [authenticate('jwt')],
     find: [],
     get: [],
     create: [processMessage()],

test/services/messages.test.js

@@ -7,4 +7,27 @@ describe('\'messages\' service', () => {
 
     assert.ok(service, 'Registered the service');
   });
+
+  it('creates and processes message, adds user information', async () => {
+    // Create a new user we can use for testing
+    const user = await app.service('users').create({
+      email: 'messagetest@example.com',
+      password: 'supersecret'
+    });
+
+    // The messages service call params (with the user we just created)
+    const params = {user };
+    const message = await app.service('messages').create({
+      text: 'a test',
+      additional: 'should be removed'
+    }, params);
+
+    assert.equal(message.text, 'a test');
+    // `userId` should be set to passed users it
+    assert.equal(message.userId, user._id);
+    // Additional property has been removed
+    assert.ok(!message.additional);
+    // `user` has been populated
+    assert.deepEqual(message.user, user);
+  });
 });

test/services/users.test.js

@@ -7,4 +7,29 @@ describe('\'users\' service', () => {
 
     assert.ok(service, 'Registered the service');
   });
+
+  it('creates a user, encrypts password and adds gravatar', async () => {
+    const user = await app.service('users').create({
+      email: 'test@example.com',
+      password: 'secret'
+    });
+
+    // Verify Gravatar has been set to what we'd expect
+    assert.equal(user.avatar, 'https://s.gravatar.com/avatar/55502f40dc8b7c769880b10874abc9d0?s=60');
+    // Makes sure the password got encrypted
+    assert.ok(user.password !== 'secret');
+  });
+
+  it('removes password for external requests', async () => {
+    // Setting `provider` indicates an external request
+    const params = { provider: 'rest' };
+
+    const user = await app.service('users').create({
+      email: 'test2@example.com',
+      password: 'secret'
+    }, params);
+
+    // Make sure password has been remove
+    assert.ok(!user.password);
+  });
 });