fix/add tests for node 6

.travis.yml

@@ -1,4 +1,5 @@
 language: node_js
 node_js:
+  - "6"
   - "5"
   - "4"

index.js

@@ -23,9 +23,9 @@ exports.register = (server, config, next) => {
       value.options.validateFunc = (request, session, callback) => {
         const method = _.get(server.methods, validateFn);
         if (!method) {
-          const msg = `could not find validate method ${validateFn} in server.methods`;
-          server.log(['error', 'hapi-strategy-loader'], msg);
-          return;
+          const err = new Error(`could not find validate method ${validateFn} in server.methods`);
+          server.log(['error', 'hapi-strategy-loader'], err);
+          return callback(err);
         }
         method(request, session, callback);
       };

package.json

@@ -4,7 +4,7 @@
   "description": "plugin to configure auth strategies for a hapi server",
   "main": "index.js",
   "scripts": {
-    "test": "lab -e test",
+    "test": "lab -e test --leaks",
     "lint": "eslint ."
   },
   "repository": {
@@ -20,9 +20,9 @@
   "devDependencies": {
     "bell": "^7.7.2",
     "code": "^2.1.0",
-    "eslint": "^3.4.0",
+    "eslint": "^3.9.1",
     "eslint-config-firstandthird": "^3.0.2",
-    "eslint-plugin-import": "^1.14.0",
+    "eslint-plugin-import": "^2.1.0",
     "hapi": "^13.4.1",
     "hapi-auth-cookie": "^6.1.1",
     "hoek": "^4.0.0",

test/server-test.js

@@ -259,10 +259,12 @@ lab.test('log and throw error if a method is not available', (done) => {
       }
     });
     server.route({
-      method: 'GET', path: '/resource', handler: (request, reply) => {
+      method: 'GET',
+      path: '/resource',
+      handler: (request, reply) => {
+        code.expect(request.auth.isAuthenticated).to.not.equal(true);
         code.expect(request.auth.error).to.exist;
-        code.expect(request.auth.error.data).to.include('gorbachev');
-        done();
+        reply(request.auth.error);
       }
     });
     server.inject('/login/valid', (res) => {
@@ -271,7 +273,11 @@ lab.test('log and throw error if a method is not available', (done) => {
       code.expect(header.length).to.equal(1);
       code.expect(header[0]).to.contain('Max-Age=60');
       const cookie = header[0].match(/(?:[^\x00-\x20\(\)<>@\,;\:\\"\/\[\]\?\=\{\}\x7F]+)\s*=\s*(?:([^\x00-\x20\"\,\;\\\x7F]*))/);
-      server.inject({ method: 'GET', url: '/resource', headers: { cookie: `${mainCookie}=${cookie[1]}` } }, (res2) => {});
+      server.inject({ method: 'GET', url: '/resource', headers: { cookie: `${mainCookie}=${cookie[1]}` } }, (response2) => {
+        code.expect(response2.result.statusCode).to.equal(401);
+        code.expect(response2.result.message).to.include('Invalid cookie');
+        done();
+      });
     });
   });
 });