Added additional tests

Author: pandres95

lib/index.js

@@ -0,0 +1,32 @@
+'use strict';
+
+var API         = require('booljs-api')
+,   passport    = require('passport');
+
+module.exports = class BoolJSPassport extends API.Middleware {
+    constructor() {
+        super('booljs-passport', [ require.resolve('./route_middleware') ]);
+    }
+
+    action(_instance) {
+        var app             = _instance.getComponents()
+        ,   configuration   = app.configuration.get('security')
+        ,   moduleName      = (
+            configuration.passport && configuration.passport.module
+        ) || 'Passport'
+        ,   strategiesList  = (
+            configuration.passport && configuration.passport.strategies
+        ) || []
+        ,   strategies = (
+            app.dao[moduleName] && new app.dao[moduleName]()
+        ) || {};
+
+        for(var strategy of strategiesList){
+            if(strategies[strategy]){
+                passport.use(strategy, strategies[strategy](passport));
+            }
+        }
+
+        return passport.initialize();
+    }
+};

lib/route_middleware.js

@@ -0,0 +1,53 @@
+'use strict';
+
+var API         = require('booljs-api')
+,   passport    = require('passport');
+
+var usedStrategies = {};
+
+module.exports = class BoolJSPassportRoute extends API.RouteMiddleware {
+    constructor() {
+        super('booljs-passport-route', 'omittable', {
+            public: true
+        });
+    }
+
+    action(_instance, router, route) {
+        var app             = _instance.getComponents()
+        ,   configuration   = app.configuration.get('security')
+        ,   strategy        = (
+            route.authentication && route.authentication.strategy
+        ) || (
+            configuration.passport && configuration.passport.defaultStrategy
+        )
+        ,   authOptions     = (
+            route.authentication && route.authentication.options
+        ) || (
+            configuration.passport && configuration.passport.strategyOptions
+        );
+
+        if(!usedStrategies[strategy]){
+            var strategyMiddleware;
+
+            if(strategy && authOptions){
+                strategyMiddleware = passport.authenticate(
+                    strategy, authOptions
+                );
+            } else if (strategy) {
+                strategyMiddleware = passport.authenticate(strategy);
+            } else {
+                strategyMiddleware = (req, res, next) => { next(); };
+            }
+
+            usedStrategies[strategy] = strategyMiddleware;
+        }
+
+        return [
+            usedStrategies[strategy],
+            (req, res, next) => {
+                if(req.query.access_token) delete req.query.access_token;
+                next();
+            }
+        ];
+    }
+};

package.json

@@ -2,7 +2,7 @@
   "name": "booljs-passport",
   "version": "0.0.3",
   "description": "Middleware to connect Passport.js with Bool.js and a RouteMiddleware to authenticate API users through Auth Strategies.",
-  "main": "index.js",
+  "main": "lib/index.js",
   "scripts": {
     "test": "NODE_ENV=test mocha --timeout 5000 --recursive test"
   },
@@ -22,10 +22,16 @@
     "url": "https://github.com/booljs/booljs-passport/issues"
   },
   "homepage": "https://github.com/booljs/booljs-passport#readme",
+  "engines": {
+    "node": ">=4.0.0",
+    "npm": ">=3.0.0"
+  },
   "peerDependencies": {
-    "passport": "^0.3.2"
+    "passport": "^0.3.2",
+    "booljs-api": "0.3.x"
   },
   "devDependencies": {
+    "chai": "^3.5.0",
     "mocha": "^2.3.4",
     "supertest": "^1.1.0",
     "supertest-as-promised": "^2.0.2"

test/_.js

@@ -7,4 +7,5 @@ booljs('com.example.api', [
     resolver(''), 'passport-http-bearer'
 ]).setBase('example').run();
 
+global.expect = require('chai').expect;
 global.Agent = require('supertest-as-promised');

test/bearer.js

@@ -1,12 +1,5 @@
 'use strict';
 
-describe('Plugin', () => {
-
-    it('Integrity test passes', () => {
-        require('..').checkIntegrity();
-    });
-});
-
 describe('Bearer', () => {
     var booljs  = require('bool.js')
     ,   agent;

test/plugin.js

@@ -0,0 +1,9 @@
+'use strict';
+
+describe('Plugin', () => {
+
+    it('Integrity test passes', () => {
+        let Plugin = require('..');
+        new Plugin();
+    });
+});

test/user.js

@@ -0,0 +1,22 @@
+'use strict';
+
+describe('User', () => {
+    var booljs  = require('bool.js')
+    ,   agent;
+
+    before(() => {
+        return booljs('com.example.api').run().then(function (api) {
+            agent = new Agent(api.server);
+        });
+    });
+
+    it(`Unauthenticated request returns 401`, () => {
+        return (agent
+            .get('/users/me')
+            .set('Authorization', 'Bearer 123456')
+            .expect(200)
+        ).then(res => res.body).then(body => {
+            /* jshint -W030 */ expect(body.data).to.exist;
+        });
+    });
+});