update: [email protected] & fix errors

Author: aladdin-add

.eslintrc

@@ -11,7 +11,7 @@ env:
 
 rules:
   indent: [2, 2] # 2 spaces indentation
-  max-len: [2, 80, 4]
+  max-len: [2, 120, 4]
   quotes: [2, "double"]
   semi: [2, "never"]
   no-multiple-empty-lines: [2, {"max": 1}]

index.js

@@ -88,11 +88,11 @@ function printLinterOutput(res, config, webpack) {
           reportOutput = messages
         }
         var filePath = loaderUtils.interpolateName(webpack,
-            config.outputReport.filePath, {
-              content: res.results.map(function(r) {
-                return r.source
-              }).join("\n"),
-            }
+          config.outputReport.filePath, {
+            content: res.results.map(function(r) {
+              return r.source
+            }).join("\n"),
+          }
         )
         webpack.emitFile(filePath, reportOutput)
       }

package.json

@@ -28,7 +28,7 @@
   },
   "devDependencies": {
     "ava": "^0.17.0",
-    "eslint": "^3.0.0",
+    "eslint": "^4.0.0-rc.0",
     "eslint-friendly-formatter": "^2.0.4",
     "npmpub": "^3.0.1",
     "webpack": "^2.2.0"

test/autofix.js

@@ -12,33 +12,33 @@ test.before(function() {
 })
 
 test.cb("loader doesn't throw error if file ok after auto-fixing",
-function(t) {
-  t.plan(2)
-  webpack(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.false(stats.hasErrors(), "a good file doesn't give any error")
-    // console.log(stats.compilation.warnings)
-    t.false(stats.hasWarnings(), "a good file doesn't give any warning")
-    t.end()
+  function(t) {
+    t.plan(2)
+    webpack(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.false(stats.hasErrors(), "a good file doesn't give any error")
+      // console.log(stats.compilation.warnings)
+      t.false(stats.hasWarnings(), "a good file doesn't give any warning")
+      t.end()
+    })
   })
-})
 
 // remove the clone
 test.after.always(function() {

test/cache.js

@@ -79,106 +79,106 @@ test.cb("should output files to cache directory", (t) => {
 })
 
 test.cb.serial("should output json.gz files to standard cache dir by default", 
-(t) => {
-  var config = assign({}, globalConfig, {
-    output: {
-      path: t.context.directory,
-    },
-    module: {
-      loaders: [
-        {
-          test: /\.jsx?/,
-          loader: eslintLoader,
-          exclude: /node_modules/,
-          query: {
-            cache: true,
+  (t) => {
+    var config = assign({}, globalConfig, {
+      output: {
+        path: t.context.directory,
+      },
+      module: {
+        loaders: [
+          {
+            test: /\.jsx?/,
+            loader: eslintLoader,
+            exclude: /node_modules/,
+            query: {
+              cache: true,
+            },
           },
-        },
-      ],
-    },
-  })
+        ],
+      },
+    })
 
-  webpack(config, (err) => {
-    t.is(err, null)
+    webpack(config, (err) => {
+      t.is(err, null)
 
-    fs.readdir(defaultCacheDir, (err, files) => {
+      fs.readdir(defaultCacheDir, (err, files) => {
       // console.log("CACHE SETTING:", t.context.cache)
-      files = files.filter((file) => /\b[0-9a-f]{5,40}\.json\.gz\b/.test(file))
-      t.is(err, null)
-      t.true(files.length > 0)
-      t.end()
+        files = files.filter((file) => /\b[0-9a-f]{5,40}\.json\.gz\b/.test(file))
+        t.is(err, null)
+        t.true(files.length > 0)
+        t.end()
+      })
     })
   })
-})
 
 test.cb.serial(
-"should output files to standard cache dir if set to true in query", 
-(t) => {
-  var config = assign({}, globalConfig, {
-    output: {
-      path: t.context.directory,
-    },
-    module: {
-      loaders: [
-        {
-          test: /\.jsx?/,
-          loader: `${eslintLoader}?cache=true`,
-          exclude: /node_modules/,
-        },
-      ],
-    },
-  })
+  "should output files to standard cache dir if set to true in query", 
+  (t) => {
+    var config = assign({}, globalConfig, {
+      output: {
+        path: t.context.directory,
+      },
+      module: {
+        loaders: [
+          {
+            test: /\.jsx?/,
+            loader: `${eslintLoader}?cache=true`,
+            exclude: /node_modules/,
+          },
+        ],
+      },
+    })
 
-  webpack(config, (err) => {
-    t.is(err, null)
+    webpack(config, (err) => {
+      t.is(err, null)
 
-    fs.readdir(defaultCacheDir, (err, files) => {
+      fs.readdir(defaultCacheDir, (err, files) => {
       // console.log("CACHE SETTING:", t.context.cache)
-      files = files.filter((file) => /\b[0-9a-f]{5,40}\.json\.gz\b/.test(file))
+        files = files.filter((file) => /\b[0-9a-f]{5,40}\.json\.gz\b/.test(file))
 
-      t.is(err, null)
-      t.true(files.length > 0)
-      t.end()
+        t.is(err, null)
+        t.true(files.length > 0)
+        t.end()
+      })
     })
   })
-})
 
 test.cb.serial("should read from cache directory if cached file exists", 
-(t) => {
-  var config = assign({}, globalConfig, {
-    output: {
-      path: t.context.directory,
-    },
-    module: {
-      loaders: [
-        {
-          test: /\.jsx?/,
-          loader: eslintLoader,
-          exclude: /node_modules/,
-          query: {
-            cache: t.context.cache,
+  (t) => {
+    var config = assign({}, globalConfig, {
+      output: {
+        path: t.context.directory,
+      },
+      module: {
+        loaders: [
+          {
+            test: /\.jsx?/,
+            loader: eslintLoader,
+            exclude: /node_modules/,
+            query: {
+              cache: t.context.cache,
+            },
           },
-        },
-      ],
-    },
-  })
-
-  // @TODO Find a way to know if the file as correctly read without relying on
-  // Istanbul for coverage.
-  webpack(config, (err) => {
-    t.is(err, null)
+        ],
+      },
+    })
 
+    // @TODO Find a way to know if the file as correctly read without relying on
+    // Istanbul for coverage.
     webpack(config, (err) => {
       t.is(err, null)
-      fs.readdir(t.context.cache, (err, files) => {
+
+      webpack(config, (err) => {
         t.is(err, null)
-        t.true(files.length > 0)
-        t.end()
+        fs.readdir(t.context.cache, (err, files) => {
+          t.is(err, null)
+          t.true(files.length > 0)
+          t.end()
+        })
       })
     })
-  })
 
-})
+  })
 
 test.cb.serial("should have one file per module", (t) => {
   var config = assign({}, globalConfig, {
@@ -283,5 +283,5 @@ function createTestDirectory(baseDirectory, testTitle, cb) {
 }
 
 function escapeDirectory(directory) {
-  return directory.replace(/[\/?<>\\:*|"\s]/g, "_")
+  return directory.replace(/[/?<>\\:*|"\s]/g, "_")
 }

test/formatter-multiple-entries.js

@@ -5,53 +5,53 @@ var fs = require("fs")
 var path = require("path")
 
 test.cb("eslint-loader can be configured to write multiple eslint result files",
-function(t) {
-  var outputFilename = "outputReport-[name].txt"
-  var config = conf(
-    {
-      entry: [
-        "./test/fixtures/error-multi-two.js",
-        "./test/fixtures/error-multi-one.js",
-        "./test/fixtures/error-multi.js",
-      ],
-    },
-    {
-      formatter: require("eslint/lib/formatters/checkstyle"),
-      outputReport: {
-        filePath: outputFilename,
+  function(t) {
+    var outputFilename = "outputReport-[name].txt"
+    var config = conf(
+      {
+        entry: [
+          "./test/fixtures/error-multi-two.js",
+          "./test/fixtures/error-multi-one.js",
+          "./test/fixtures/error-multi.js",
+        ],
       },
-    }
-  )
+      {
+        formatter: require("eslint/lib/formatters/checkstyle"),
+        outputReport: {
+          filePath: outputFilename,
+        },
+      }
+    )
 
-  /* Plan for the success count. Failure cases are going to fail anyway so the
+    /* Plan for the success count. Failure cases are going to fail anyway so the
    * count being off for those cases doesn't matter. */
-  t.plan(config.entry.length * 2)
+    t.plan(config.entry.length * 2)
 
-  webpack(config,
-  function(err, stats) {
-    if (err) {
-      throw err
-    }
+    webpack(config,
+      function(err, stats) {
+        if (err) {
+          throw err
+        }
 
-    stats.compilation.errors.forEach(function(error) {
-      var basename = path.basename(error.module.resource, ".js")
-      var filename = config.output.path + "outputReport-" + basename + ".txt"
+        stats.compilation.errors.forEach(function(error) {
+          var basename = path.basename(error.module.resource, ".js")
+          var filename = config.output.path + "outputReport-" + basename + ".txt"
 
-      try {
-        var contents = fs.readFileSync(filename, "utf8")
+          try {
+            var contents = fs.readFileSync(filename, "utf8")
 
-        t.pass("File '" + filename + "' has been created")
-        t.is(
-          error.message,
-          contents,
-          "File '" + filename + "' Contents should equal output"
-        )
-      }
-      catch (e) {
-        t.fail("Expected file '" + filename + "' to have been created:" + e)
-      }
-    })
+            t.pass("File '" + filename + "' has been created")
+            t.is(
+              error.message,
+              contents,
+              "File '" + filename + "' Contents should equal output"
+            )
+          }
+          catch (e) {
+            t.fail("Expected file '" + filename + "' to have been created:" + e)
+          }
+        })
 
-    t.end()
+        t.end()
+      })
   })
-})