Fix: parserServices must exist always

Author: mysticatea

src/index.ts

@@ -56,28 +56,30 @@ export function parseForESLint(code: string, options: any): AST.ESLintExtendedPr
         tokens: true,
     }, options || {})
 
+    let result: AST.ESLintExtendedProgram
     if (!isVueFile(code, options)) {
-        return parseScript(code, options)
+        result = parseScript(code, options)
     }
+    else {
+        const tokenizer = new HTMLTokenizer(code)
+        const rootAST = new HTMLParser(tokenizer, options).parse()
+        const locationCalcurator = new LocationCalculator(tokenizer.gaps, tokenizer.lineTerminators)
+        const script = rootAST.children.find(isScriptElement) as AST.VElement | undefined // https://github.com/Microsoft/TypeScript/issues/7657
+        const template = rootAST.children.find(isTemplateElement) as AST.VElement | undefined
+        const concreteInfo: AST.HasConcreteInfo = {
+            tokens: rootAST.tokens,
+            comments: rootAST.comments,
+            errors: rootAST.errors,
+        }
 
-    const tokenizer = new HTMLTokenizer(code)
-    const rootAST = new HTMLParser(tokenizer, options).parse()
-    const locationCalcurator = new LocationCalculator(tokenizer.gaps, tokenizer.lineTerminators)
-    const script = rootAST.children.find(isScriptElement) as AST.VElement | undefined // https://github.com/Microsoft/TypeScript/issues/7657
-    const template = rootAST.children.find(isTemplateElement) as AST.VElement | undefined
-    const result = (script != null)
-        ? parseScriptElement(script, locationCalcurator, options)
-        : parseScript("", options)
-    const concreteInfo: AST.HasConcreteInfo = {
-        tokens: rootAST.tokens,
-        comments: rootAST.comments,
-        errors: rootAST.errors,
+        result = (script != null)
+            ? parseScriptElement(script, locationCalcurator, options)
+            : parseScript("", options)
+        result.ast.templateBody = (template != null)
+            ? Object.assign(template, concreteInfo)
+            : undefined
     }
-    const templateBody = (template != null)
-        ? Object.assign(template, concreteInfo)
-        : undefined
 
-    result.ast.templateBody = templateBody
     result.services = Object.assign(result.services || {}, services.define(result.ast))
 
     return result

test/index.js

@@ -13,6 +13,7 @@ const assert = require("assert")
 const path = require("path")
 const fs = require("fs-extra")
 const parse = require("..").parse
+const parseForESLint = require("..").parseForESLint
 const CLIEngine = require("./fixtures/eslint").CLIEngine
 
 //------------------------------------------------------------------------------
@@ -505,4 +506,14 @@ describe("Basic tests", () => {
             assert.equal(errors.length, 0)
         })
     })
+
+    describe("About parserServices", () => {
+        it("should exist if the source code is a Vue SFC file.", () => {
+            assert.notEqual(parseForESLint("test", {filePath: "test.vue"}).services, undefined)
+        })
+
+        it("should exist even if the source code is not Vue SFC file.", () => {
+            assert.notEqual(parseForESLint("test", {filePath: "test.js"}).services, undefined)
+        })
+    })
 })