style: lint the code

Author: Tom Kirkpatrick

.eslintignore

@@ -0,0 +1,4 @@
+.git/
+node_modules/
+coverage
+.nyc_output

.eslintrc

@@ -0,0 +1,4 @@
+{
+  "extends": "fullcube",
+  "root": true
+}

index.js

@@ -1,10 +1,12 @@
+/* eslint global-require: 0 */
+
 'use strict'
 
 const kebabCase = require('lodash.kebabcase')
-const fs = require('fs')
 const path = require('path')
 const debug = require('debug')('loopback:component:model-extender')
 const appRoot = require('app-root-path')
+const requireAll = require('require-all')
 
 module.exports = (app, options) => {
   options = options || {}
@@ -23,27 +25,24 @@ module.exports = (app, options) => {
 
     // Set up logging class.
     let log = options.log || console
+
     if (typeof log === 'string') {
       log = require(log)
     }
 
     // Apply models customisations.
-    function customizeModel(sourcePath) {
-      fs.readdirSync(sourcePath).forEach(sourceFile => {
-        const filePath = path.join(sourcePath, sourceFile)
-
-        if (path.extname(sourceFile) === '.js') {
-          debug('Loading customization script %s', filePath)
-          const code = require(filePath)
-
+    function customizeModel(dirname) {
+      requireAll({
+        dirname,
+        resolve: code => {
+          debug('Loading customization script %s', code)
           if (typeof code === 'function') {
             debug('Customizing model %s', Model.modelName)
-            code(Model)
-          }
-          else {
-            debug('Skipping model file %s - `module.exports` is not a function', sourceFile)
+            return code(Model)
           }
-        }
+          debug('Skipping model file %s - `module.exports` is not a function', code)
+          return code
+        },
       })
     }
 
@@ -60,7 +59,7 @@ module.exports = (app, options) => {
       // anything other than 'file not found' should be a hard fail.
       if (err.code !== 'ENOENT') {
         log.error(`Failure loading component path: ${requirePath}`, err.stack)
-        process.exit(1)
+        throw err
       }
     }
   }

package.json

@@ -4,6 +4,8 @@
   "description": "Loopback Component Model Extender",
   "main": "index.js",
   "scripts": {
+    "lint": "eslint .",
+    "pretest": "npm run lint",
     "test": "nyc  --reporter=lcov --reporter=text --reporter=text-summary mocha test/*test.js",
     "test:watch": "npm run test -- -w"
   },
@@ -28,10 +30,13 @@
   "devDependencies": {
     "chai": "3.5.0",
     "cors": "2.8.3",
+    "eslint": "3.19.0",
+    "eslint-config-fullcube": "2.0.35-latest-20170602-0428.0",
     "loopback": "3.8.0",
     "loopback-boot": "2.24.0",
     "mocha": "3.1.2",
     "nyc": "11.0.2",
+    "require-all": "2.2.0",
     "strong-error-handler": "2.1.0"
   }
 }

test/.eslintrc

@@ -0,0 +1,4 @@
+{
+  "extends": "fullcube/mocha",
+  "root": true
+}

test/test-server/server/server.js

@@ -1,29 +1,26 @@
-'use strict';
+'use strict'
 
-var loopback = require('loopback');
-var boot = require('loopback-boot');
+const loopback = require('loopback')
+const boot = require('loopback-boot')
 
-var app = module.exports = loopback();
+const app = module.exports = loopback()
 
 app.start = function() {
   // start the web server
   return app.listen(function() {
-    app.emit('started');
-    var baseUrl = app.get('url').replace(/\/$/, '');
-    console.log('Web server listening at: %s', baseUrl);
-    if (app.get('loopback-component-explorer')) {
-      var explorerPath = app.get('loopback-component-explorer').mountPath;
-      console.log('Browse your REST API at %s%s', baseUrl, explorerPath);
-    }
-  });
-};
+    app.emit('started')
+  })
+}
 
 // Bootstrap the application, configure models, datasources and middleware.
 // Sub-apps like REST API are mounted via boot scripts.
 boot(app, __dirname, function(err) {
-  if (err) throw err;
+  if (err) {
+    throw err
+  }
 
   // start the server if `$ node server.js`
-  if (require.main === module)
-    app.start();
-});
+  if (require.main === module) {
+    app.start()
+  }
+})