General improvements

.eslintrc

@@ -1,11 +1,13 @@
 {
+    "extends": "eslint:recommended",
     "parserOptions": {
         "ecmaVersion": 8
     },
     "rules": {
         "quotes": [2, "single", { "allowTemplateLiterals": true }]
     },
     "env": {
-        "node": true
+        "node": true,
+        "es6": true
     }
 }

README.md

@@ -5,11 +5,11 @@ hapi.js plugin for the Sequelize ORM
 ### What's the difference between hapi-sequelize and hapi-sequelizejs?
 
 The reason of creating this fork is the inactivity of the original [hapi-sequelize](https://github.com/danecando/hapi-sequelize) plugin.
-See the [Migration guide](https://github.com/valtlfelipe/hapi-sequelizejs/wiki/Migration-guide)
+See the [Migration guide](https://github.com/valtlfelipe/hapi-sequelizejs/wiki/Migration-guide).
 
 ### Compatibility
 
-Compatible with hapi.js version `16.x` and sequelize `4.x`.
+Compatible with hapi.js version `17.x` and sequelize `4.x`.
 
 ### Installation
 

lib/models.js

@@ -20,7 +20,7 @@ async function getFiles(paths, ignored) {
 }
 
 async function load(files, fn) {
-    if (!files) return resolve([]);
+    if (!files) return Promise.resolve([]);
     if (!Array.isArray(files)) files = [files];
 
     return files.reduce((acc, file) => {

test/index.js

@@ -9,9 +9,6 @@ const Sequelize = require('sequelize');
 
 const DB = require('../lib/DB');
 
-// Module globals
-const internals = {};
-
 // Test shortcutse
 const lab = (exports.lab = Lab.script());
 const { suite, test } = lab;
@@ -231,6 +228,30 @@ suite('hapi-sequelizejs', () => {
             },
         ]);
     });
+
+    test('should call onConnect with a promise', () => {
+          const server = new Hapi.Server();
+
+          return server.register([
+              {
+                  plugin: require('../lib/'),
+                  options: [
+                      {
+                          name: 'test',
+                          sequelize: new Sequelize('test', null, null, {
+                              logging: false,
+                              dialect: 'sqlite',
+                              storage: Path.join(__dirname, 'db.sqlite'),
+                          }),
+                          onConnect: instance => {
+                              expect(instance).to.be.instanceof(DB);
+                            return Promise.resolve();
+                          },
+                      },
+                  ],
+              },
+          ]);
+      });
 
     test('should throw error on getting invalid named DB instance', async () => {
         const server = new Hapi.Server();
@@ -257,7 +278,7 @@ suite('hapi-sequelizejs', () => {
             {
                 method: 'GET',
                 path: '/',
-                handler(request, reply) {
+                handler(request) {
                     try {
                         request.getDb('inexistent');
                     } catch (err) {