Merge pull request #65 from JamesHenry/master

Update ESLint peerDependency and add support for auto-fixing

Close #64

index.js

@@ -46,6 +46,11 @@ function lint(input, config, webpack, callback) {
         })
     }
 
+    // if enabled, use eslint auto-fixing where possible
+    if (config.fix && res.results[0].output) {
+      eslint.CLIEngine.outputFixes(res)
+    }
+
     if (res.errorCount || res.warningCount) {
       // add filename for each results so formatter can have relevant filename
       res.results.forEach(function(r) {

package.json

@@ -25,7 +25,7 @@
     "index.js"
   ],
   "peerDependencies": {
-    "eslint": "^1.0.0"
+    "eslint": "^1.6.0"
   },
   "dependencies": {
     "loader-utils": "^0.2.7",

test/autofix.js

@@ -0,0 +1,49 @@
+var test = require("tape")
+var webpack = require("webpack")
+var assign = require("object-assign")
+var conf = require("./utils/conf")
+var fs = require("fs")
+
+// clone the "fixable" file, so that we do not lose the original contents
+// when the fixes are applied to disk 
+test("setup", function(t) {
+  fs
+    .createReadStream("./test/fixtures/fixable.js")
+    .pipe(fs.createWriteStream("./test/fixtures/fixable-clone.js"))
+
+  t.end()
+})
+
+test("loader doesn't throw error if file ok after auto-fixing", function(t) {
+  webpack(assign({},
+    conf,
+    {
+      entry: "./test/fixtures/fixable-clone.js",
+      module: {
+        loaders: [
+          {
+            test: /\.js$/,
+            loader: "./index?fix=true",
+            exclude: /node_modules/,
+          },
+        ],
+      },
+    }
+  ),
+  function(err, stats) {
+    if (err) {
+      throw err
+    }
+    // console.log(stats.compilation.errors)
+    t.notOk(stats.hasErrors(), "a good file doesn't give any error")
+    // console.log(stats.compilation.warnings)
+    t.notOk(stats.hasWarnings(), "a good file doesn't give any warning")
+    t.end()
+  })
+})
+
+// remove the clone
+test("teardown", function(t) {
+  fs.unlinkSync("./test/fixtures/fixable-clone.js")
+  t.end()
+})

test/fixtures/fixable.js

@@ -0,0 +1,7 @@
+'use strict';
+
+function foo() {
+return true;
+}  
+
+ foo();