Merge pull request #193 from Unleash/add-app-hooks

add app hooks

lib/app.js

@@ -23,6 +23,11 @@ module.exports = function (config) {
     app.set('trust proxy');
     app.set('port', config.port);
     app.locals.baseUriPath = baseUriPath;
+
+    if (typeof config.preHook === 'function') {
+        config.preHook(app);
+    }
+
     app.use(cookieParser());
 
     if (publicFolder) {
@@ -49,6 +54,10 @@ module.exports = function (config) {
         }));
     }
 
+    if (typeof config.preRouterHook === 'function') {
+        config.preRouterHook(app);
+    }
+
     // Setup API routes
     const apiRouter = express.Router(); // eslint-disable-line new-cap
     routes.createAPI(apiRouter, config);

lib/app.test.js

@@ -0,0 +1,31 @@
+'use strict';
+
+const test = require('ava');
+const proxyquire = require('proxyquire');
+const getApp = proxyquire('./app', {
+    './routes': {
+        createAPI: () => {},
+        createLegacy: () => {},
+    },
+});
+
+test('should not throw when valid config', t => {
+    const app = getApp({});
+    t.true(typeof app.listen === 'function');
+});
+
+test('should call preHook', t => {
+    let called = 0;
+    getApp({ preHook: () => {
+        called++;
+    } });
+    t.true(called === 1);
+});
+
+test('should call preRouterHook', t => {
+    let called = 0;
+    getApp({ preRouterHook: () => {
+        called++;
+    } });
+    t.true(called === 1);
+});

package.json

@@ -85,6 +85,7 @@
     "eslint": "^3.11.1",
     "eslint-config-finn": "^1.0.0-beta.1",
     "nyc": "^9.0.1",
+    "proxyquire": "^1.7.10",
     "sinon": "^1.17.5",
     "superagent": "^2.3.0",
     "supertest": "^2.0.1",