1 | /**
|
---|
2 | * @fileoverview Mocha/Jest test wrapper
|
---|
3 | * @author Ilya Volodin
|
---|
4 | */
|
---|
5 | "use strict";
|
---|
6 |
|
---|
7 | /* globals describe, it -- Mocha globals */
|
---|
8 |
|
---|
9 | //------------------------------------------------------------------------------
|
---|
10 | // Requirements
|
---|
11 | //------------------------------------------------------------------------------
|
---|
12 |
|
---|
13 | const
|
---|
14 | assert = require("assert"),
|
---|
15 | util = require("util"),
|
---|
16 | path = require("path"),
|
---|
17 | equal = require("fast-deep-equal"),
|
---|
18 | Traverser = require("../shared/traverser"),
|
---|
19 | { getRuleOptionsSchema } = require("../config/flat-config-helpers"),
|
---|
20 | { Linter, SourceCodeFixer, interpolate } = require("../linter"),
|
---|
21 | CodePath = require("../linter/code-path-analysis/code-path");
|
---|
22 |
|
---|
23 | const { FlatConfigArray } = require("../config/flat-config-array");
|
---|
24 | const { defaultConfig } = require("../config/default-config");
|
---|
25 |
|
---|
26 | const ajv = require("../shared/ajv")({ strictDefaults: true });
|
---|
27 |
|
---|
28 | const parserSymbol = Symbol.for("eslint.RuleTester.parser");
|
---|
29 | const { SourceCode } = require("../source-code");
|
---|
30 | const { ConfigArraySymbol } = require("@humanwhocodes/config-array");
|
---|
31 |
|
---|
32 | //------------------------------------------------------------------------------
|
---|
33 | // Typedefs
|
---|
34 | //------------------------------------------------------------------------------
|
---|
35 |
|
---|
36 | /** @typedef {import("../shared/types").Parser} Parser */
|
---|
37 | /** @typedef {import("../shared/types").LanguageOptions} LanguageOptions */
|
---|
38 | /** @typedef {import("../shared/types").Rule} Rule */
|
---|
39 |
|
---|
40 |
|
---|
41 | /**
|
---|
42 | * A test case that is expected to pass lint.
|
---|
43 | * @typedef {Object} ValidTestCase
|
---|
44 | * @property {string} [name] Name for the test case.
|
---|
45 | * @property {string} code Code for the test case.
|
---|
46 | * @property {any[]} [options] Options for the test case.
|
---|
47 | * @property {LanguageOptions} [languageOptions] The language options to use in the test case.
|
---|
48 | * @property {{ [name: string]: any }} [settings] Settings for the test case.
|
---|
49 | * @property {string} [filename] The fake filename for the test case. Useful for rules that make assertion about filenames.
|
---|
50 | * @property {boolean} [only] Run only this test case or the subset of test cases with this property.
|
---|
51 | */
|
---|
52 |
|
---|
53 | /**
|
---|
54 | * A test case that is expected to fail lint.
|
---|
55 | * @typedef {Object} InvalidTestCase
|
---|
56 | * @property {string} [name] Name for the test case.
|
---|
57 | * @property {string} code Code for the test case.
|
---|
58 | * @property {number | Array<TestCaseError | string | RegExp>} errors Expected errors.
|
---|
59 | * @property {string | null} [output] The expected code after autofixes are applied. If set to `null`, the test runner will assert that no autofix is suggested.
|
---|
60 | * @property {any[]} [options] Options for the test case.
|
---|
61 | * @property {{ [name: string]: any }} [settings] Settings for the test case.
|
---|
62 | * @property {string} [filename] The fake filename for the test case. Useful for rules that make assertion about filenames.
|
---|
63 | * @property {LanguageOptions} [languageOptions] The language options to use in the test case.
|
---|
64 | * @property {boolean} [only] Run only this test case or the subset of test cases with this property.
|
---|
65 | */
|
---|
66 |
|
---|
67 | /**
|
---|
68 | * A description of a reported error used in a rule tester test.
|
---|
69 | * @typedef {Object} TestCaseError
|
---|
70 | * @property {string | RegExp} [message] Message.
|
---|
71 | * @property {string} [messageId] Message ID.
|
---|
72 | * @property {string} [type] The type of the reported AST node.
|
---|
73 | * @property {{ [name: string]: string }} [data] The data used to fill the message template.
|
---|
74 | * @property {number} [line] The 1-based line number of the reported start location.
|
---|
75 | * @property {number} [column] The 1-based column number of the reported start location.
|
---|
76 | * @property {number} [endLine] The 1-based line number of the reported end location.
|
---|
77 | * @property {number} [endColumn] The 1-based column number of the reported end location.
|
---|
78 | */
|
---|
79 |
|
---|
80 | //------------------------------------------------------------------------------
|
---|
81 | // Private Members
|
---|
82 | //------------------------------------------------------------------------------
|
---|
83 |
|
---|
84 | /*
|
---|
85 | * testerDefaultConfig must not be modified as it allows to reset the tester to
|
---|
86 | * the initial default configuration
|
---|
87 | */
|
---|
88 | const testerDefaultConfig = { rules: {} };
|
---|
89 |
|
---|
90 | /*
|
---|
91 | * RuleTester uses this config as its default. This can be overwritten via
|
---|
92 | * setDefaultConfig().
|
---|
93 | */
|
---|
94 | let sharedDefaultConfig = { rules: {} };
|
---|
95 |
|
---|
96 | /*
|
---|
97 | * List every parameters possible on a test case that are not related to eslint
|
---|
98 | * configuration
|
---|
99 | */
|
---|
100 | const RuleTesterParameters = [
|
---|
101 | "name",
|
---|
102 | "code",
|
---|
103 | "filename",
|
---|
104 | "options",
|
---|
105 | "errors",
|
---|
106 | "output",
|
---|
107 | "only"
|
---|
108 | ];
|
---|
109 |
|
---|
110 | /*
|
---|
111 | * All allowed property names in error objects.
|
---|
112 | */
|
---|
113 | const errorObjectParameters = new Set([
|
---|
114 | "message",
|
---|
115 | "messageId",
|
---|
116 | "data",
|
---|
117 | "type",
|
---|
118 | "line",
|
---|
119 | "column",
|
---|
120 | "endLine",
|
---|
121 | "endColumn",
|
---|
122 | "suggestions"
|
---|
123 | ]);
|
---|
124 | const friendlyErrorObjectParameterList = `[${[...errorObjectParameters].map(key => `'${key}'`).join(", ")}]`;
|
---|
125 |
|
---|
126 | /*
|
---|
127 | * All allowed property names in suggestion objects.
|
---|
128 | */
|
---|
129 | const suggestionObjectParameters = new Set([
|
---|
130 | "desc",
|
---|
131 | "messageId",
|
---|
132 | "data",
|
---|
133 | "output"
|
---|
134 | ]);
|
---|
135 | const friendlySuggestionObjectParameterList = `[${[...suggestionObjectParameters].map(key => `'${key}'`).join(", ")}]`;
|
---|
136 |
|
---|
137 | const forbiddenMethods = [
|
---|
138 | "applyInlineConfig",
|
---|
139 | "applyLanguageOptions",
|
---|
140 | "finalize"
|
---|
141 | ];
|
---|
142 |
|
---|
143 | /** @type {Map<string,WeakSet>} */
|
---|
144 | const forbiddenMethodCalls = new Map(forbiddenMethods.map(methodName => ([methodName, new WeakSet()])));
|
---|
145 |
|
---|
146 | const hasOwnProperty = Function.call.bind(Object.hasOwnProperty);
|
---|
147 |
|
---|
148 | /**
|
---|
149 | * Clones a given value deeply.
|
---|
150 | * Note: This ignores `parent` property.
|
---|
151 | * @param {any} x A value to clone.
|
---|
152 | * @returns {any} A cloned value.
|
---|
153 | */
|
---|
154 | function cloneDeeplyExcludesParent(x) {
|
---|
155 | if (typeof x === "object" && x !== null) {
|
---|
156 | if (Array.isArray(x)) {
|
---|
157 | return x.map(cloneDeeplyExcludesParent);
|
---|
158 | }
|
---|
159 |
|
---|
160 | const retv = {};
|
---|
161 |
|
---|
162 | for (const key in x) {
|
---|
163 | if (key !== "parent" && hasOwnProperty(x, key)) {
|
---|
164 | retv[key] = cloneDeeplyExcludesParent(x[key]);
|
---|
165 | }
|
---|
166 | }
|
---|
167 |
|
---|
168 | return retv;
|
---|
169 | }
|
---|
170 |
|
---|
171 | return x;
|
---|
172 | }
|
---|
173 |
|
---|
174 | /**
|
---|
175 | * Freezes a given value deeply.
|
---|
176 | * @param {any} x A value to freeze.
|
---|
177 | * @returns {void}
|
---|
178 | */
|
---|
179 | function freezeDeeply(x) {
|
---|
180 | if (typeof x === "object" && x !== null) {
|
---|
181 | if (Array.isArray(x)) {
|
---|
182 | x.forEach(freezeDeeply);
|
---|
183 | } else {
|
---|
184 | for (const key in x) {
|
---|
185 | if (key !== "parent" && hasOwnProperty(x, key)) {
|
---|
186 | freezeDeeply(x[key]);
|
---|
187 | }
|
---|
188 | }
|
---|
189 | }
|
---|
190 | Object.freeze(x);
|
---|
191 | }
|
---|
192 | }
|
---|
193 |
|
---|
194 | /**
|
---|
195 | * Replace control characters by `\u00xx` form.
|
---|
196 | * @param {string} text The text to sanitize.
|
---|
197 | * @returns {string} The sanitized text.
|
---|
198 | */
|
---|
199 | function sanitize(text) {
|
---|
200 | if (typeof text !== "string") {
|
---|
201 | return "";
|
---|
202 | }
|
---|
203 | return text.replace(
|
---|
204 | /[\u0000-\u0009\u000b-\u001a]/gu, // eslint-disable-line no-control-regex -- Escaping controls
|
---|
205 | c => `\\u${c.codePointAt(0).toString(16).padStart(4, "0")}`
|
---|
206 | );
|
---|
207 | }
|
---|
208 |
|
---|
209 | /**
|
---|
210 | * Define `start`/`end` properties as throwing error.
|
---|
211 | * @param {string} objName Object name used for error messages.
|
---|
212 | * @param {ASTNode} node The node to define.
|
---|
213 | * @returns {void}
|
---|
214 | */
|
---|
215 | function defineStartEndAsError(objName, node) {
|
---|
216 | Object.defineProperties(node, {
|
---|
217 | start: {
|
---|
218 | get() {
|
---|
219 | throw new Error(`Use ${objName}.range[0] instead of ${objName}.start`);
|
---|
220 | },
|
---|
221 | configurable: true,
|
---|
222 | enumerable: false
|
---|
223 | },
|
---|
224 | end: {
|
---|
225 | get() {
|
---|
226 | throw new Error(`Use ${objName}.range[1] instead of ${objName}.end`);
|
---|
227 | },
|
---|
228 | configurable: true,
|
---|
229 | enumerable: false
|
---|
230 | }
|
---|
231 | });
|
---|
232 | }
|
---|
233 |
|
---|
234 |
|
---|
235 | /**
|
---|
236 | * Define `start`/`end` properties of all nodes of the given AST as throwing error.
|
---|
237 | * @param {ASTNode} ast The root node to errorize `start`/`end` properties.
|
---|
238 | * @param {Object} [visitorKeys] Visitor keys to be used for traversing the given ast.
|
---|
239 | * @returns {void}
|
---|
240 | */
|
---|
241 | function defineStartEndAsErrorInTree(ast, visitorKeys) {
|
---|
242 | Traverser.traverse(ast, { visitorKeys, enter: defineStartEndAsError.bind(null, "node") });
|
---|
243 | ast.tokens.forEach(defineStartEndAsError.bind(null, "token"));
|
---|
244 | ast.comments.forEach(defineStartEndAsError.bind(null, "token"));
|
---|
245 | }
|
---|
246 |
|
---|
247 | /**
|
---|
248 | * Wraps the given parser in order to intercept and modify return values from the `parse` and `parseForESLint` methods, for test purposes.
|
---|
249 | * In particular, to modify ast nodes, tokens and comments to throw on access to their `start` and `end` properties.
|
---|
250 | * @param {Parser} parser Parser object.
|
---|
251 | * @returns {Parser} Wrapped parser object.
|
---|
252 | */
|
---|
253 | function wrapParser(parser) {
|
---|
254 |
|
---|
255 | if (typeof parser.parseForESLint === "function") {
|
---|
256 | return {
|
---|
257 | [parserSymbol]: parser,
|
---|
258 | parseForESLint(...args) {
|
---|
259 | const ret = parser.parseForESLint(...args);
|
---|
260 |
|
---|
261 | defineStartEndAsErrorInTree(ret.ast, ret.visitorKeys);
|
---|
262 | return ret;
|
---|
263 | }
|
---|
264 | };
|
---|
265 | }
|
---|
266 |
|
---|
267 | return {
|
---|
268 | [parserSymbol]: parser,
|
---|
269 | parse(...args) {
|
---|
270 | const ast = parser.parse(...args);
|
---|
271 |
|
---|
272 | defineStartEndAsErrorInTree(ast);
|
---|
273 | return ast;
|
---|
274 | }
|
---|
275 | };
|
---|
276 | }
|
---|
277 |
|
---|
278 | /**
|
---|
279 | * Function to replace `SourceCode.prototype.getComments`.
|
---|
280 | * @returns {void}
|
---|
281 | * @throws {Error} Deprecation message.
|
---|
282 | */
|
---|
283 | function getCommentsDeprecation() {
|
---|
284 | throw new Error(
|
---|
285 | "`SourceCode#getComments()` is deprecated and will be removed in a future major version. Use `getCommentsBefore()`, `getCommentsAfter()`, and `getCommentsInside()` instead."
|
---|
286 | );
|
---|
287 | }
|
---|
288 |
|
---|
289 | /**
|
---|
290 | * Emit a deprecation warning if rule uses CodePath#currentSegments.
|
---|
291 | * @param {string} ruleName Name of the rule.
|
---|
292 | * @returns {void}
|
---|
293 | */
|
---|
294 | function emitCodePathCurrentSegmentsWarning(ruleName) {
|
---|
295 | if (!emitCodePathCurrentSegmentsWarning[`warned-${ruleName}`]) {
|
---|
296 | emitCodePathCurrentSegmentsWarning[`warned-${ruleName}`] = true;
|
---|
297 | process.emitWarning(
|
---|
298 | `"${ruleName}" rule uses CodePath#currentSegments and will stop working in ESLint v9. Please read the documentation for how to update your code: https://eslint.org/docs/latest/extend/code-path-analysis#usage-examples`,
|
---|
299 | "DeprecationWarning"
|
---|
300 | );
|
---|
301 | }
|
---|
302 | }
|
---|
303 |
|
---|
304 | /**
|
---|
305 | * Function to replace forbidden `SourceCode` methods. Allows just one call per method.
|
---|
306 | * @param {string} methodName The name of the method to forbid.
|
---|
307 | * @param {Function} prototype The prototype with the original method to call.
|
---|
308 | * @returns {Function} The function that throws the error.
|
---|
309 | */
|
---|
310 | function throwForbiddenMethodError(methodName, prototype) {
|
---|
311 |
|
---|
312 | const original = prototype[methodName];
|
---|
313 |
|
---|
314 | return function(...args) {
|
---|
315 |
|
---|
316 | const called = forbiddenMethodCalls.get(methodName);
|
---|
317 |
|
---|
318 | /* eslint-disable no-invalid-this -- needed to operate as a method. */
|
---|
319 | if (!called.has(this)) {
|
---|
320 | called.add(this);
|
---|
321 |
|
---|
322 | return original.apply(this, args);
|
---|
323 | }
|
---|
324 | /* eslint-enable no-invalid-this -- not needed past this point */
|
---|
325 |
|
---|
326 | throw new Error(
|
---|
327 | `\`SourceCode#${methodName}()\` cannot be called inside a rule.`
|
---|
328 | );
|
---|
329 | };
|
---|
330 | }
|
---|
331 |
|
---|
332 | //------------------------------------------------------------------------------
|
---|
333 | // Public Interface
|
---|
334 | //------------------------------------------------------------------------------
|
---|
335 |
|
---|
336 | // default separators for testing
|
---|
337 | const DESCRIBE = Symbol("describe");
|
---|
338 | const IT = Symbol("it");
|
---|
339 | const IT_ONLY = Symbol("itOnly");
|
---|
340 |
|
---|
341 | /**
|
---|
342 | * This is `it` default handler if `it` don't exist.
|
---|
343 | * @this {Mocha}
|
---|
344 | * @param {string} text The description of the test case.
|
---|
345 | * @param {Function} method The logic of the test case.
|
---|
346 | * @throws {Error} Any error upon execution of `method`.
|
---|
347 | * @returns {any} Returned value of `method`.
|
---|
348 | */
|
---|
349 | function itDefaultHandler(text, method) {
|
---|
350 | try {
|
---|
351 | return method.call(this);
|
---|
352 | } catch (err) {
|
---|
353 | if (err instanceof assert.AssertionError) {
|
---|
354 | err.message += ` (${util.inspect(err.actual)} ${err.operator} ${util.inspect(err.expected)})`;
|
---|
355 | }
|
---|
356 | throw err;
|
---|
357 | }
|
---|
358 | }
|
---|
359 |
|
---|
360 | /**
|
---|
361 | * This is `describe` default handler if `describe` don't exist.
|
---|
362 | * @this {Mocha}
|
---|
363 | * @param {string} text The description of the test case.
|
---|
364 | * @param {Function} method The logic of the test case.
|
---|
365 | * @returns {any} Returned value of `method`.
|
---|
366 | */
|
---|
367 | function describeDefaultHandler(text, method) {
|
---|
368 | return method.call(this);
|
---|
369 | }
|
---|
370 |
|
---|
371 | /**
|
---|
372 | * Mocha test wrapper.
|
---|
373 | */
|
---|
374 | class FlatRuleTester {
|
---|
375 |
|
---|
376 | /**
|
---|
377 | * Creates a new instance of RuleTester.
|
---|
378 | * @param {Object} [testerConfig] Optional, extra configuration for the tester
|
---|
379 | */
|
---|
380 | constructor(testerConfig = {}) {
|
---|
381 |
|
---|
382 | /**
|
---|
383 | * The configuration to use for this tester. Combination of the tester
|
---|
384 | * configuration and the default configuration.
|
---|
385 | * @type {Object}
|
---|
386 | */
|
---|
387 | this.testerConfig = [
|
---|
388 | sharedDefaultConfig,
|
---|
389 | testerConfig,
|
---|
390 | { rules: { "rule-tester/validate-ast": "error" } }
|
---|
391 | ];
|
---|
392 |
|
---|
393 | this.linter = new Linter({ configType: "flat" });
|
---|
394 | }
|
---|
395 |
|
---|
396 | /**
|
---|
397 | * Set the configuration to use for all future tests
|
---|
398 | * @param {Object} config the configuration to use.
|
---|
399 | * @throws {TypeError} If non-object config.
|
---|
400 | * @returns {void}
|
---|
401 | */
|
---|
402 | static setDefaultConfig(config) {
|
---|
403 | if (typeof config !== "object" || config === null) {
|
---|
404 | throw new TypeError("FlatRuleTester.setDefaultConfig: config must be an object");
|
---|
405 | }
|
---|
406 | sharedDefaultConfig = config;
|
---|
407 |
|
---|
408 | // Make sure the rules object exists since it is assumed to exist later
|
---|
409 | sharedDefaultConfig.rules = sharedDefaultConfig.rules || {};
|
---|
410 | }
|
---|
411 |
|
---|
412 | /**
|
---|
413 | * Get the current configuration used for all tests
|
---|
414 | * @returns {Object} the current configuration
|
---|
415 | */
|
---|
416 | static getDefaultConfig() {
|
---|
417 | return sharedDefaultConfig;
|
---|
418 | }
|
---|
419 |
|
---|
420 | /**
|
---|
421 | * Reset the configuration to the initial configuration of the tester removing
|
---|
422 | * any changes made until now.
|
---|
423 | * @returns {void}
|
---|
424 | */
|
---|
425 | static resetDefaultConfig() {
|
---|
426 | sharedDefaultConfig = {
|
---|
427 | rules: {
|
---|
428 | ...testerDefaultConfig.rules
|
---|
429 | }
|
---|
430 | };
|
---|
431 | }
|
---|
432 |
|
---|
433 |
|
---|
434 | /*
|
---|
435 | * If people use `mocha test.js --watch` command, `describe` and `it` function
|
---|
436 | * instances are different for each execution. So `describe` and `it` should get fresh instance
|
---|
437 | * always.
|
---|
438 | */
|
---|
439 | static get describe() {
|
---|
440 | return (
|
---|
441 | this[DESCRIBE] ||
|
---|
442 | (typeof describe === "function" ? describe : describeDefaultHandler)
|
---|
443 | );
|
---|
444 | }
|
---|
445 |
|
---|
446 | static set describe(value) {
|
---|
447 | this[DESCRIBE] = value;
|
---|
448 | }
|
---|
449 |
|
---|
450 | static get it() {
|
---|
451 | return (
|
---|
452 | this[IT] ||
|
---|
453 | (typeof it === "function" ? it : itDefaultHandler)
|
---|
454 | );
|
---|
455 | }
|
---|
456 |
|
---|
457 | static set it(value) {
|
---|
458 | this[IT] = value;
|
---|
459 | }
|
---|
460 |
|
---|
461 | /**
|
---|
462 | * Adds the `only` property to a test to run it in isolation.
|
---|
463 | * @param {string | ValidTestCase | InvalidTestCase} item A single test to run by itself.
|
---|
464 | * @returns {ValidTestCase | InvalidTestCase} The test with `only` set.
|
---|
465 | */
|
---|
466 | static only(item) {
|
---|
467 | if (typeof item === "string") {
|
---|
468 | return { code: item, only: true };
|
---|
469 | }
|
---|
470 |
|
---|
471 | return { ...item, only: true };
|
---|
472 | }
|
---|
473 |
|
---|
474 | static get itOnly() {
|
---|
475 | if (typeof this[IT_ONLY] === "function") {
|
---|
476 | return this[IT_ONLY];
|
---|
477 | }
|
---|
478 | if (typeof this[IT] === "function" && typeof this[IT].only === "function") {
|
---|
479 | return Function.bind.call(this[IT].only, this[IT]);
|
---|
480 | }
|
---|
481 | if (typeof it === "function" && typeof it.only === "function") {
|
---|
482 | return Function.bind.call(it.only, it);
|
---|
483 | }
|
---|
484 |
|
---|
485 | if (typeof this[DESCRIBE] === "function" || typeof this[IT] === "function") {
|
---|
486 | throw new Error(
|
---|
487 | "Set `RuleTester.itOnly` to use `only` with a custom test framework.\n" +
|
---|
488 | "See https://eslint.org/docs/latest/integrate/nodejs-api#customizing-ruletester for more."
|
---|
489 | );
|
---|
490 | }
|
---|
491 | if (typeof it === "function") {
|
---|
492 | throw new Error("The current test framework does not support exclusive tests with `only`.");
|
---|
493 | }
|
---|
494 | throw new Error("To use `only`, use RuleTester with a test framework that provides `it.only()` like Mocha.");
|
---|
495 | }
|
---|
496 |
|
---|
497 | static set itOnly(value) {
|
---|
498 | this[IT_ONLY] = value;
|
---|
499 | }
|
---|
500 |
|
---|
501 |
|
---|
502 | /**
|
---|
503 | * Adds a new rule test to execute.
|
---|
504 | * @param {string} ruleName The name of the rule to run.
|
---|
505 | * @param {Function | Rule} rule The rule to test.
|
---|
506 | * @param {{
|
---|
507 | * valid: (ValidTestCase | string)[],
|
---|
508 | * invalid: InvalidTestCase[]
|
---|
509 | * }} test The collection of tests to run.
|
---|
510 | * @throws {TypeError|Error} If non-object `test`, or if a required
|
---|
511 | * scenario of the given type is missing.
|
---|
512 | * @returns {void}
|
---|
513 | */
|
---|
514 | run(ruleName, rule, test) {
|
---|
515 |
|
---|
516 | const testerConfig = this.testerConfig,
|
---|
517 | requiredScenarios = ["valid", "invalid"],
|
---|
518 | scenarioErrors = [],
|
---|
519 | linter = this.linter,
|
---|
520 | ruleId = `rule-to-test/${ruleName}`;
|
---|
521 |
|
---|
522 | if (!test || typeof test !== "object") {
|
---|
523 | throw new TypeError(`Test Scenarios for rule ${ruleName} : Could not find test scenario object`);
|
---|
524 | }
|
---|
525 |
|
---|
526 | requiredScenarios.forEach(scenarioType => {
|
---|
527 | if (!test[scenarioType]) {
|
---|
528 | scenarioErrors.push(`Could not find any ${scenarioType} test scenarios`);
|
---|
529 | }
|
---|
530 | });
|
---|
531 |
|
---|
532 | if (scenarioErrors.length > 0) {
|
---|
533 | throw new Error([
|
---|
534 | `Test Scenarios for rule ${ruleName} is invalid:`
|
---|
535 | ].concat(scenarioErrors).join("\n"));
|
---|
536 | }
|
---|
537 |
|
---|
538 | const baseConfig = [
|
---|
539 | { files: ["**"] }, // Make sure the default config matches for all files
|
---|
540 | {
|
---|
541 | plugins: {
|
---|
542 |
|
---|
543 | // copy root plugin over
|
---|
544 | "@": {
|
---|
545 |
|
---|
546 | /*
|
---|
547 | * Parsers are wrapped to detect more errors, so this needs
|
---|
548 | * to be a new object for each call to run(), otherwise the
|
---|
549 | * parsers will be wrapped multiple times.
|
---|
550 | */
|
---|
551 | parsers: {
|
---|
552 | ...defaultConfig[0].plugins["@"].parsers
|
---|
553 | },
|
---|
554 |
|
---|
555 | /*
|
---|
556 | * The rules key on the default plugin is a proxy to lazy-load
|
---|
557 | * just the rules that are needed. So, don't create a new object
|
---|
558 | * here, just use the default one to keep that performance
|
---|
559 | * enhancement.
|
---|
560 | */
|
---|
561 | rules: defaultConfig[0].plugins["@"].rules
|
---|
562 | },
|
---|
563 | "rule-to-test": {
|
---|
564 | rules: {
|
---|
565 | [ruleName]: Object.assign({}, rule, {
|
---|
566 |
|
---|
567 | // Create a wrapper rule that freezes the `context` properties.
|
---|
568 | create(context) {
|
---|
569 | freezeDeeply(context.options);
|
---|
570 | freezeDeeply(context.settings);
|
---|
571 | freezeDeeply(context.parserOptions);
|
---|
572 |
|
---|
573 | // freezeDeeply(context.languageOptions);
|
---|
574 |
|
---|
575 | return (typeof rule === "function" ? rule : rule.create)(context);
|
---|
576 | }
|
---|
577 | })
|
---|
578 | }
|
---|
579 | }
|
---|
580 | },
|
---|
581 | languageOptions: {
|
---|
582 | ...defaultConfig[0].languageOptions
|
---|
583 | }
|
---|
584 | },
|
---|
585 | ...defaultConfig.slice(1)
|
---|
586 | ];
|
---|
587 |
|
---|
588 | /**
|
---|
589 | * Run the rule for the given item
|
---|
590 | * @param {string|Object} item Item to run the rule against
|
---|
591 | * @throws {Error} If an invalid schema.
|
---|
592 | * @returns {Object} Eslint run result
|
---|
593 | * @private
|
---|
594 | */
|
---|
595 | function runRuleForItem(item) {
|
---|
596 | const flatConfigArrayOptions = {
|
---|
597 | baseConfig
|
---|
598 | };
|
---|
599 |
|
---|
600 | if (item.filename) {
|
---|
601 | flatConfigArrayOptions.basePath = path.parse(item.filename).root;
|
---|
602 | }
|
---|
603 |
|
---|
604 | const configs = new FlatConfigArray(testerConfig, flatConfigArrayOptions);
|
---|
605 |
|
---|
606 | /*
|
---|
607 | * Modify the returned config so that the parser is wrapped to catch
|
---|
608 | * access of the start/end properties. This method is called just
|
---|
609 | * once per code snippet being tested, so each test case gets a clean
|
---|
610 | * parser.
|
---|
611 | */
|
---|
612 | configs[ConfigArraySymbol.finalizeConfig] = function(...args) {
|
---|
613 |
|
---|
614 | // can't do super here :(
|
---|
615 | const proto = Object.getPrototypeOf(this);
|
---|
616 | const calculatedConfig = proto[ConfigArraySymbol.finalizeConfig].apply(this, args);
|
---|
617 |
|
---|
618 | // wrap the parser to catch start/end property access
|
---|
619 | calculatedConfig.languageOptions.parser = wrapParser(calculatedConfig.languageOptions.parser);
|
---|
620 | return calculatedConfig;
|
---|
621 | };
|
---|
622 |
|
---|
623 | let code, filename, output, beforeAST, afterAST;
|
---|
624 |
|
---|
625 | if (typeof item === "string") {
|
---|
626 | code = item;
|
---|
627 | } else {
|
---|
628 | code = item.code;
|
---|
629 |
|
---|
630 | /*
|
---|
631 | * Assumes everything on the item is a config except for the
|
---|
632 | * parameters used by this tester
|
---|
633 | */
|
---|
634 | const itemConfig = { ...item };
|
---|
635 |
|
---|
636 | for (const parameter of RuleTesterParameters) {
|
---|
637 | delete itemConfig[parameter];
|
---|
638 | }
|
---|
639 |
|
---|
640 | // wrap any parsers
|
---|
641 | if (itemConfig.languageOptions && itemConfig.languageOptions.parser) {
|
---|
642 |
|
---|
643 | const parser = itemConfig.languageOptions.parser;
|
---|
644 |
|
---|
645 | if (parser && typeof parser !== "object") {
|
---|
646 | throw new Error("Parser must be an object with a parse() or parseForESLint() method.");
|
---|
647 | }
|
---|
648 |
|
---|
649 | }
|
---|
650 |
|
---|
651 | /*
|
---|
652 | * Create the config object from the tester config and this item
|
---|
653 | * specific configurations.
|
---|
654 | */
|
---|
655 | configs.push(itemConfig);
|
---|
656 | }
|
---|
657 |
|
---|
658 | if (item.filename) {
|
---|
659 | filename = item.filename;
|
---|
660 | }
|
---|
661 |
|
---|
662 | let ruleConfig = 1;
|
---|
663 |
|
---|
664 | if (hasOwnProperty(item, "options")) {
|
---|
665 | assert(Array.isArray(item.options), "options must be an array");
|
---|
666 | ruleConfig = [1, ...item.options];
|
---|
667 | }
|
---|
668 |
|
---|
669 | configs.push({
|
---|
670 | rules: {
|
---|
671 | [ruleId]: ruleConfig
|
---|
672 | }
|
---|
673 | });
|
---|
674 |
|
---|
675 | const schema = getRuleOptionsSchema(rule);
|
---|
676 |
|
---|
677 | /*
|
---|
678 | * Setup AST getters.
|
---|
679 | * The goal is to check whether or not AST was modified when
|
---|
680 | * running the rule under test.
|
---|
681 | */
|
---|
682 | configs.push({
|
---|
683 | plugins: {
|
---|
684 | "rule-tester": {
|
---|
685 | rules: {
|
---|
686 | "validate-ast": {
|
---|
687 | create() {
|
---|
688 | return {
|
---|
689 | Program(node) {
|
---|
690 | beforeAST = cloneDeeplyExcludesParent(node);
|
---|
691 | },
|
---|
692 | "Program:exit"(node) {
|
---|
693 | afterAST = node;
|
---|
694 | }
|
---|
695 | };
|
---|
696 | }
|
---|
697 | }
|
---|
698 | }
|
---|
699 | }
|
---|
700 | }
|
---|
701 | });
|
---|
702 |
|
---|
703 | if (schema) {
|
---|
704 | ajv.validateSchema(schema);
|
---|
705 |
|
---|
706 | if (ajv.errors) {
|
---|
707 | const errors = ajv.errors.map(error => {
|
---|
708 | const field = error.dataPath[0] === "." ? error.dataPath.slice(1) : error.dataPath;
|
---|
709 |
|
---|
710 | return `\t${field}: ${error.message}`;
|
---|
711 | }).join("\n");
|
---|
712 |
|
---|
713 | throw new Error([`Schema for rule ${ruleName} is invalid:`, errors]);
|
---|
714 | }
|
---|
715 |
|
---|
716 | /*
|
---|
717 | * `ajv.validateSchema` checks for errors in the structure of the schema (by comparing the schema against a "meta-schema"),
|
---|
718 | * and it reports those errors individually. However, there are other types of schema errors that only occur when compiling
|
---|
719 | * the schema (e.g. using invalid defaults in a schema), and only one of these errors can be reported at a time. As a result,
|
---|
720 | * the schema is compiled here separately from checking for `validateSchema` errors.
|
---|
721 | */
|
---|
722 | try {
|
---|
723 | ajv.compile(schema);
|
---|
724 | } catch (err) {
|
---|
725 | throw new Error(`Schema for rule ${ruleName} is invalid: ${err.message}`);
|
---|
726 | }
|
---|
727 | }
|
---|
728 |
|
---|
729 | // check for validation errors
|
---|
730 | try {
|
---|
731 | configs.normalizeSync();
|
---|
732 | configs.getConfig("test.js");
|
---|
733 | } catch (error) {
|
---|
734 | error.message = `ESLint configuration in rule-tester is invalid: ${error.message}`;
|
---|
735 | throw error;
|
---|
736 | }
|
---|
737 |
|
---|
738 | // Verify the code.
|
---|
739 | const { getComments, applyLanguageOptions, applyInlineConfig, finalize } = SourceCode.prototype;
|
---|
740 | const originalCurrentSegments = Object.getOwnPropertyDescriptor(CodePath.prototype, "currentSegments");
|
---|
741 | let messages;
|
---|
742 |
|
---|
743 | try {
|
---|
744 | SourceCode.prototype.getComments = getCommentsDeprecation;
|
---|
745 | Object.defineProperty(CodePath.prototype, "currentSegments", {
|
---|
746 | get() {
|
---|
747 | emitCodePathCurrentSegmentsWarning(ruleName);
|
---|
748 | return originalCurrentSegments.get.call(this);
|
---|
749 | }
|
---|
750 | });
|
---|
751 |
|
---|
752 | forbiddenMethods.forEach(methodName => {
|
---|
753 | SourceCode.prototype[methodName] = throwForbiddenMethodError(methodName, SourceCode.prototype);
|
---|
754 | });
|
---|
755 |
|
---|
756 | messages = linter.verify(code, configs, filename);
|
---|
757 | } finally {
|
---|
758 | SourceCode.prototype.getComments = getComments;
|
---|
759 | Object.defineProperty(CodePath.prototype, "currentSegments", originalCurrentSegments);
|
---|
760 | SourceCode.prototype.applyInlineConfig = applyInlineConfig;
|
---|
761 | SourceCode.prototype.applyLanguageOptions = applyLanguageOptions;
|
---|
762 | SourceCode.prototype.finalize = finalize;
|
---|
763 | }
|
---|
764 |
|
---|
765 |
|
---|
766 | const fatalErrorMessage = messages.find(m => m.fatal);
|
---|
767 |
|
---|
768 | assert(!fatalErrorMessage, `A fatal parsing error occurred: ${fatalErrorMessage && fatalErrorMessage.message}`);
|
---|
769 |
|
---|
770 | // Verify if autofix makes a syntax error or not.
|
---|
771 | if (messages.some(m => m.fix)) {
|
---|
772 | output = SourceCodeFixer.applyFixes(code, messages).output;
|
---|
773 | const errorMessageInFix = linter.verify(output, configs, filename).find(m => m.fatal);
|
---|
774 |
|
---|
775 | assert(!errorMessageInFix, [
|
---|
776 | "A fatal parsing error occurred in autofix.",
|
---|
777 | `Error: ${errorMessageInFix && errorMessageInFix.message}`,
|
---|
778 | "Autofix output:",
|
---|
779 | output
|
---|
780 | ].join("\n"));
|
---|
781 | } else {
|
---|
782 | output = code;
|
---|
783 | }
|
---|
784 |
|
---|
785 | return {
|
---|
786 | messages,
|
---|
787 | output,
|
---|
788 | beforeAST,
|
---|
789 | afterAST: cloneDeeplyExcludesParent(afterAST)
|
---|
790 | };
|
---|
791 | }
|
---|
792 |
|
---|
793 | /**
|
---|
794 | * Check if the AST was changed
|
---|
795 | * @param {ASTNode} beforeAST AST node before running
|
---|
796 | * @param {ASTNode} afterAST AST node after running
|
---|
797 | * @returns {void}
|
---|
798 | * @private
|
---|
799 | */
|
---|
800 | function assertASTDidntChange(beforeAST, afterAST) {
|
---|
801 | if (!equal(beforeAST, afterAST)) {
|
---|
802 | assert.fail("Rule should not modify AST.");
|
---|
803 | }
|
---|
804 | }
|
---|
805 |
|
---|
806 | /**
|
---|
807 | * Check if the template is valid or not
|
---|
808 | * all valid cases go through this
|
---|
809 | * @param {string|Object} item Item to run the rule against
|
---|
810 | * @returns {void}
|
---|
811 | * @private
|
---|
812 | */
|
---|
813 | function testValidTemplate(item) {
|
---|
814 | const code = typeof item === "object" ? item.code : item;
|
---|
815 |
|
---|
816 | assert.ok(typeof code === "string", "Test case must specify a string value for 'code'");
|
---|
817 | if (item.name) {
|
---|
818 | assert.ok(typeof item.name === "string", "Optional test case property 'name' must be a string");
|
---|
819 | }
|
---|
820 |
|
---|
821 | const result = runRuleForItem(item);
|
---|
822 | const messages = result.messages;
|
---|
823 |
|
---|
824 | assert.strictEqual(messages.length, 0, util.format("Should have no errors but had %d: %s",
|
---|
825 | messages.length,
|
---|
826 | util.inspect(messages)));
|
---|
827 |
|
---|
828 | assertASTDidntChange(result.beforeAST, result.afterAST);
|
---|
829 | }
|
---|
830 |
|
---|
831 | /**
|
---|
832 | * Asserts that the message matches its expected value. If the expected
|
---|
833 | * value is a regular expression, it is checked against the actual
|
---|
834 | * value.
|
---|
835 | * @param {string} actual Actual value
|
---|
836 | * @param {string|RegExp} expected Expected value
|
---|
837 | * @returns {void}
|
---|
838 | * @private
|
---|
839 | */
|
---|
840 | function assertMessageMatches(actual, expected) {
|
---|
841 | if (expected instanceof RegExp) {
|
---|
842 |
|
---|
843 | // assert.js doesn't have a built-in RegExp match function
|
---|
844 | assert.ok(
|
---|
845 | expected.test(actual),
|
---|
846 | `Expected '${actual}' to match ${expected}`
|
---|
847 | );
|
---|
848 | } else {
|
---|
849 | assert.strictEqual(actual, expected);
|
---|
850 | }
|
---|
851 | }
|
---|
852 |
|
---|
853 | /**
|
---|
854 | * Check if the template is invalid or not
|
---|
855 | * all invalid cases go through this.
|
---|
856 | * @param {string|Object} item Item to run the rule against
|
---|
857 | * @returns {void}
|
---|
858 | * @private
|
---|
859 | */
|
---|
860 | function testInvalidTemplate(item) {
|
---|
861 | assert.ok(typeof item.code === "string", "Test case must specify a string value for 'code'");
|
---|
862 | if (item.name) {
|
---|
863 | assert.ok(typeof item.name === "string", "Optional test case property 'name' must be a string");
|
---|
864 | }
|
---|
865 | assert.ok(item.errors || item.errors === 0,
|
---|
866 | `Did not specify errors for an invalid test of ${ruleName}`);
|
---|
867 |
|
---|
868 | if (Array.isArray(item.errors) && item.errors.length === 0) {
|
---|
869 | assert.fail("Invalid cases must have at least one error");
|
---|
870 | }
|
---|
871 |
|
---|
872 | const ruleHasMetaMessages = hasOwnProperty(rule, "meta") && hasOwnProperty(rule.meta, "messages");
|
---|
873 | const friendlyIDList = ruleHasMetaMessages ? `[${Object.keys(rule.meta.messages).map(key => `'${key}'`).join(", ")}]` : null;
|
---|
874 |
|
---|
875 | const result = runRuleForItem(item);
|
---|
876 | const messages = result.messages;
|
---|
877 |
|
---|
878 | if (typeof item.errors === "number") {
|
---|
879 |
|
---|
880 | if (item.errors === 0) {
|
---|
881 | assert.fail("Invalid cases must have 'error' value greater than 0");
|
---|
882 | }
|
---|
883 |
|
---|
884 | assert.strictEqual(messages.length, item.errors, util.format("Should have %d error%s but had %d: %s",
|
---|
885 | item.errors,
|
---|
886 | item.errors === 1 ? "" : "s",
|
---|
887 | messages.length,
|
---|
888 | util.inspect(messages)));
|
---|
889 | } else {
|
---|
890 | assert.strictEqual(
|
---|
891 | messages.length, item.errors.length, util.format(
|
---|
892 | "Should have %d error%s but had %d: %s",
|
---|
893 | item.errors.length,
|
---|
894 | item.errors.length === 1 ? "" : "s",
|
---|
895 | messages.length,
|
---|
896 | util.inspect(messages)
|
---|
897 | )
|
---|
898 | );
|
---|
899 |
|
---|
900 | const hasMessageOfThisRule = messages.some(m => m.ruleId === ruleId);
|
---|
901 |
|
---|
902 | for (let i = 0, l = item.errors.length; i < l; i++) {
|
---|
903 | const error = item.errors[i];
|
---|
904 | const message = messages[i];
|
---|
905 |
|
---|
906 | assert(hasMessageOfThisRule, "Error rule name should be the same as the name of the rule being tested");
|
---|
907 |
|
---|
908 | if (typeof error === "string" || error instanceof RegExp) {
|
---|
909 |
|
---|
910 | // Just an error message.
|
---|
911 | assertMessageMatches(message.message, error);
|
---|
912 | } else if (typeof error === "object" && error !== null) {
|
---|
913 |
|
---|
914 | /*
|
---|
915 | * Error object.
|
---|
916 | * This may have a message, messageId, data, node type, line, and/or
|
---|
917 | * column.
|
---|
918 | */
|
---|
919 |
|
---|
920 | Object.keys(error).forEach(propertyName => {
|
---|
921 | assert.ok(
|
---|
922 | errorObjectParameters.has(propertyName),
|
---|
923 | `Invalid error property name '${propertyName}'. Expected one of ${friendlyErrorObjectParameterList}.`
|
---|
924 | );
|
---|
925 | });
|
---|
926 |
|
---|
927 | if (hasOwnProperty(error, "message")) {
|
---|
928 | assert.ok(!hasOwnProperty(error, "messageId"), "Error should not specify both 'message' and a 'messageId'.");
|
---|
929 | assert.ok(!hasOwnProperty(error, "data"), "Error should not specify both 'data' and 'message'.");
|
---|
930 | assertMessageMatches(message.message, error.message);
|
---|
931 | } else if (hasOwnProperty(error, "messageId")) {
|
---|
932 | assert.ok(
|
---|
933 | ruleHasMetaMessages,
|
---|
934 | "Error can not use 'messageId' if rule under test doesn't define 'meta.messages'."
|
---|
935 | );
|
---|
936 | if (!hasOwnProperty(rule.meta.messages, error.messageId)) {
|
---|
937 | assert(false, `Invalid messageId '${error.messageId}'. Expected one of ${friendlyIDList}.`);
|
---|
938 | }
|
---|
939 | assert.strictEqual(
|
---|
940 | message.messageId,
|
---|
941 | error.messageId,
|
---|
942 | `messageId '${message.messageId}' does not match expected messageId '${error.messageId}'.`
|
---|
943 | );
|
---|
944 | if (hasOwnProperty(error, "data")) {
|
---|
945 |
|
---|
946 | /*
|
---|
947 | * if data was provided, then directly compare the returned message to a synthetic
|
---|
948 | * interpolated message using the same message ID and data provided in the test.
|
---|
949 | * See https://github.com/eslint/eslint/issues/9890 for context.
|
---|
950 | */
|
---|
951 | const unformattedOriginalMessage = rule.meta.messages[error.messageId];
|
---|
952 | const rehydratedMessage = interpolate(unformattedOriginalMessage, error.data);
|
---|
953 |
|
---|
954 | assert.strictEqual(
|
---|
955 | message.message,
|
---|
956 | rehydratedMessage,
|
---|
957 | `Hydrated message "${rehydratedMessage}" does not match "${message.message}"`
|
---|
958 | );
|
---|
959 | }
|
---|
960 | }
|
---|
961 |
|
---|
962 | assert.ok(
|
---|
963 | hasOwnProperty(error, "data") ? hasOwnProperty(error, "messageId") : true,
|
---|
964 | "Error must specify 'messageId' if 'data' is used."
|
---|
965 | );
|
---|
966 |
|
---|
967 | if (error.type) {
|
---|
968 | assert.strictEqual(message.nodeType, error.type, `Error type should be ${error.type}, found ${message.nodeType}`);
|
---|
969 | }
|
---|
970 |
|
---|
971 | if (hasOwnProperty(error, "line")) {
|
---|
972 | assert.strictEqual(message.line, error.line, `Error line should be ${error.line}`);
|
---|
973 | }
|
---|
974 |
|
---|
975 | if (hasOwnProperty(error, "column")) {
|
---|
976 | assert.strictEqual(message.column, error.column, `Error column should be ${error.column}`);
|
---|
977 | }
|
---|
978 |
|
---|
979 | if (hasOwnProperty(error, "endLine")) {
|
---|
980 | assert.strictEqual(message.endLine, error.endLine, `Error endLine should be ${error.endLine}`);
|
---|
981 | }
|
---|
982 |
|
---|
983 | if (hasOwnProperty(error, "endColumn")) {
|
---|
984 | assert.strictEqual(message.endColumn, error.endColumn, `Error endColumn should be ${error.endColumn}`);
|
---|
985 | }
|
---|
986 |
|
---|
987 | if (hasOwnProperty(error, "suggestions")) {
|
---|
988 |
|
---|
989 | // Support asserting there are no suggestions
|
---|
990 | if (!error.suggestions || (Array.isArray(error.suggestions) && error.suggestions.length === 0)) {
|
---|
991 | if (Array.isArray(message.suggestions) && message.suggestions.length > 0) {
|
---|
992 | assert.fail(`Error should have no suggestions on error with message: "${message.message}"`);
|
---|
993 | }
|
---|
994 | } else {
|
---|
995 | assert.strictEqual(Array.isArray(message.suggestions), true, `Error should have an array of suggestions. Instead received "${message.suggestions}" on error with message: "${message.message}"`);
|
---|
996 | assert.strictEqual(message.suggestions.length, error.suggestions.length, `Error should have ${error.suggestions.length} suggestions. Instead found ${message.suggestions.length} suggestions`);
|
---|
997 |
|
---|
998 | error.suggestions.forEach((expectedSuggestion, index) => {
|
---|
999 | assert.ok(
|
---|
1000 | typeof expectedSuggestion === "object" && expectedSuggestion !== null,
|
---|
1001 | "Test suggestion in 'suggestions' array must be an object."
|
---|
1002 | );
|
---|
1003 | Object.keys(expectedSuggestion).forEach(propertyName => {
|
---|
1004 | assert.ok(
|
---|
1005 | suggestionObjectParameters.has(propertyName),
|
---|
1006 | `Invalid suggestion property name '${propertyName}'. Expected one of ${friendlySuggestionObjectParameterList}.`
|
---|
1007 | );
|
---|
1008 | });
|
---|
1009 |
|
---|
1010 | const actualSuggestion = message.suggestions[index];
|
---|
1011 | const suggestionPrefix = `Error Suggestion at index ${index} :`;
|
---|
1012 |
|
---|
1013 | if (hasOwnProperty(expectedSuggestion, "desc")) {
|
---|
1014 | assert.ok(
|
---|
1015 | !hasOwnProperty(expectedSuggestion, "data"),
|
---|
1016 | `${suggestionPrefix} Test should not specify both 'desc' and 'data'.`
|
---|
1017 | );
|
---|
1018 | assert.strictEqual(
|
---|
1019 | actualSuggestion.desc,
|
---|
1020 | expectedSuggestion.desc,
|
---|
1021 | `${suggestionPrefix} desc should be "${expectedSuggestion.desc}" but got "${actualSuggestion.desc}" instead.`
|
---|
1022 | );
|
---|
1023 | }
|
---|
1024 |
|
---|
1025 | if (hasOwnProperty(expectedSuggestion, "messageId")) {
|
---|
1026 | assert.ok(
|
---|
1027 | ruleHasMetaMessages,
|
---|
1028 | `${suggestionPrefix} Test can not use 'messageId' if rule under test doesn't define 'meta.messages'.`
|
---|
1029 | );
|
---|
1030 | assert.ok(
|
---|
1031 | hasOwnProperty(rule.meta.messages, expectedSuggestion.messageId),
|
---|
1032 | `${suggestionPrefix} Test has invalid messageId '${expectedSuggestion.messageId}', the rule under test allows only one of ${friendlyIDList}.`
|
---|
1033 | );
|
---|
1034 | assert.strictEqual(
|
---|
1035 | actualSuggestion.messageId,
|
---|
1036 | expectedSuggestion.messageId,
|
---|
1037 | `${suggestionPrefix} messageId should be '${expectedSuggestion.messageId}' but got '${actualSuggestion.messageId}' instead.`
|
---|
1038 | );
|
---|
1039 | if (hasOwnProperty(expectedSuggestion, "data")) {
|
---|
1040 | const unformattedMetaMessage = rule.meta.messages[expectedSuggestion.messageId];
|
---|
1041 | const rehydratedDesc = interpolate(unformattedMetaMessage, expectedSuggestion.data);
|
---|
1042 |
|
---|
1043 | assert.strictEqual(
|
---|
1044 | actualSuggestion.desc,
|
---|
1045 | rehydratedDesc,
|
---|
1046 | `${suggestionPrefix} Hydrated test desc "${rehydratedDesc}" does not match received desc "${actualSuggestion.desc}".`
|
---|
1047 | );
|
---|
1048 | }
|
---|
1049 | } else {
|
---|
1050 | assert.ok(
|
---|
1051 | !hasOwnProperty(expectedSuggestion, "data"),
|
---|
1052 | `${suggestionPrefix} Test must specify 'messageId' if 'data' is used.`
|
---|
1053 | );
|
---|
1054 | }
|
---|
1055 |
|
---|
1056 | if (hasOwnProperty(expectedSuggestion, "output")) {
|
---|
1057 | const codeWithAppliedSuggestion = SourceCodeFixer.applyFixes(item.code, [actualSuggestion]).output;
|
---|
1058 |
|
---|
1059 | assert.strictEqual(codeWithAppliedSuggestion, expectedSuggestion.output, `Expected the applied suggestion fix to match the test suggestion output for suggestion at index: ${index} on error with message: "${message.message}"`);
|
---|
1060 | }
|
---|
1061 | });
|
---|
1062 | }
|
---|
1063 | }
|
---|
1064 | } else {
|
---|
1065 |
|
---|
1066 | // Message was an unexpected type
|
---|
1067 | assert.fail(`Error should be a string, object, or RegExp, but found (${util.inspect(message)})`);
|
---|
1068 | }
|
---|
1069 | }
|
---|
1070 | }
|
---|
1071 |
|
---|
1072 | if (hasOwnProperty(item, "output")) {
|
---|
1073 | if (item.output === null) {
|
---|
1074 | assert.strictEqual(
|
---|
1075 | result.output,
|
---|
1076 | item.code,
|
---|
1077 | "Expected no autofixes to be suggested"
|
---|
1078 | );
|
---|
1079 | } else {
|
---|
1080 | assert.strictEqual(result.output, item.output, "Output is incorrect.");
|
---|
1081 | }
|
---|
1082 | } else {
|
---|
1083 | assert.strictEqual(
|
---|
1084 | result.output,
|
---|
1085 | item.code,
|
---|
1086 | "The rule fixed the code. Please add 'output' property."
|
---|
1087 | );
|
---|
1088 | }
|
---|
1089 |
|
---|
1090 | assertASTDidntChange(result.beforeAST, result.afterAST);
|
---|
1091 | }
|
---|
1092 |
|
---|
1093 | /*
|
---|
1094 | * This creates a mocha test suite and pipes all supplied info through
|
---|
1095 | * one of the templates above.
|
---|
1096 | * The test suites for valid/invalid are created conditionally as
|
---|
1097 | * test runners (eg. vitest) fail for empty test suites.
|
---|
1098 | */
|
---|
1099 | this.constructor.describe(ruleName, () => {
|
---|
1100 | if (test.valid.length > 0) {
|
---|
1101 | this.constructor.describe("valid", () => {
|
---|
1102 | test.valid.forEach(valid => {
|
---|
1103 | this.constructor[valid.only ? "itOnly" : "it"](
|
---|
1104 | sanitize(typeof valid === "object" ? valid.name || valid.code : valid),
|
---|
1105 | () => {
|
---|
1106 | testValidTemplate(valid);
|
---|
1107 | }
|
---|
1108 | );
|
---|
1109 | });
|
---|
1110 | });
|
---|
1111 | }
|
---|
1112 |
|
---|
1113 | if (test.invalid.length > 0) {
|
---|
1114 | this.constructor.describe("invalid", () => {
|
---|
1115 | test.invalid.forEach(invalid => {
|
---|
1116 | this.constructor[invalid.only ? "itOnly" : "it"](
|
---|
1117 | sanitize(invalid.name || invalid.code),
|
---|
1118 | () => {
|
---|
1119 | testInvalidTemplate(invalid);
|
---|
1120 | }
|
---|
1121 | );
|
---|
1122 | });
|
---|
1123 | });
|
---|
1124 | }
|
---|
1125 | });
|
---|
1126 | }
|
---|
1127 | }
|
---|
1128 |
|
---|
1129 | FlatRuleTester[DESCRIBE] = FlatRuleTester[IT] = FlatRuleTester[IT_ONLY] = null;
|
---|
1130 |
|
---|
1131 | module.exports = FlatRuleTester;
|
---|