diff --git a/conf/config-schema.js b/conf/config-schema.js index 626e1d54c52..a32e5cd6194 100644 --- a/conf/config-schema.js +++ b/conf/config-schema.js @@ -11,6 +11,7 @@ const baseConfigProperties = { parser: { type: ["string", "null"] }, parserOptions: { type: "object" }, plugins: { type: "array" }, + processor: { type: "string" }, rules: { type: "object" }, settings: { type: "object" }, diff --git a/docs/developer-guide/working-with-plugins.md b/docs/developer-guide/working-with-plugins.md index 83f107512d0..6c05bbf83fc 100644 --- a/docs/developer-guide/working-with-plugins.md +++ b/docs/developer-guide/working-with-plugins.md @@ -54,15 +54,16 @@ You can also create plugins that would tell ESLint how to process files other th ```js module.exports = { processors: { - - // assign to the file extension you want (.js, .jsx, .html, etc.) - ".ext": { + "processor-name": { // takes text of the file and filename preprocess: function(text, filename) { // here, you can strip out any non-JS content // and split into multiple strings to lint - return [string]; // return an array of strings to lint + return [ // return an array of code blocks to lint + { text: code1, filename: "0.js" }, + { text: code2, filename: "1.js" }, + ]; }, // takes a Message[][] and filename @@ -72,7 +73,7 @@ module.exports = { // to the text that was returned in array from preprocess() method // you need to return a one-dimensional array of the messages you want to keep - return messages[0]; + return [].concat(...messages); }, supportsAutofix: true // (optional, defaults to false) @@ -81,9 +82,13 @@ module.exports = { }; ``` -The `preprocess` method takes the file contents and filename as arguments, and returns an array of strings to lint. The strings will be linted separately but still be registered to the filename. It's up to the plugin to decide if it needs to return just one part, or multiple pieces. For example in the case of processing `.html` files, you might want to return just one item in the array by combining all scripts, but for `.md` file where each JavaScript block might be independent, you can return multiple items. +**The `preprocess` method** takes the file contents and filename as arguments, and returns an array of code blocks to lint. The code blocks will be linted separately but still be registered to the filename. + +A code block has two properties `text` and `filename`; the `text` property is the content of the block and the `filename` property is the name of the block. ESLint checks the file extension of the `filename` property to know the kind of the code block. If the file extension was included in [`--ext` CLI option](../user-guide/command-line-interface.md#--ext), ESLint resolves `overrides` configs by the `filename` property then lint the code block. Otherwise, ESLint just ignores the code block. + +It's up to the plugin to decide if it needs to return just one part, or multiple pieces. For example in the case of processing `.html` files, you might want to return just one item in the array by combining all scripts, but for `.md` file where each JavaScript block might be independent, you can return multiple items. -The `postprocess` method takes a two-dimensional array of arrays of lint messages and the filename. Each item in the input array corresponds to the part that was returned from the `preprocess` method. The `postprocess` method must adjust the locations of all errors to correspond to locations in the original, unprocessed code, and aggregate them into a single flat array and return it. +**The `postprocess` method** takes a two-dimensional array of arrays of lint messages and the filename. Each item in the input array corresponds to the part that was returned from the `preprocess` method. The `postprocess` method must adjust the locations of all errors to correspond to locations in the original, unprocessed code, and aggregate them into a single flat array and return it. Reported problems have the following location information: @@ -117,6 +122,41 @@ By default, ESLint will not perform autofixes when a processor is used, even whe You can have both rules and processors in a single plugin. You can also have multiple processors in one plugin. To support multiple extensions, add each one to the `processors` element and point them to the same object. +#### Specifying Processor in Config Files + +People use processors by the `processor` key with a processor ID in config files. The processor ID is the concatenated string of a plugin name and a processor name by a slash. And people can use the `overrides` property to tie a file type and a processor. + +For example: + +```yml +plugins: + - a-plugin +overrides: + - files: "*.md" + processor: a-plugin/markdown +``` + +See [Specifying Processor](../user-guide/configuring.md#specifying-processor) for details. + +#### File Extension-named Processor + +If a processor name starts with `.`, ESLint handles the processor as a **file extension-named processor** especially and applies the processor to the kind of files automatically. People don't need to specify the file extension-named processors in their config files. + +For example: + +```js +module.exports = { + processors: { + // This processor will be applied to `*.md` files automatically. + // Also, people can use this processor as "plugin-id/.md" explicitly. + ".md": { + preprocess(text, filename) { /* ... */ }, + postprocess(messageLists, filename) { /* ... */ } + } + } +} +``` + ### Configs in Plugins You can bundle configurations inside a plugin by specifying them under the `configs` key. This can be useful when you want to provide not just code style, but also some custom rules to support it. Multiple configurations are supported per plugin. Note that it is not possible to specify a default configuration for a given plugin and that users must specify in their configuration file when they want to use one. diff --git a/docs/user-guide/configuring.md b/docs/user-guide/configuring.md index ff8021b6459..7cc48c8a418 100644 --- a/docs/user-guide/configuring.md +++ b/docs/user-guide/configuring.md @@ -80,6 +80,55 @@ The following parsers are compatible with ESLint: Note when using a custom parser, the `parserOptions` configuration property is still required for ESLint to work properly with features not in ECMAScript 5 by default. Parsers are all passed `parserOptions` and may or may not use them to determine which features to enable. +## Specifying Processor + +Plugins may provide processors. Processors can extract JavaScript code from another kind of files, then lets ESLint lint the JavaScript code. Or processors can convert JavaScript code in preprocessing for some purpose. + +To specify processors in a configuration file, use the `processor` key with the concatenated string of a plugin name and a processor name by a slash. For example, the following enables the processor `a-processor` that the plugin `a-plugin` provided: + +```json +{ + "plugins": ["a-plugin"], + "processor": "a-plugin/a-processor" +} +``` + +To specify processors for a specific kind of files, use the combination of the `overrides` key and the `processor` key. For example, the following uses the processor `a-plugin/markdown` for `*.md` files. + +```json +{ + "plugins": ["a-plugin"], + "overrides": [ + { + "files": ["*.md"], + "processor": "a-plugin/markdown" + } + ] +} +``` + +Processors may make named code blocks such as `0.js` and `1.js`. ESLint handles such a named code block as like a child file of the original file. You can specify additional configurations for named code blocks by the `overrides` key. For example, the following disables `strict` rule for the named code blocks which end with `.js` in markdown files. + +```json +{ + "plugins": ["a-plugin"], + "overrides": [ + { + "files": ["*.md"], + "processor": "a-plugin/markdown" + }, + { + "files": ["**/*.md/*.js"], + "rules": { + "strict": "off" + } + } + ] +} +``` + +ESLint checks the file extension of named code blocks then ignores those if [`--ext` CLI option](../user-guide/command-line-interface.md#--ext) didn't include the file extension. Be sure to specify the `--ext` option if you wanted to lint named code blocks other than `*.js`. + ## Specifying Environments An environment defines global variables that are predefined. The available environments are: diff --git a/lib/cli-engine.js b/lib/cli-engine.js index b97772133ed..28c7b9e38a1 100644 --- a/lib/cli-engine.js +++ b/lib/cli-engine.js @@ -201,6 +201,7 @@ function calculateStatsPerRun(results) { * @param {boolean} config.fix If `true` then it does fix. * @param {boolean} config.allowInlineConfig If `true` then it uses directive comments. * @param {boolean} config.reportUnusedDisableDirectives If `true` then it reports unused `eslint-disable` comments. + * @param {RegExp} config.extRegExp The `RegExp` object that tests if a file path has the allowed file extensions. * @param {Linter} config.linter The linter instance to verify. * @returns {LintResult} The result of linting. * @private @@ -213,6 +214,7 @@ function verifyText({ fix, allowInlineConfig, reportUnusedDisableDirectives, + extRegExp, linter }) { const filePath = providedFilePath || ""; @@ -232,7 +234,8 @@ function verifyText({ allowInlineConfig, filename: filePathToVerify, fix, - reportUnusedDisableDirectives + reportUnusedDisableDirectives, + extRegExp } ); @@ -771,6 +774,7 @@ class CLIEngine { fix, allowInlineConfig, reportUnusedDisableDirectives, + extRegExp: fileEnumerator.extRegExp, linter }); @@ -815,6 +819,7 @@ class CLIEngine { executeOnText(text, filename, warnIgnored) { const { configArrayFactory, + fileEnumerator, ignoredPaths, lastConfigArrays, linter, @@ -858,6 +863,7 @@ class CLIEngine { fix, allowInlineConfig, reportUnusedDisableDirectives, + extRegExp: fileEnumerator.extRegExp, linter })); } diff --git a/lib/cli-engine/config-array-factory.js b/lib/cli-engine/config-array-factory.js index 1ee5902f7b1..9bece1e402d 100644 --- a/lib/cli-engine/config-array-factory.js +++ b/lib/cli-engine/config-array-factory.js @@ -523,7 +523,7 @@ class ConfigArrayFactory { parser: parserName, parserOptions, plugins: pluginList, - processor, // processor is only for file extension processors. + processor, root, rules, settings, diff --git a/lib/cli-engine/file-enumerator.js b/lib/cli-engine/file-enumerator.js index d796a6755b9..5382028b487 100644 --- a/lib/cli-engine/file-enumerator.js +++ b/lib/cli-engine/file-enumerator.js @@ -217,6 +217,14 @@ class FileEnumerator { }); } + /** + * The `RegExp` object that tests if a file path has the allowed file extensions. + * @type {RegExp} + */ + get extRegExp() { + return internalSlotsMap.get(this).extRegExp; + } + /** * Iterate files which are matched by given glob patterns. * @param {string|string[]} patternOrPatterns The glob patterns to iterate files. diff --git a/lib/config/config-validator.js b/lib/config/config-validator.js index d37a50f1c13..b6d145c37b2 100644 --- a/lib/config/config-validator.js +++ b/lib/config/config-validator.js @@ -215,6 +215,19 @@ function validateGlobals(globalsConfig, source = null) { }); } +/** + * Validate `processor` configuration. + * @param {string|undefined} processor The processor name. + * @param {string} source The name of config file. + * @param {function(id:string): Processor} getProcessor The getter of defined processors. + * @returns {void} + */ +function validateProcessor(processor, source, getProcessor) { + if (processor && !getProcessor(processor)) { + throw new Error(`ESLint configuration of processor in '${source}' is invalid: '${processor}' was not found.`); + } +} + /** * Formats an array of schema validation errors. * @param {Array} errors An array of error messages to format. @@ -308,6 +321,7 @@ const validated = new WeakSet(); */ function validateConfigArray(configArray) { const getPluginEnv = Map.prototype.get.bind(configArray.pluginEnvironments); + const getPluginProcessor = Map.prototype.get.bind(configArray.pluginProcessors); const getPluginRule = Map.prototype.get.bind(configArray.pluginRules); // Validate. @@ -319,6 +333,7 @@ function validateConfigArray(configArray) { validateEnvironment(element.env, element.name, getPluginEnv); validateGlobals(element.globals, element.name); + validateProcessor(element.processor, element.name, getPluginProcessor); validateRules(element.rules, element.name, getPluginRule); } } diff --git a/lib/linter.js b/lib/linter.js index f2bb7425932..c691112da37 100644 --- a/lib/linter.js +++ b/lib/linter.js @@ -1003,6 +1003,7 @@ class Linter { * @param {function(Array): LintMessage[]} [filenameOrOptions.postprocess] postprocessor for report messages. If provided, * this should accept an array of the message lists for each code block returned from the preprocessor, * apply a mapping to the messages as appropriate, and return a one-dimensional array of messages + * @param {boolean} [filenameOrOptions.disableFixes] if `true` then the linter doesn't make `fix` properties into the lint result. * @returns {LintMessage[]} The results as an array of messages or an empty array if no messages. */ verify(textOrSourceCode, config, filenameOrOptions) { @@ -1034,17 +1035,16 @@ class Linter { * @returns {LintMessage[]} The found problems. */ _verifyWithConfigArray(text, configArray, providedOptions) { - debug("Verify with ConfigArray"); + debug("Verify with ConfigArray: %s", providedOptions.filename); // Store the config array in order to get plugin envs and rules later. internalSlotsMap.get(this).lastConfigArray = configArray; - /* - * TODO: implement https://github.com/eslint/rfcs/tree/master/designs/2018-processors-improvements here. - */ - // Extract the final config for this file. const config = configArray.extractConfig(providedOptions.filename); + const processor = + config.processor && + configArray.pluginProcessors.get(config.processor); /* * Convert "/path/to/" to "". @@ -1063,21 +1063,76 @@ class Linter { filename }; - // Apply processor. - if (config.processor) { - const processor = configArray.pluginProcessors.get(config.processor); + // Verify. + if (!processor) { + return this._verifyWithoutProcessors(text, config, options); + } + debug("Apply the processor: %j", config.processor); - options.preprocess = processor.preprocess; - options.postprocess = processor.postprocess; - if (!processor.supportsAutofix) { + /* + * If the processor doesn't support autofix, don't make `fix` + * properties for this code block. + */ + if (!processor.supportsAutofix) { + debug("The processor %j doesn't support autofix.", config.processor); + options.disableFixes = true; + } + + // Does lint with preprocessing. + const originalExtname = path.extname(filename); + const preprocess = processor.preprocess || (rawText => [rawText]); + const postprocess = processor.postprocess || lodash.flatten; + const extRegExp = providedOptions.extRegExp || /\.js$/u; + const messageLists = preprocess(text, filename).map((block, i) => { + debug("A code block was found: %j", block); - // Use `disableFixes` of https://github.com/eslint/rfcs/tree/master/designs/2018-processors-improvements - options.disableFixes = true; + /* + * Keep the legacy behavior; skip the `--ext` check if the block is + * not an object. If people specified files by globs, the `--ext` + * check causes a breaking change. + */ + if (typeof block === "string") { + return this._verifyWithoutProcessors( + block, + config, + options + ); } - } - // Verify. - return this.verify(text, config, options); + const blockText = block.text; + const blockName = path.join( + providedOptions.filename, + `${i}_${block.filename}` + ); + + /* + * Skip this block if the block name didn't match the allowed file + * extensions. + */ + if (!extRegExp.test(blockName)) { + debug("This code block was skipped by the file extension."); + return []; + } + + // Resolve configuration again if the file extension was changed. + if (path.extname(blockName) !== originalExtname) { + debug("Resolving config again because of the file extension changed."); + return this._verifyWithConfigArray( + blockText, + configArray, + { ...options, filename: blockName } + ); + } + + // Does lint. + return this._verifyWithoutProcessors( + blockText, + config, + { ...options, filename: blockName } + ); + }); + + return postprocess(messageLists, filename); } /** diff --git a/tests/fixtures/processors/pattern-processor.js b/tests/fixtures/processors/pattern-processor.js new file mode 100644 index 00000000000..462c591edab --- /dev/null +++ b/tests/fixtures/processors/pattern-processor.js @@ -0,0 +1,63 @@ +"use strict"; + +/** + * Define a processor which extract code blocks `pattern` regexp matched. + * The defined processor supports autofix, but doesn't have `supportsAutofix` property. + * + * @param {RegExp} pattern The regular expression pattern for code blocks. + * The first capture group becomes the file extension of the code block. + * The second capture group becomes the text of the code block. + * @returns {Processor} The defined processor. + */ +exports.defineProcessor = (pattern, legacy = false) => { + const blocksMap = new Map(); + + return { + preprocess(wholeCode, filename) { + const blocks = []; + blocksMap.set(filename, blocks); + + // Extract code blocks. + let match; + while ((match = pattern.exec(wholeCode)) !== null) { + const [codeBlock, ext, text] = match; + const filename = `${blocks.length}.${ext}`; + const offset = match.index + codeBlock.indexOf(text); + const lineOffset = wholeCode.slice(0, match.index).split("\n").length; + + blocks.push({ text, filename, lineOffset, offset }); + } + + if (legacy) { + return blocks.map(b => b.text); + } + return blocks; + }, + + postprocess(messageLists, filename) { + const blocks = blocksMap.get(filename); + blocksMap.delete(filename); + + // Fix the location of reports. + if (blocks) { + for (let i = 0; i < messageLists.length; ++i) { + const messages = messageLists[i]; + const { lineOffset, offset } = blocks[i]; + + for (const message of messages) { + message.line += lineOffset; + if (message.endLine != null) { + message.endLine += lineOffset; + } + if (message.fix != null) { + message.fix.range[0] += offset; + message.fix.range[1] += offset; + } + } + } + } + + return [].concat(...messageLists); + } + }; +}; diff --git a/tests/lib/cli-engine.js b/tests/lib/cli-engine.js index 2e609a65061..42ccc5e27f9 100644 --- a/tests/lib/cli-engine.js +++ b/tests/lib/cli-engine.js @@ -21,7 +21,6 @@ const assert = require("chai").assert, { defineCLIEngineWithInMemoryFileSystem } = require("./cli-engine/_utils"); const proxyquire = require("proxyquire").noCallThru().noPreserveCache(); - const fCache = require("file-entry-cache"); //------------------------------------------------------------------------------ @@ -3013,6 +3012,320 @@ describe("CLIEngine", () => { assert.deepStrictEqual(results[0].output, "fixed;"); }); }); + + describe("multiple processors", () => { + + /** + * Unindent template strings. + * @param {string[]} strings Strings. + * @param {...any} values Values. + * @returns {string} Unindented string. + */ + function unindent(strings, ...values) { + const text = strings + .map((s, i) => (i === 0 ? s : values[i - 1] + s)) + .join(""); + const lines = text.split("\n").filter(Boolean); + const indentLen = /[^ ]/u.exec(lines[0]).index; + + return lines + .map(line => line.slice(indentLen)) + .join("\n"); + } + + const root = path.join(os.tmpdir(), "eslint/cli-engine/multiple-processors"); + const commonFiles = { + "node_modules/pattern-processor/index.js": fs.readFileSync( + require.resolve("../fixtures/processors/pattern-processor"), + "utf8" + ), + "node_modules/eslint-plugin-markdown/index.js": ` + const { defineProcessor } = require("pattern-processor"); + const processor = defineProcessor(${/```(\w+)\n([\s\S]+?)\n```/gu}); + exports.processors = { + ".md": { ...processor, supportsAutofix: true }, + "non-fixable": processor + }; + `, + "node_modules/eslint-plugin-html/index.js": ` + const { defineProcessor } = require("pattern-processor"); + const processor = defineProcessor(${/ + + \`\`\` + ` + }; + + it("should lint only JavaScript blocks if '--ext' was not given.", () => { + CLIEngine = defineCLIEngineWithInMemoryFileSystem({ + cwd: () => root, + files: { + ...commonFiles, + ".eslintrc.json": JSON.stringify({ + plugins: ["markdown", "html"], + rules: { semi: "error" } + }) + } + }).CLIEngine; + engine = new CLIEngine({ cwd: root }); + + const { results } = engine.executeOnFiles(["test.md"]); + + assert.strictEqual(results.length, 1); + assert.strictEqual(results[0].messages.length, 1); + assert.strictEqual(results[0].messages[0].ruleId, "semi"); + assert.strictEqual(results[0].messages[0].line, 2); + }); + + it("should fix only JavaScript blocks if '--ext' was not given.", () => { + CLIEngine = defineCLIEngineWithInMemoryFileSystem({ + cwd: () => root, + files: { + ...commonFiles, + ".eslintrc.json": JSON.stringify({ + plugins: ["markdown", "html"], + rules: { semi: "error" } + }) + } + }).CLIEngine; + engine = new CLIEngine({ cwd: root, fix: true }); + + const { results } = engine.executeOnFiles(["test.md"]); + + assert.strictEqual(results.length, 1); + assert.strictEqual(results[0].messages.length, 0); + assert.strictEqual(results[0].output, unindent` + \`\`\`js + console.log("hello");${/* ← fixed */""} + \`\`\` + \`\`\`html +
Hello
+ + + \`\`\` + `); + }); + + it("should lint HTML blocks as well with multiple processors if '--ext' option was given.", () => { + CLIEngine = defineCLIEngineWithInMemoryFileSystem({ + cwd: () => root, + files: { + ...commonFiles, + ".eslintrc.json": JSON.stringify({ + plugins: ["markdown", "html"], + rules: { semi: "error" } + }) + } + }).CLIEngine; + engine = new CLIEngine({ cwd: root, extensions: ["js", "html"] }); + + const { results } = engine.executeOnFiles(["test.md"]); + + assert.strictEqual(results.length, 1); + assert.strictEqual(results[0].messages.length, 2); + assert.strictEqual(results[0].messages[0].ruleId, "semi"); // JS block + assert.strictEqual(results[0].messages[0].line, 2); + assert.strictEqual(results[0].messages[1].ruleId, "semi"); // JS block in HTML block + assert.strictEqual(results[0].messages[1].line, 7); + }); + + it("should fix HTML blocks as well with multiple processors if '--ext' option was given.", () => { + CLIEngine = defineCLIEngineWithInMemoryFileSystem({ + cwd: () => root, + files: { + ...commonFiles, + ".eslintrc.json": JSON.stringify({ + plugins: ["markdown", "html"], + rules: { semi: "error" } + }) + } + }).CLIEngine; + engine = new CLIEngine({ cwd: root, extensions: ["js", "html"], fix: true }); + + const { results } = engine.executeOnFiles(["test.md"]); + + assert.strictEqual(results.length, 1); + assert.strictEqual(results[0].messages.length, 0); + assert.strictEqual(results[0].output, unindent` + \`\`\`js + console.log("hello");${/* ← fixed */""} + \`\`\` + \`\`\`html +
Hello
+ + + \`\`\` + `); + }); + + it("should use overriden processor; should report HTML blocks but not fix HTML blocks if the processor for '*.html' didn't support autofix.", () => { + CLIEngine = defineCLIEngineWithInMemoryFileSystem({ + cwd: () => root, + files: { + ...commonFiles, + ".eslintrc.json": JSON.stringify({ + plugins: ["markdown", "html"], + rules: { semi: "error" }, + overrides: [ + { + files: "*.html", + processor: "html/non-fixable" // supportsAutofix: false + } + ] + }) + } + }).CLIEngine; + engine = new CLIEngine({ cwd: root, extensions: ["js", "html"], fix: true }); + + const { results } = engine.executeOnFiles(["test.md"]); + + assert.strictEqual(results.length, 1); + assert.strictEqual(results[0].messages.length, 1); + assert.strictEqual(results[0].messages[0].ruleId, "semi"); // JS Block in HTML Block + assert.strictEqual(results[0].messages[0].line, 7); + assert.strictEqual(results[0].messages[0].fix, void 0); + assert.strictEqual(results[0].output, unindent` + \`\`\`js + console.log("hello");${/* ← fixed */""} + \`\`\` + \`\`\`html +
Hello
+ + + \`\`\` + `); + }); + + it("should use the config '**/*.html/*.js' to lint JavaScript blocks in HTML.", () => { + CLIEngine = defineCLIEngineWithInMemoryFileSystem({ + cwd: () => root, + files: { + ...commonFiles, + ".eslintrc.json": JSON.stringify({ + plugins: ["markdown", "html"], + rules: { semi: "error" }, + overrides: [ + { + files: "*.html", + + // this rules are not used because ESLint re-resolve configs if a code block had a different file extension. + rules: { + semi: "error", + "no-console": "off" + } + }, + { + files: "**/*.html/*.js", + rules: { + semi: "off", + "no-console": "error" + } + } + ] + }) + } + }).CLIEngine; + engine = new CLIEngine({ cwd: root, extensions: ["js", "html"] }); + + const { results } = engine.executeOnFiles(["test.md"]); + + assert.strictEqual(results.length, 1); + assert.strictEqual(results[0].messages.length, 2); + assert.strictEqual(results[0].messages[0].ruleId, "semi"); + assert.strictEqual(results[0].messages[0].line, 2); + assert.strictEqual(results[0].messages[1].ruleId, "no-console"); + assert.strictEqual(results[0].messages[1].line, 7); + }); + + it("should use the same config as one which has 'processor' property in order to lint blocks in HTML if the processor was legacy style.", () => { + CLIEngine = defineCLIEngineWithInMemoryFileSystem({ + cwd: () => root, + files: { + ...commonFiles, + ".eslintrc.json": JSON.stringify({ + plugins: ["markdown", "html"], + rules: { semi: "error" }, + overrides: [ + { + files: "*.html", + processor: "html/legacy", // this processor returns strings rather than `{text, filename}` + rules: { + semi: "off", + "no-console": "error" + } + }, + { + files: "**/*.html/*.js", + rules: { + semi: "error", + "no-console": "off" + } + } + ] + }) + } + }).CLIEngine; + engine = new CLIEngine({ cwd: root, extensions: ["js", "html"] }); + + const { results } = engine.executeOnFiles(["test.md"]); + + assert.strictEqual(results.length, 1); + assert.strictEqual(results[0].messages.length, 3); + assert.strictEqual(results[0].messages[0].ruleId, "semi"); + assert.strictEqual(results[0].messages[0].line, 2); + assert.strictEqual(results[0].messages[1].ruleId, "no-console"); + assert.strictEqual(results[0].messages[1].line, 7); + assert.strictEqual(results[0].messages[2].ruleId, "no-console"); + assert.strictEqual(results[0].messages[2].line, 10); + }); + + it("should throw an error if invalid processor was specified.", () => { + CLIEngine = defineCLIEngineWithInMemoryFileSystem({ + cwd: () => root, + files: { + ...commonFiles, + ".eslintrc.json": JSON.stringify({ + plugins: ["markdown", "html"], + processor: "markdown/unknown" + }) + } + }).CLIEngine; + engine = new CLIEngine({ cwd: root }); + + assert.throws(() => { + engine.executeOnFiles(["test.md"]); + }, /ESLint configuration of processor in '\.eslintrc\.json' is invalid: 'markdown\/unknown' was not found\./u); + }); + }); }); describe("getConfigForFile", () => {