Configure the linter

.editorconfig

@@ -0,0 +1,15 @@
+root = true
+
+[*]
+indent_style = space
+indent_size = 2
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true
+
+[*.md]
+trim_trailing_whitespace = false
+
+[Makefile]
+indent_style = tab

.eslintrc

@@ -0,0 +1,3 @@
+{
+  "extends": "tschaub"
+}

appveyor.yml

@@ -1,6 +1,5 @@
 environment:
   matrix:
-    - nodejs_version: "0.8"
     - nodejs_version: "0.10"
     - nodejs_version: "1.0"
 
@@ -13,7 +12,7 @@ test_script:
   - set PATH=%APPDATA%\npm;%PATH%
   - npm --version
   - npm install
-  - node ./tasks.js test
+  - npm test
 
 build: "off"
 

benchmarks/write-mock.js

@@ -1,5 +1,4 @@
 var fs = require('fs');
-var path = require('path');
 
 var mock = require('..');
 

lib/binding.js

@@ -47,7 +47,6 @@ function notImplemented() {
 }
 
 
-
 /**
  * Create a new stats object.
  * @param {Object} config Stats properties.
@@ -126,7 +125,6 @@ Stats.prototype.isSocket = function() {
 };
 
 
-
 /**
  * Create a new binding with the given file system.
  * @param {FileSystem} system Mock file system.

lib/descriptor.js

@@ -3,7 +3,6 @@
 var constants = process.binding('constants');
 
 
-
 /**
  * Create a new file descriptor.
  * @param {number} flags Flags.

lib/directory.js

@@ -1,14 +1,12 @@
 'use strict';
 
-var path = require('path');
 var util = require('util');
 
 var Item = require('./item');
 
 var constants = process.binding('constants');
 
 
-
 /**
  * A directory.
  * @constructor

lib/error.js

@@ -245,7 +245,6 @@ var codes = {
 };
 
 
-
 /**
  * Create an error.
  * @param {string} code Error code.

lib/file.js

@@ -1,6 +1,5 @@
 'use strict';
 
-var path = require('path');
 var util = require('util');
 
 var Item = require('./item');
@@ -9,7 +8,6 @@ var EMPTY = new Buffer(0);
 var constants = process.binding('constants');
 
 
-
 /**
  * A directory.
  * @constructor

lib/filesystem.js

@@ -28,7 +28,6 @@ function getPathParts(filepath) {
 }
 
 
-
 /**
  * Create a new file system.
  * @constructor

lib/index.js

@@ -108,13 +108,13 @@ exports.fs = function(config) {
   var binding = new Binding(system);
 
   // inject the mock binding
-  var mockFs = rewire(path.join(__dirname, '..', 'node', fsName));
-  mockFs.__set__('binding', binding);
+  var newMockFs = rewire(path.join(__dirname, '..', 'node', fsName));
+  newMockFs.__set__('binding', binding);
 
   // overwrite fs.Stats from original binding
-  mockFs.Stats = binding.Stats;
+  newMockFs.Stats = binding.Stats;
 
-  return mockFs;
+  return newMockFs;
 };
 
 

lib/item.js

@@ -1,7 +1,5 @@
 'use strict';
 
-var path = require('path');
-
 var counter = 0;
 
 
@@ -30,7 +28,6 @@ function getGid() {
 }
 
 
-
 /**
  * A filesystem item.
  * @constructor

lib/symlink.js

@@ -1,14 +1,12 @@
 'use strict';
 
-var path = require('path');
 var util = require('util');
 
 var Item = require('./item');
 
 var constants = process.binding('constants');
 
 
-
 /**
  * A directory.
  * @constructor

package.json

@@ -25,17 +25,18 @@
   },
   "license": "MIT",
   "scripts": {
-    "debug": "node --debug-brk ./tasks.js test",
-    "test": "node ./tasks.js lint test",
+    "pretest": "eslint benchmarks lib test",
+    "test": "mocha --recursive test",
     "bench": "bench benchmarks"
   },
   "devDependencies": {
-    "chai": "~1.8.1",
-    "jshint": "~2.4.1",
-    "mocha": "~1.17.0",
+    "bench-it": "^0.3.0",
+    "chai": "^3.0.0",
+    "eslint": "^0.23.0",
+    "eslint-config-tschaub": "^1.0.0",
     "glob": "~3.2.8",
-    "rimraf": "~2.2.6",
-    "bench-it": "~0.1.0"
+    "mocha": "^2.2.5",
+    "rimraf": "~2.2.6"
   },
   "dependencies": {
     "rewire": "~2.0.0",

test/helper.js

@@ -4,7 +4,7 @@ var chai = require('chai');
 
 
 /** @type {boolean} */
-chai.Assertion.includeStack = true;
+chai.config.includeStack = true;
 
 
 /**
@@ -60,5 +60,7 @@ exports.flags = function(str) {
     case 'xa+':
       return constants.O_APPEND | constants.O_CREAT | constants.O_RDWR |
           constants.O_EXCL;
+    default:
+      throw new Error('Unsupported flag: ' + str);
   }
 };

test/integration/filecount.js

@@ -13,6 +13,9 @@ function numFiles(dir, items, callback) {
   }
   items.forEach(function(item) {
     fs.stat(path.join(dir, item), function(err, stats) {
+      if (err) {
+        return callback(err);
+      }
       if (stats && stats.isFile()) {
         ++files;
       }

test/integration/filecount.spec.js

@@ -1,3 +1,4 @@
+/* eslint-env mocha */
 'use strict';
 
 var mock = require('../../lib/index');