Cleanup for #119

lib/private.js

@@ -3,20 +3,25 @@
 var Basic = require('hapi-auth-basic');
 var Users = require('./users.json');
 
+
 // Declare internals
 
 var internals = {};
 
+
 internals.validateFunc = function (username, password, callback) {
 
     var user = Users[username];
     if (!user || user.password !== password) {
         return callback(null, false);
     }
 
+    user.username = username;
+
     return callback(null, true, user);
 };
 
+
 exports.register = function (server, options, next) {
 
     server.register(Basic, function (err) {
@@ -25,12 +30,12 @@ exports.register = function (server, options, next) {
             return next(err);
         }
 
-        server.auth.strategy('simple', 'basic', { validateFunc: internals.validateFunc });
+        server.auth.strategy('basic', 'basic', { validateFunc: internals.validateFunc });
         server.route({
             method: 'GET',
             path: '/private',
             config: {
-                auth: 'simple',
+                auth: 'basic',
                 description: 'Returns a greeting message to the authenticated user',
                 handler: function (request, reply) {
 
@@ -44,6 +49,7 @@ exports.register = function (server, options, next) {
     });
 };
 
+
 exports.register.attributes = {
     name: 'Private'
 };

lib/users.json

@@ -1,10 +1,8 @@
 {
-	"Foo": {
-		"username": "Foo",
-		"password": "mysupersecuredpassword"
+	"foo": {
+		"password": "foo"
 	},
-	"Bar": {
-		"username": "Bar",
-		"password": "mysupersecuredpassword2"
+	"bar": {
+		"password": "bar"
 	}
 }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "hueniversity",
-  "version": "0.0.3",
+  "version": "0.0.4",
   "description": "Community learning experiment",
   "main": "lib/index.js",
   "repository": {
@@ -23,7 +23,7 @@
     "hoek": "2.x.x"
   },
   "scripts": {
-    "test": "node node_modules/lab/bin/lab -a code -t 100 -Lv --lint-errors-threshold 0 --lint-warnings-threshold 0",
+    "test": "node node_modules/lab/bin/lab -a code -t 100 -L -v",
     "start": "node lib/start.js"
   },
   "devDependencies": {

test/index.js

@@ -49,7 +49,7 @@ it('handles register plugin errors', { parallel: false }, function (done) {
         name: 'fake version'
     };
 
-    Hueniversity.init(0, function (err) {
+    Hueniversity.init(0, function (err, server) {
 
         expect(err).to.exist();
         expect(err.message).to.equal('register version failed');

test/private.js

@@ -6,6 +6,7 @@ var Hueniversity = require('../lib');
 var Users = require('../lib/users.json');
 var Basic = require('hapi-auth-basic');
 
+
 // Declare internals
 
 var internals = {};
@@ -27,24 +28,24 @@ describe('/private', function () {
 
             expect(err).to.not.exist();
 
-            var request = { method: 'GET', url: '/private', headers: { authorization: internals.header(Users.Foo.username, Users.Foo.password) } };
+            var request = { method: 'GET', url: '/private', headers: { authorization: internals.header('foo', Users.foo.password) } };
             server.inject(request, function (res) {
 
                 expect(res.statusCode, 'Status code').to.equal(200);
-                expect(res.result, 'result').to.equal('<div>Hello Foo</div>');
+                expect(res.result, 'result').to.equal('<div>Hello foo</div>');
 
                 server.stop(done);
             });
         });
     });
 
-    it('returns error on wrong password', function (done) {
+    it('errors on wrong password', function (done) {
 
         Hueniversity.init(0, function (err, server) {
 
             expect(err).to.not.exist();
 
-            var request = { method: 'GET', url: '/private', headers: { authorization: internals.header(Users.Foo.username, '') } };
+            var request = { method: 'GET', url: '/private', headers: { authorization: internals.header('foo', '') } };
             server.inject(request, function (res) {
 
                 expect(res.statusCode, 'Status code').to.equal(401);
@@ -54,7 +55,7 @@ describe('/private', function () {
         });
     });
 
-    it('returns error on failed auth', function (done) {
+    it('errors on failed auth', function (done) {
 
         Hueniversity.init(0, function (err, server) {
 
@@ -70,14 +71,16 @@ describe('/private', function () {
         });
     });
 
-    it('returns error on failed registering of auth', { parallel: false }, function (done) {
+    it('errors on failed registering of auth', { parallel: false }, function (done) {
 
         var orig = Basic.register;
+
         Basic.register = function (plugin, options, next) {
 
             Basic.register = orig;
             return next(new Error('fail'));
         };
+
         Basic.register.attributes = {
             name: 'fake hapi-auth-basic'
         };
@@ -91,6 +94,7 @@ describe('/private', function () {
     });
 });
 
+
 internals.header = function (username, password) {
 
     return 'Basic ' + (new Buffer(username + ':' + password, 'utf8')).toString('base64');