make eslint happy

Author: fgnass

lib/hook.js

@@ -22,7 +22,10 @@ module.exports = function (cfg, wrapper, callback) {
     if (!orig) return;
     obj[method] = function () {
       var opts = arguments[optionsArgIndex];
-      var file = typeof opts == 'string' ? opts : opts.filename;
+      var file = null;
+      if (opts) {
+        file = typeof opts == 'string' ? opts : opts.filename;
+      }
       if (file) callback(file);
       return orig.apply(this, arguments);
     };
@@ -32,12 +35,12 @@ module.exports = function (cfg, wrapper, callback) {
    * (Re-)install hooks for all registered file extensions.
    */
   function updateHooks() {
-    for (var ext in require.extensions) {
+    Object.keys(require.extensions).forEach(function (ext) {
       var fn = require.extensions[ext];
-      if (fn.name !== 'nodeDevHook') {
+      if (typeof fn === 'function' && fn.name !== 'nodeDevHook') {
         require.extensions[ext] = createHook(fn);
       }
-    }
+    });
   }
 
   /**

package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-dev",
-  "version": "3.1.1",
+  "version": "3.1.2",
   "description": "Restarts your app when files are modified",
   "keywords": [
     "restart",
@@ -40,7 +40,8 @@
   "devDependencies": {
     "coffee-script": "^1.8.0",
     "eslint": "^2.0.0",
-    "eslint-config-airbnb": "^8.0.0",
+    "eslint-config-airbnb-base": "^3.0.1",
+    "eslint-plugin-import": "^1.8.1",
     "tap": "^5.2.0",
     "touch": "^1.0.0"
   }

test/fixture/uncaughtExceptionHandler.js

@@ -4,5 +4,5 @@ process.on('uncaughtException', function (e) {
   }, 100);
 });
 
-/* eslint-disable no-undef */
+// eslint-disable-next-line no-undef
 foo(); // undefined / throws exception

test/index.js

@@ -1,21 +1,15 @@
-var fs = require('fs');
 var child = require('child_process');
 var test = require('tap').test;
 var touch = require('touch');
 
 var dir = __dirname + '/fixture';
 var bin = __dirname + '/../bin/node-dev';
 var msgFile = dir + '/message.js';
-var ignoredFile = dir + '/ignoredModule.js';
-
-
-// Constants
-var MESSAGE = fs.readFileSync(msgFile).toString();
 
 // Helpers
 function touchFile(file) {
   return function () {
-    touch.sync(file ? file : msgFile);
+    touch.sync(file || msgFile);
   };
 }