Update: support normal js files

Author: mysticatea

index.js

@@ -9,6 +9,7 @@
 // Requirements
 //------------------------------------------------------------------------------
 
+const path = require("path")
 const espree = require("espree")
 const SAXParser = require("parse5").SAXParser
 
@@ -55,7 +56,23 @@ function extractFirstScript(originalText) {
 // Exports
 //------------------------------------------------------------------------------
 
+/**
+ * Parses the source code.
+ *
+ * If `options.filePath` is a `.vue` file, this extracts the first `<script>`
+ * element then parses it.
+ *
+ * @memberof module:vue-eslint-parser
+ * @function parse
+ * @param {string} text - The source code to be parsed.
+ * @param {object} options - The option object for espree.
+ * @returns {ASTNode} The AST object as the result of parsing.
+ */
 module.exports.parse = function parse(text, options) {
+    if (path.extname(options.filePath || "unknown.js") !== ".vue") {
+        return espree.parse(text, options)
+    }
+
     const script = extractFirstScript(text)
     const ast = espree.parse(script.text, options)
 

package.json

@@ -39,7 +39,7 @@
     "rimraf": "^2.5.4"
   },
   "peerDependencies": {
-    "eslint": ">=3"
+    "eslint": ">=3.5.0"
   },
   "repository": {
     "type": "git",

test/fixtures/.eslintrc.json

@@ -0,0 +1,3 @@
+{
+    "parser": "../../index.js"
+}

test/fixtures/notvue.js

@@ -0,0 +1 @@
+console.log("hello")

test/fixtures/notvue.js.fixed

@@ -0,0 +1 @@
+console.log("hello");

test/index.js

@@ -73,3 +73,47 @@ describe("About fixtures/hello.vue", () => {
         assert(actual === expected)
     })
 })
+
+describe("About fixtures/notvue.js", () => {
+    beforeEach(() => {
+        fs.copySync(ORIGINAL_FIXTURE_DIR, FIXTURE_DIR)
+    })
+    afterEach(() => {
+        fs.removeSync(FIXTURE_DIR)
+    })
+
+    it("should notify a 'semi' error", () => {
+        const cli = new CLIEngine({
+            cwd: FIXTURE_DIR,
+            envs: ["es6", "node"],
+            parser: PARSER_PATH,
+            rules: {semi: "error"},
+            useEslintrc: false,
+        })
+        const report = cli.executeOnFiles(["notvue.js"])
+        const messages = report.results[0].messages
+
+        assert(messages.length === 1)
+        assert(messages[0].ruleId === "semi")
+        assert(messages[0].line === 1)
+        assert(messages[0].column === 21)
+        assert(messages[0].source === "console.log(\"hello\")")
+    })
+
+    it("should fix a 'semi' error with --fix option", () => {
+        const cli = new CLIEngine({
+            cwd: FIXTURE_DIR,
+            envs: ["es6", "node"],
+            fix: true,
+            parser: PARSER_PATH,
+            rules: {semi: "error"},
+            useEslintrc: false,
+        })
+        CLIEngine.outputFixes(cli.executeOnFiles(["notvue.js"]))
+
+        const actual = fs.readFileSync(path.join(FIXTURE_DIR, "notvue.js"), "utf8")
+        const expected = fs.readFileSync(path.join(FIXTURE_DIR, "notvue.js.fixed"), "utf8")
+
+        assert(actual === expected)
+    })
+})