| 1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261126212631264126512661267126812691270127112721273127412751276127712781279128012811282128312841285128612871288128912901291129212931294129512961297129812991300130113021303130413051306130713081309131013111312131313141315131613171318131913201321132213231324132513261327132813291330133113321333133413351336133713381339134013411342134313441345134613471348134913501351135213531354135513561357135813591360136113621363136413651366136713681369137013711372137313741375137613771378137913801381138213831384138513861387138813891390139113921393139413951396139713981399140014011402140314041405140614071408140914101411141214131414141514161417141814191420142114221423142414251426142714281429143014311432143314341435143614371438143914401441144214431444144514461447144814491450145114521453145414551456145714581459146014611462146314641465146614671468146914701471147214731474147514761477147814791480148114821483148414851486148714881489149014911492149314941495149614971498149915001501150215031504150515061507150815091510151115121513151415151516151715181519152015211522152315241525152615271528152915301531153215331534153515361537153815391540154115421543154415451546154715481549155015511552155315541555155615571558155915601561156215631564156515661567156815691570157115721573157415751576157715781579 |
- /**
- * @fileoverview Mocha/Jest test wrapper
- * @author Ilya Volodin
- */
- "use strict";
- /* globals describe, it -- Mocha globals */
- //------------------------------------------------------------------------------
- // Requirements
- //------------------------------------------------------------------------------
- const assert = require("node:assert"),
- util = require("node:util"),
- path = require("node:path"),
- equal = require("fast-deep-equal"),
- Traverser = require("../shared/traverser"),
- { Config } = require("../config/config"),
- { Linter, SourceCodeFixer } = require("../linter"),
- { interpolate, getPlaceholderMatcher } = require("../linter/interpolate"),
- stringify = require("json-stable-stringify-without-jsonify");
- const { FlatConfigArray } = require("../config/flat-config-array");
- const {
- defaultConfig,
- defaultRuleTesterConfig,
- } = require("../config/default-config");
- const ajv = require("../shared/ajv")({ strictDefaults: true });
- const parserSymbol = Symbol.for("eslint.RuleTester.parser");
- const { ConfigArraySymbol } = require("@eslint/config-array");
- const { isSerializable } = require("../shared/serialization");
- const jslang = require("../languages/js");
- const { SourceCode } = require("../languages/js/source-code");
- //------------------------------------------------------------------------------
- // Typedefs
- //------------------------------------------------------------------------------
- /** @import { LanguageOptions, RuleDefinition } from "@eslint/core" */
- /** @typedef {import("../types").Linter.Parser} Parser */
- /**
- * A test case that is expected to pass lint.
- * @typedef {Object} ValidTestCase
- * @property {string} [name] Name for the test case.
- * @property {string} code Code for the test case.
- * @property {any[]} [options] Options for the test case.
- * @property {Function} [before] Function to execute before testing the case.
- * @property {Function} [after] Function to execute after testing the case regardless of its result.
- * @property {LanguageOptions} [languageOptions] The language options to use in the test case.
- * @property {{ [name: string]: any }} [settings] Settings for the test case.
- * @property {string} [filename] The fake filename for the test case. Useful for rules that make assertion about filenames.
- * @property {boolean} [only] Run only this test case or the subset of test cases with this property.
- */
- /**
- * A test case that is expected to fail lint.
- * @typedef {Object} InvalidTestCase
- * @property {string} [name] Name for the test case.
- * @property {string} code Code for the test case.
- * @property {number | Array<TestCaseError | string | RegExp>} errors Expected errors.
- * @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.
- * @property {any[]} [options] Options for the test case.
- * @property {Function} [before] Function to execute before testing the case.
- * @property {Function} [after] Function to execute after testing the case regardless of its result.
- * @property {{ [name: string]: any }} [settings] Settings for the test case.
- * @property {string} [filename] The fake filename for the test case. Useful for rules that make assertion about filenames.
- * @property {LanguageOptions} [languageOptions] The language options to use in the test case.
- * @property {boolean} [only] Run only this test case or the subset of test cases with this property.
- */
- /**
- * A description of a reported error used in a rule tester test.
- * @typedef {Object} TestCaseError
- * @property {string | RegExp} [message] Message.
- * @property {string} [messageId] Message ID.
- * @property {string} [type] The type of the reported AST node.
- * @property {{ [name: string]: string }} [data] The data used to fill the message template.
- * @property {number} [line] The 1-based line number of the reported start location.
- * @property {number} [column] The 1-based column number of the reported start location.
- * @property {number} [endLine] The 1-based line number of the reported end location.
- * @property {number} [endColumn] The 1-based column number of the reported end location.
- */
- //------------------------------------------------------------------------------
- // Private Members
- //------------------------------------------------------------------------------
- /*
- * testerDefaultConfig must not be modified as it allows to reset the tester to
- * the initial default configuration
- */
- const testerDefaultConfig = { rules: {} };
- /*
- * RuleTester uses this config as its default. This can be overwritten via
- * setDefaultConfig().
- */
- let sharedDefaultConfig = { rules: {} };
- /*
- * List every parameters possible on a test case that are not related to eslint
- * configuration
- */
- const RuleTesterParameters = [
- "name",
- "code",
- "filename",
- "options",
- "before",
- "after",
- "errors",
- "output",
- "only",
- ];
- /*
- * All allowed property names in error objects.
- */
- const errorObjectParameters = new Set([
- "message",
- "messageId",
- "data",
- "type",
- "line",
- "column",
- "endLine",
- "endColumn",
- "suggestions",
- ]);
- const friendlyErrorObjectParameterList = `[${[...errorObjectParameters].map(key => `'${key}'`).join(", ")}]`;
- /*
- * All allowed property names in suggestion objects.
- */
- const suggestionObjectParameters = new Set([
- "desc",
- "messageId",
- "data",
- "output",
- ]);
- const friendlySuggestionObjectParameterList = `[${[...suggestionObjectParameters].map(key => `'${key}'`).join(", ")}]`;
- /*
- * Ignored test case properties when checking for test case duplicates.
- */
- const duplicationIgnoredParameters = new Set(["name", "errors", "output"]);
- const forbiddenMethods = [
- "applyInlineConfig",
- "applyLanguageOptions",
- "finalize",
- ];
- /** @type {Map<string,WeakSet>} */
- const forbiddenMethodCalls = new Map(
- forbiddenMethods.map(methodName => [methodName, new WeakSet()]),
- );
- const hasOwnProperty = Function.call.bind(Object.hasOwnProperty);
- /**
- * Clones a given value deeply.
- * Note: This ignores `parent` property.
- * @param {any} x A value to clone.
- * @returns {any} A cloned value.
- */
- function cloneDeeplyExcludesParent(x) {
- if (typeof x === "object" && x !== null) {
- if (Array.isArray(x)) {
- return x.map(cloneDeeplyExcludesParent);
- }
- const retv = {};
- for (const key in x) {
- if (key !== "parent" && hasOwnProperty(x, key)) {
- retv[key] = cloneDeeplyExcludesParent(x[key]);
- }
- }
- return retv;
- }
- return x;
- }
- /**
- * Freezes a given value deeply.
- * @param {any} x A value to freeze.
- * @param {Set<Object>} seenObjects Objects already seen during the traversal.
- * @returns {void}
- */
- function freezeDeeply(x, seenObjects = new Set()) {
- if (typeof x === "object" && x !== null) {
- if (seenObjects.has(x)) {
- return; // skip to avoid infinite recursion
- }
- seenObjects.add(x);
- if (Array.isArray(x)) {
- x.forEach(element => {
- freezeDeeply(element, seenObjects);
- });
- } else {
- for (const key in x) {
- if (key !== "parent" && hasOwnProperty(x, key)) {
- freezeDeeply(x[key], seenObjects);
- }
- }
- }
- Object.freeze(x);
- }
- }
- /**
- * Replace control characters by `\u00xx` form.
- * @param {string} text The text to sanitize.
- * @returns {string} The sanitized text.
- */
- function sanitize(text) {
- if (typeof text !== "string") {
- return "";
- }
- return text.replace(
- /[\u0000-\u0009\u000b-\u001a]/gu, // eslint-disable-line no-control-regex -- Escaping controls
- c => `\\u${c.codePointAt(0).toString(16).padStart(4, "0")}`,
- );
- }
- /**
- * Define `start`/`end` properties as throwing error.
- * @param {string} objName Object name used for error messages.
- * @param {ASTNode} node The node to define.
- * @returns {void}
- */
- function defineStartEndAsError(objName, node) {
- Object.defineProperties(node, {
- start: {
- get() {
- throw new Error(
- `Use ${objName}.range[0] instead of ${objName}.start`,
- );
- },
- configurable: true,
- enumerable: false,
- },
- end: {
- get() {
- throw new Error(
- `Use ${objName}.range[1] instead of ${objName}.end`,
- );
- },
- configurable: true,
- enumerable: false,
- },
- });
- }
- /**
- * Define `start`/`end` properties of all nodes of the given AST as throwing error.
- * @param {ASTNode} ast The root node to errorize `start`/`end` properties.
- * @param {Object} [visitorKeys] Visitor keys to be used for traversing the given ast.
- * @returns {void}
- */
- function defineStartEndAsErrorInTree(ast, visitorKeys) {
- Traverser.traverse(ast, {
- visitorKeys,
- enter: defineStartEndAsError.bind(null, "node"),
- });
- ast.tokens.forEach(defineStartEndAsError.bind(null, "token"));
- ast.comments.forEach(defineStartEndAsError.bind(null, "token"));
- }
- /**
- * Wraps the given parser in order to intercept and modify return values from the `parse` and `parseForESLint` methods, for test purposes.
- * In particular, to modify ast nodes, tokens and comments to throw on access to their `start` and `end` properties.
- * @param {Parser} parser Parser object.
- * @returns {Parser} Wrapped parser object.
- */
- function wrapParser(parser) {
- if (typeof parser.parseForESLint === "function") {
- return {
- [parserSymbol]: parser,
- parseForESLint(...args) {
- const ret = parser.parseForESLint(...args);
- defineStartEndAsErrorInTree(ret.ast, ret.visitorKeys);
- return ret;
- },
- };
- }
- return {
- [parserSymbol]: parser,
- parse(...args) {
- const ast = parser.parse(...args);
- defineStartEndAsErrorInTree(ast);
- return ast;
- },
- };
- }
- /**
- * Function to replace forbidden `SourceCode` methods. Allows just one call per method.
- * @param {string} methodName The name of the method to forbid.
- * @param {Function} prototype The prototype with the original method to call.
- * @returns {Function} The function that throws the error.
- */
- function throwForbiddenMethodError(methodName, prototype) {
- const original = prototype[methodName];
- return function (...args) {
- const called = forbiddenMethodCalls.get(methodName);
- /* eslint-disable no-invalid-this -- needed to operate as a method. */
- if (!called.has(this)) {
- called.add(this);
- return original.apply(this, args);
- }
- /* eslint-enable no-invalid-this -- not needed past this point */
- throw new Error(
- `\`SourceCode#${methodName}()\` cannot be called inside a rule.`,
- );
- };
- }
- /**
- * Extracts names of {{ placeholders }} from the reported message.
- * @param {string} message Reported message
- * @returns {string[]} Array of placeholder names
- */
- function getMessagePlaceholders(message) {
- const matcher = getPlaceholderMatcher();
- return Array.from(message.matchAll(matcher), ([, name]) => name.trim());
- }
- /**
- * Returns the placeholders in the reported messages but
- * only includes the placeholders available in the raw message and not in the provided data.
- * @param {string} message The reported message
- * @param {string} raw The raw message specified in the rule meta.messages
- * @param {undefined|Record<unknown, unknown>} data The passed
- * @returns {string[]} Missing placeholder names
- */
- function getUnsubstitutedMessagePlaceholders(message, raw, data = {}) {
- const unsubstituted = getMessagePlaceholders(message);
- if (unsubstituted.length === 0) {
- return [];
- }
- // Remove false positives by only counting placeholders in the raw message, which were not provided in the data matcher or added with a data property
- const known = getMessagePlaceholders(raw);
- const provided = Object.keys(data);
- return unsubstituted.filter(
- name => known.includes(name) && !provided.includes(name),
- );
- }
- const metaSchemaDescription = `
- \t- If the rule has options, set \`meta.schema\` to an array or non-empty object to enable options validation.
- \t- If the rule doesn't have options, omit \`meta.schema\` to enforce that no options can be passed to the rule.
- \t- You can also set \`meta.schema\` to \`false\` to opt-out of options validation (not recommended).
- \thttps://eslint.org/docs/latest/extend/custom-rules#options-schemas
- `;
- //------------------------------------------------------------------------------
- // Public Interface
- //------------------------------------------------------------------------------
- // default separators for testing
- const DESCRIBE = Symbol("describe");
- const IT = Symbol("it");
- const IT_ONLY = Symbol("itOnly");
- /**
- * This is `it` default handler if `it` don't exist.
- * @this {Mocha}
- * @param {string} text The description of the test case.
- * @param {Function} method The logic of the test case.
- * @throws {Error} Any error upon execution of `method`.
- * @returns {any} Returned value of `method`.
- */
- function itDefaultHandler(text, method) {
- try {
- return method.call(this);
- } catch (err) {
- if (err instanceof assert.AssertionError) {
- err.message += ` (${util.inspect(err.actual)} ${err.operator} ${util.inspect(err.expected)})`;
- }
- throw err;
- }
- }
- /**
- * This is `describe` default handler if `describe` don't exist.
- * @this {Mocha}
- * @param {string} text The description of the test case.
- * @param {Function} method The logic of the test case.
- * @returns {any} Returned value of `method`.
- */
- function describeDefaultHandler(text, method) {
- return method.call(this);
- }
- /**
- * Mocha test wrapper.
- */
- class RuleTester {
- /**
- * Creates a new instance of RuleTester.
- * @param {Object} [testerConfig] Optional, extra configuration for the tester
- */
- constructor(testerConfig = {}) {
- /**
- * The configuration to use for this tester. Combination of the tester
- * configuration and the default configuration.
- * @type {Object}
- */
- this.testerConfig = [
- sharedDefaultConfig,
- testerConfig,
- { rules: { "rule-tester/validate-ast": "error" } },
- ];
- this.linter = new Linter({ configType: "flat" });
- }
- /**
- * Set the configuration to use for all future tests
- * @param {Object} config the configuration to use.
- * @throws {TypeError} If non-object config.
- * @returns {void}
- */
- static setDefaultConfig(config) {
- if (typeof config !== "object" || config === null) {
- throw new TypeError(
- "RuleTester.setDefaultConfig: config must be an object",
- );
- }
- sharedDefaultConfig = config;
- // Make sure the rules object exists since it is assumed to exist later
- sharedDefaultConfig.rules = sharedDefaultConfig.rules || {};
- }
- /**
- * Get the current configuration used for all tests
- * @returns {Object} the current configuration
- */
- static getDefaultConfig() {
- return sharedDefaultConfig;
- }
- /**
- * Reset the configuration to the initial configuration of the tester removing
- * any changes made until now.
- * @returns {void}
- */
- static resetDefaultConfig() {
- sharedDefaultConfig = {
- rules: {
- ...testerDefaultConfig.rules,
- },
- };
- }
- /*
- * If people use `mocha test.js --watch` command, `describe` and `it` function
- * instances are different for each execution. So `describe` and `it` should get fresh instance
- * always.
- */
- static get describe() {
- return (
- this[DESCRIBE] ||
- (typeof describe === "function" ? describe : describeDefaultHandler)
- );
- }
- static set describe(value) {
- this[DESCRIBE] = value;
- }
- static get it() {
- return this[IT] || (typeof it === "function" ? it : itDefaultHandler);
- }
- static set it(value) {
- this[IT] = value;
- }
- /**
- * Adds the `only` property to a test to run it in isolation.
- * @param {string | ValidTestCase | InvalidTestCase} item A single test to run by itself.
- * @returns {ValidTestCase | InvalidTestCase} The test with `only` set.
- */
- static only(item) {
- if (typeof item === "string") {
- return { code: item, only: true };
- }
- return { ...item, only: true };
- }
- static get itOnly() {
- if (typeof this[IT_ONLY] === "function") {
- return this[IT_ONLY];
- }
- if (
- typeof this[IT] === "function" &&
- typeof this[IT].only === "function"
- ) {
- return Function.bind.call(this[IT].only, this[IT]);
- }
- if (typeof it === "function" && typeof it.only === "function") {
- return Function.bind.call(it.only, it);
- }
- if (
- typeof this[DESCRIBE] === "function" ||
- typeof this[IT] === "function"
- ) {
- throw new Error(
- "Set `RuleTester.itOnly` to use `only` with a custom test framework.\n" +
- "See https://eslint.org/docs/latest/integrate/nodejs-api#customizing-ruletester for more.",
- );
- }
- if (typeof it === "function") {
- throw new Error(
- "The current test framework does not support exclusive tests with `only`.",
- );
- }
- throw new Error(
- "To use `only`, use RuleTester with a test framework that provides `it.only()` like Mocha.",
- );
- }
- static set itOnly(value) {
- this[IT_ONLY] = value;
- }
- /**
- * Adds a new rule test to execute.
- * @param {string} ruleName The name of the rule to run.
- * @param {RuleDefinition} rule The rule to test.
- * @param {{
- * valid: (ValidTestCase | string)[],
- * invalid: InvalidTestCase[]
- * }} test The collection of tests to run.
- * @throws {TypeError|Error} If `rule` is not an object with a `create` method,
- * or if non-object `test`, or if a required scenario of the given type is missing.
- * @returns {void}
- */
- run(ruleName, rule, test) {
- const testerConfig = this.testerConfig,
- requiredScenarios = ["valid", "invalid"],
- scenarioErrors = [],
- linter = this.linter,
- ruleId = `rule-to-test/${ruleName}`;
- const seenValidTestCases = new Set();
- const seenInvalidTestCases = new Set();
- if (
- !rule ||
- typeof rule !== "object" ||
- typeof rule.create !== "function"
- ) {
- throw new TypeError(
- "Rule must be an object with a `create` method",
- );
- }
- if (!test || typeof test !== "object") {
- throw new TypeError(
- `Test Scenarios for rule ${ruleName} : Could not find test scenario object`,
- );
- }
- requiredScenarios.forEach(scenarioType => {
- if (!test[scenarioType]) {
- scenarioErrors.push(
- `Could not find any ${scenarioType} test scenarios`,
- );
- }
- });
- if (scenarioErrors.length > 0) {
- throw new Error(
- [`Test Scenarios for rule ${ruleName} is invalid:`]
- .concat(scenarioErrors)
- .join("\n"),
- );
- }
- const baseConfig = [
- {
- plugins: {
- // copy root plugin over
- "@": {
- /*
- * Parsers are wrapped to detect more errors, so this needs
- * to be a new object for each call to run(), otherwise the
- * parsers will be wrapped multiple times.
- */
- parsers: {
- ...defaultConfig[0].plugins["@"].parsers,
- },
- /*
- * The rules key on the default plugin is a proxy to lazy-load
- * just the rules that are needed. So, don't create a new object
- * here, just use the default one to keep that performance
- * enhancement.
- */
- rules: defaultConfig[0].plugins["@"].rules,
- languages: defaultConfig[0].plugins["@"].languages,
- },
- "rule-to-test": {
- rules: {
- [ruleName]: Object.assign({}, rule, {
- // Create a wrapper rule that freezes the `context` properties.
- create(context) {
- freezeDeeply(context.options);
- freezeDeeply(context.settings);
- freezeDeeply(context.parserOptions);
- // freezeDeeply(context.languageOptions);
- return rule.create(context);
- },
- }),
- },
- },
- },
- language: defaultConfig[0].language,
- },
- ...defaultRuleTesterConfig,
- ];
- /**
- * Runs a hook on the given item when it's assigned to the given property
- * @param {string|Object} item Item to run the hook on
- * @param {string} prop The property having the hook assigned to
- * @throws {Error} If the property is not a function or that function throws an error
- * @returns {void}
- * @private
- */
- function runHook(item, prop) {
- if (typeof item === "object" && hasOwnProperty(item, prop)) {
- assert.strictEqual(
- typeof item[prop],
- "function",
- `Optional test case property '${prop}' must be a function`,
- );
- item[prop]();
- }
- }
- /**
- * Run the rule for the given item
- * @param {string|Object} item Item to run the rule against
- * @throws {Error} If an invalid schema.
- * @returns {Object} Eslint run result
- * @private
- */
- function runRuleForItem(item) {
- const flatConfigArrayOptions = {
- baseConfig,
- };
- if (item.filename) {
- flatConfigArrayOptions.basePath =
- path.parse(item.filename).root || void 0;
- }
- const configs = new FlatConfigArray(
- testerConfig,
- flatConfigArrayOptions,
- );
- /*
- * Modify the returned config so that the parser is wrapped to catch
- * access of the start/end properties. This method is called just
- * once per code snippet being tested, so each test case gets a clean
- * parser.
- */
- configs[ConfigArraySymbol.finalizeConfig] = function (...args) {
- // can't do super here :(
- const proto = Object.getPrototypeOf(this);
- const calculatedConfig = proto[
- ConfigArraySymbol.finalizeConfig
- ].apply(this, args);
- // wrap the parser to catch start/end property access
- if (calculatedConfig.language === jslang) {
- calculatedConfig.languageOptions.parser = wrapParser(
- calculatedConfig.languageOptions.parser,
- );
- }
- return calculatedConfig;
- };
- let code, filename, output, beforeAST, afterAST;
- if (typeof item === "string") {
- code = item;
- } else {
- code = item.code;
- /*
- * Assumes everything on the item is a config except for the
- * parameters used by this tester
- */
- const itemConfig = { ...item };
- for (const parameter of RuleTesterParameters) {
- delete itemConfig[parameter];
- }
- /*
- * Create the config object from the tester config and this item
- * specific configurations.
- */
- configs.push(itemConfig);
- }
- if (hasOwnProperty(item, "only")) {
- assert.ok(
- typeof item.only === "boolean",
- "Optional test case property 'only' must be a boolean",
- );
- }
- if (hasOwnProperty(item, "filename")) {
- assert.ok(
- typeof item.filename === "string",
- "Optional test case property 'filename' must be a string",
- );
- filename = item.filename;
- }
- let ruleConfig = 1;
- if (hasOwnProperty(item, "options")) {
- assert(Array.isArray(item.options), "options must be an array");
- ruleConfig = [1, ...item.options];
- }
- configs.push({
- rules: {
- [ruleId]: ruleConfig,
- },
- });
- let schema;
- try {
- schema = Config.getRuleOptionsSchema(rule);
- } catch (err) {
- err.message += metaSchemaDescription;
- throw err;
- }
- /*
- * Check and throw an error if the schema is an empty object (`schema:{}`), because such schema
- * doesn't validate or enforce anything and is therefore considered a possible error. If the intent
- * was to skip options validation, `schema:false` should be set instead (explicit opt-out).
- *
- * For this purpose, a schema object is considered empty if it doesn't have any own enumerable string-keyed
- * properties. While `ajv.compile()` does use enumerable properties from the prototype chain as well,
- * it caches compiled schemas by serializing only own enumerable properties, so it's generally not a good idea
- * to use inherited properties in schemas because schemas that differ only in inherited properties would end up
- * having the same cache entry that would be correct for only one of them.
- *
- * At this point, `schema` can only be an object or `null`.
- */
- if (schema && Object.keys(schema).length === 0) {
- throw new Error(
- `\`schema: {}\` is a no-op${metaSchemaDescription}`,
- );
- }
- /*
- * Setup AST getters.
- * The goal is to check whether or not AST was modified when
- * running the rule under test.
- */
- configs.push({
- plugins: {
- "rule-tester": {
- rules: {
- "validate-ast": {
- create() {
- return {
- Program(node) {
- beforeAST =
- cloneDeeplyExcludesParent(node);
- },
- "Program:exit"(node) {
- afterAST = node;
- },
- };
- },
- },
- },
- },
- },
- });
- if (schema) {
- ajv.validateSchema(schema);
- if (ajv.errors) {
- const errors = ajv.errors
- .map(error => {
- const field =
- error.dataPath[0] === "."
- ? error.dataPath.slice(1)
- : error.dataPath;
- return `\t${field}: ${error.message}`;
- })
- .join("\n");
- throw new Error([
- `Schema for rule ${ruleName} is invalid:`,
- errors,
- ]);
- }
- /*
- * `ajv.validateSchema` checks for errors in the structure of the schema (by comparing the schema against a "meta-schema"),
- * and it reports those errors individually. However, there are other types of schema errors that only occur when compiling
- * 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,
- * the schema is compiled here separately from checking for `validateSchema` errors.
- */
- try {
- ajv.compile(schema);
- } catch (err) {
- throw new Error(
- `Schema for rule ${ruleName} is invalid: ${err.message}`,
- {
- cause: err,
- },
- );
- }
- }
- // check for validation errors
- try {
- configs.normalizeSync();
- configs.getConfig("test.js");
- } catch (error) {
- error.message = `ESLint configuration in rule-tester is invalid: ${error.message}`;
- throw error;
- }
- // Verify the code.
- const { applyLanguageOptions, applyInlineConfig, finalize } =
- SourceCode.prototype;
- let messages;
- try {
- forbiddenMethods.forEach(methodName => {
- SourceCode.prototype[methodName] =
- throwForbiddenMethodError(
- methodName,
- SourceCode.prototype,
- );
- });
- messages = linter.verify(code, configs, filename);
- } finally {
- SourceCode.prototype.applyInlineConfig = applyInlineConfig;
- SourceCode.prototype.applyLanguageOptions =
- applyLanguageOptions;
- SourceCode.prototype.finalize = finalize;
- }
- const fatalErrorMessage = messages.find(m => m.fatal);
- assert(
- !fatalErrorMessage,
- `A fatal parsing error occurred: ${fatalErrorMessage && fatalErrorMessage.message}`,
- );
- // Verify if autofix makes a syntax error or not.
- if (messages.some(m => m.fix)) {
- output = SourceCodeFixer.applyFixes(code, messages).output;
- const errorMessageInFix = linter
- .verify(output, configs, filename)
- .find(m => m.fatal);
- assert(
- !errorMessageInFix,
- [
- "A fatal parsing error occurred in autofix.",
- `Error: ${errorMessageInFix && errorMessageInFix.message}`,
- "Autofix output:",
- output,
- ].join("\n"),
- );
- } else {
- output = code;
- }
- return {
- messages,
- output,
- beforeAST,
- afterAST: cloneDeeplyExcludesParent(afterAST),
- configs,
- filename,
- };
- }
- /**
- * Check if the AST was changed
- * @param {ASTNode} beforeAST AST node before running
- * @param {ASTNode} afterAST AST node after running
- * @returns {void}
- * @private
- */
- function assertASTDidntChange(beforeAST, afterAST) {
- if (!equal(beforeAST, afterAST)) {
- assert.fail("Rule should not modify AST.");
- }
- }
- /**
- * Check if this test case is a duplicate of one we have seen before.
- * @param {string|Object} item test case object
- * @param {Set<string>} seenTestCases set of serialized test cases we have seen so far (managed by this function)
- * @returns {void}
- * @private
- */
- function checkDuplicateTestCase(item, seenTestCases) {
- if (!isSerializable(item)) {
- /*
- * If we can't serialize a test case (because it contains a function, RegExp, etc), skip the check.
- * This might happen with properties like: options, plugins, settings, languageOptions.parser, languageOptions.parserOptions.
- */
- return;
- }
- const normalizedItem =
- typeof item === "string" ? { code: item } : item;
- const serializedTestCase = stringify(normalizedItem, {
- replacer(key, value) {
- // "this" is the currently stringified object --> only ignore top-level properties
- return normalizedItem !== this ||
- !duplicationIgnoredParameters.has(key)
- ? value
- : void 0;
- },
- });
- assert(
- !seenTestCases.has(serializedTestCase),
- "detected duplicate test case",
- );
- seenTestCases.add(serializedTestCase);
- }
- /**
- * Check if the template is valid or not
- * all valid cases go through this
- * @param {string|Object} item Item to run the rule against
- * @returns {void}
- * @private
- */
- function testValidTemplate(item) {
- const code = typeof item === "object" ? item.code : item;
- assert.ok(
- typeof code === "string",
- "Test case must specify a string value for 'code'",
- );
- if (item.name) {
- assert.ok(
- typeof item.name === "string",
- "Optional test case property 'name' must be a string",
- );
- }
- checkDuplicateTestCase(item, seenValidTestCases);
- const result = runRuleForItem(item);
- const messages = result.messages;
- assert.strictEqual(
- messages.length,
- 0,
- util.format(
- "Should have no errors but had %d: %s",
- messages.length,
- util.inspect(messages),
- ),
- );
- assertASTDidntChange(result.beforeAST, result.afterAST);
- }
- /**
- * Asserts that the message matches its expected value. If the expected
- * value is a regular expression, it is checked against the actual
- * value.
- * @param {string} actual Actual value
- * @param {string|RegExp} expected Expected value
- * @returns {void}
- * @private
- */
- function assertMessageMatches(actual, expected) {
- if (expected instanceof RegExp) {
- // assert.js doesn't have a built-in RegExp match function
- assert.ok(
- expected.test(actual),
- `Expected '${actual}' to match ${expected}`,
- );
- } else {
- assert.strictEqual(actual, expected);
- }
- }
- /**
- * Check if the template is invalid or not
- * all invalid cases go through this.
- * @param {string|Object} item Item to run the rule against
- * @returns {void}
- * @private
- */
- function testInvalidTemplate(item) {
- assert.ok(
- typeof item.code === "string",
- "Test case must specify a string value for 'code'",
- );
- if (item.name) {
- assert.ok(
- typeof item.name === "string",
- "Optional test case property 'name' must be a string",
- );
- }
- assert.ok(
- item.errors || item.errors === 0,
- `Did not specify errors for an invalid test of ${ruleName}`,
- );
- if (Array.isArray(item.errors) && item.errors.length === 0) {
- assert.fail("Invalid cases must have at least one error");
- }
- checkDuplicateTestCase(item, seenInvalidTestCases);
- const ruleHasMetaMessages =
- hasOwnProperty(rule, "meta") &&
- hasOwnProperty(rule.meta, "messages");
- const friendlyIDList = ruleHasMetaMessages
- ? `[${Object.keys(rule.meta.messages)
- .map(key => `'${key}'`)
- .join(", ")}]`
- : null;
- const result = runRuleForItem(item);
- const messages = result.messages;
- for (const message of messages) {
- if (hasOwnProperty(message, "suggestions")) {
- /** @type {Map<string, number>} */
- const seenMessageIndices = new Map();
- for (let i = 0; i < message.suggestions.length; i += 1) {
- const suggestionMessage = message.suggestions[i].desc;
- const previous =
- seenMessageIndices.get(suggestionMessage);
- assert.ok(
- !seenMessageIndices.has(suggestionMessage),
- `Suggestion message '${suggestionMessage}' reported from suggestion ${i} was previously reported by suggestion ${previous}. Suggestion messages should be unique within an error.`,
- );
- seenMessageIndices.set(suggestionMessage, i);
- }
- }
- }
- if (typeof item.errors === "number") {
- if (item.errors === 0) {
- assert.fail(
- "Invalid cases must have 'error' value greater than 0",
- );
- }
- assert.strictEqual(
- messages.length,
- item.errors,
- util.format(
- "Should have %d error%s but had %d: %s",
- item.errors,
- item.errors === 1 ? "" : "s",
- messages.length,
- util.inspect(messages),
- ),
- );
- } else {
- assert.strictEqual(
- messages.length,
- item.errors.length,
- util.format(
- "Should have %d error%s but had %d: %s",
- item.errors.length,
- item.errors.length === 1 ? "" : "s",
- messages.length,
- util.inspect(messages),
- ),
- );
- const hasMessageOfThisRule = messages.some(
- m => m.ruleId === ruleId,
- );
- for (let i = 0, l = item.errors.length; i < l; i++) {
- const error = item.errors[i];
- const message = messages[i];
- assert(
- hasMessageOfThisRule,
- "Error rule name should be the same as the name of the rule being tested",
- );
- if (typeof error === "string" || error instanceof RegExp) {
- // Just an error message.
- assertMessageMatches(message.message, error);
- assert.ok(
- message.suggestions === void 0,
- `Error at index ${i} has suggestions. Please convert the test error into an object and specify 'suggestions' property on it to test suggestions.`,
- );
- } else if (typeof error === "object" && error !== null) {
- /*
- * Error object.
- * This may have a message, messageId, data, node type, line, and/or
- * column.
- */
- Object.keys(error).forEach(propertyName => {
- assert.ok(
- errorObjectParameters.has(propertyName),
- `Invalid error property name '${propertyName}'. Expected one of ${friendlyErrorObjectParameterList}.`,
- );
- });
- if (hasOwnProperty(error, "message")) {
- assert.ok(
- !hasOwnProperty(error, "messageId"),
- "Error should not specify both 'message' and a 'messageId'.",
- );
- assert.ok(
- !hasOwnProperty(error, "data"),
- "Error should not specify both 'data' and 'message'.",
- );
- assertMessageMatches(
- message.message,
- error.message,
- );
- } else if (hasOwnProperty(error, "messageId")) {
- assert.ok(
- ruleHasMetaMessages,
- "Error can not use 'messageId' if rule under test doesn't define 'meta.messages'.",
- );
- if (
- !hasOwnProperty(
- rule.meta.messages,
- error.messageId,
- )
- ) {
- assert(
- false,
- `Invalid messageId '${error.messageId}'. Expected one of ${friendlyIDList}.`,
- );
- }
- assert.strictEqual(
- message.messageId,
- error.messageId,
- `messageId '${message.messageId}' does not match expected messageId '${error.messageId}'.`,
- );
- const unsubstitutedPlaceholders =
- getUnsubstitutedMessagePlaceholders(
- message.message,
- rule.meta.messages[message.messageId],
- error.data,
- );
- assert.ok(
- unsubstitutedPlaceholders.length === 0,
- `The reported message has ${unsubstitutedPlaceholders.length > 1 ? `unsubstituted placeholders: ${unsubstitutedPlaceholders.map(name => `'${name}'`).join(", ")}` : `an unsubstituted placeholder '${unsubstitutedPlaceholders[0]}'`}. Please provide the missing ${unsubstitutedPlaceholders.length > 1 ? "values" : "value"} via the 'data' property in the context.report() call.`,
- );
- if (hasOwnProperty(error, "data")) {
- /*
- * if data was provided, then directly compare the returned message to a synthetic
- * interpolated message using the same message ID and data provided in the test.
- * See https://github.com/eslint/eslint/issues/9890 for context.
- */
- const unformattedOriginalMessage =
- rule.meta.messages[error.messageId];
- const rehydratedMessage = interpolate(
- unformattedOriginalMessage,
- error.data,
- );
- assert.strictEqual(
- message.message,
- rehydratedMessage,
- `Hydrated message "${rehydratedMessage}" does not match "${message.message}"`,
- );
- }
- } else {
- assert.fail(
- "Test error must specify either a 'messageId' or 'message'.",
- );
- }
- if (error.type) {
- assert.strictEqual(
- message.nodeType,
- error.type,
- `Error type should be ${error.type}, found ${message.nodeType}`,
- );
- }
- const actualLocation = {};
- const expectedLocation = {};
- if (hasOwnProperty(error, "line")) {
- actualLocation.line = message.line;
- expectedLocation.line = error.line;
- }
- if (hasOwnProperty(error, "column")) {
- actualLocation.column = message.column;
- expectedLocation.column = error.column;
- }
- if (hasOwnProperty(error, "endLine")) {
- actualLocation.endLine = message.endLine;
- expectedLocation.endLine = error.endLine;
- }
- if (hasOwnProperty(error, "endColumn")) {
- actualLocation.endColumn = message.endColumn;
- expectedLocation.endColumn = error.endColumn;
- }
- if (Object.keys(expectedLocation).length > 0) {
- assert.deepStrictEqual(
- actualLocation,
- expectedLocation,
- "Actual error location does not match expected error location.",
- );
- }
- assert.ok(
- !message.suggestions ||
- hasOwnProperty(error, "suggestions"),
- `Error at index ${i} has suggestions. Please specify 'suggestions' property on the test error object.`,
- );
- if (hasOwnProperty(error, "suggestions")) {
- // Support asserting there are no suggestions
- const expectsSuggestions = Array.isArray(
- error.suggestions,
- )
- ? error.suggestions.length > 0
- : Boolean(error.suggestions);
- const hasSuggestions =
- message.suggestions !== void 0;
- if (!hasSuggestions && expectsSuggestions) {
- assert.ok(
- !error.suggestions,
- `Error should have suggestions on error with message: "${message.message}"`,
- );
- } else if (hasSuggestions) {
- assert.ok(
- expectsSuggestions,
- `Error should have no suggestions on error with message: "${message.message}"`,
- );
- if (typeof error.suggestions === "number") {
- assert.strictEqual(
- message.suggestions.length,
- error.suggestions,
- `Error should have ${error.suggestions} suggestions. Instead found ${message.suggestions.length} suggestions`,
- );
- } else if (Array.isArray(error.suggestions)) {
- assert.strictEqual(
- message.suggestions.length,
- error.suggestions.length,
- `Error should have ${error.suggestions.length} suggestions. Instead found ${message.suggestions.length} suggestions`,
- );
- error.suggestions.forEach(
- (expectedSuggestion, index) => {
- assert.ok(
- typeof expectedSuggestion ===
- "object" &&
- expectedSuggestion !== null,
- "Test suggestion in 'suggestions' array must be an object.",
- );
- Object.keys(
- expectedSuggestion,
- ).forEach(propertyName => {
- assert.ok(
- suggestionObjectParameters.has(
- propertyName,
- ),
- `Invalid suggestion property name '${propertyName}'. Expected one of ${friendlySuggestionObjectParameterList}.`,
- );
- });
- const actualSuggestion =
- message.suggestions[index];
- const suggestionPrefix = `Error Suggestion at index ${index}:`;
- if (
- hasOwnProperty(
- expectedSuggestion,
- "desc",
- )
- ) {
- assert.ok(
- !hasOwnProperty(
- expectedSuggestion,
- "data",
- ),
- `${suggestionPrefix} Test should not specify both 'desc' and 'data'.`,
- );
- assert.ok(
- !hasOwnProperty(
- expectedSuggestion,
- "messageId",
- ),
- `${suggestionPrefix} Test should not specify both 'desc' and 'messageId'.`,
- );
- assert.strictEqual(
- actualSuggestion.desc,
- expectedSuggestion.desc,
- `${suggestionPrefix} desc should be "${expectedSuggestion.desc}" but got "${actualSuggestion.desc}" instead.`,
- );
- } else if (
- hasOwnProperty(
- expectedSuggestion,
- "messageId",
- )
- ) {
- assert.ok(
- ruleHasMetaMessages,
- `${suggestionPrefix} Test can not use 'messageId' if rule under test doesn't define 'meta.messages'.`,
- );
- assert.ok(
- hasOwnProperty(
- rule.meta.messages,
- expectedSuggestion.messageId,
- ),
- `${suggestionPrefix} Test has invalid messageId '${expectedSuggestion.messageId}', the rule under test allows only one of ${friendlyIDList}.`,
- );
- assert.strictEqual(
- actualSuggestion.messageId,
- expectedSuggestion.messageId,
- `${suggestionPrefix} messageId should be '${expectedSuggestion.messageId}' but got '${actualSuggestion.messageId}' instead.`,
- );
- const unsubstitutedPlaceholders =
- getUnsubstitutedMessagePlaceholders(
- actualSuggestion.desc,
- rule.meta.messages[
- expectedSuggestion
- .messageId
- ],
- expectedSuggestion.data,
- );
- assert.ok(
- unsubstitutedPlaceholders.length ===
- 0,
- `The message of the suggestion has ${unsubstitutedPlaceholders.length > 1 ? `unsubstituted placeholders: ${unsubstitutedPlaceholders.map(name => `'${name}'`).join(", ")}` : `an unsubstituted placeholder '${unsubstitutedPlaceholders[0]}'`}. Please provide the missing ${unsubstitutedPlaceholders.length > 1 ? "values" : "value"} via the 'data' property for the suggestion in the context.report() call.`,
- );
- if (
- hasOwnProperty(
- expectedSuggestion,
- "data",
- )
- ) {
- const unformattedMetaMessage =
- rule.meta.messages[
- expectedSuggestion
- .messageId
- ];
- const rehydratedDesc =
- interpolate(
- unformattedMetaMessage,
- expectedSuggestion.data,
- );
- assert.strictEqual(
- actualSuggestion.desc,
- rehydratedDesc,
- `${suggestionPrefix} Hydrated test desc "${rehydratedDesc}" does not match received desc "${actualSuggestion.desc}".`,
- );
- }
- } else if (
- hasOwnProperty(
- expectedSuggestion,
- "data",
- )
- ) {
- assert.fail(
- `${suggestionPrefix} Test must specify 'messageId' if 'data' is used.`,
- );
- } else {
- assert.fail(
- `${suggestionPrefix} Test must specify either 'messageId' or 'desc'.`,
- );
- }
- assert.ok(
- hasOwnProperty(
- expectedSuggestion,
- "output",
- ),
- `${suggestionPrefix} The "output" property is required.`,
- );
- const codeWithAppliedSuggestion =
- SourceCodeFixer.applyFixes(
- item.code,
- [actualSuggestion],
- ).output;
- // Verify if suggestion fix makes a syntax error or not.
- const errorMessageInSuggestion =
- linter
- .verify(
- codeWithAppliedSuggestion,
- result.configs,
- result.filename,
- )
- .find(m => m.fatal);
- assert(
- !errorMessageInSuggestion,
- [
- "A fatal parsing error occurred in suggestion fix.",
- `Error: ${errorMessageInSuggestion && errorMessageInSuggestion.message}`,
- "Suggestion output:",
- codeWithAppliedSuggestion,
- ].join("\n"),
- );
- 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}"`,
- );
- assert.notStrictEqual(
- expectedSuggestion.output,
- item.code,
- `The output of a suggestion should differ from the original source code for suggestion at index: ${index} on error with message: "${message.message}"`,
- );
- },
- );
- } else {
- assert.fail(
- "Test error object property 'suggestions' should be an array or a number",
- );
- }
- }
- }
- } else {
- // Message was an unexpected type
- assert.fail(
- `Error should be a string, object, or RegExp, but found (${util.inspect(message)})`,
- );
- }
- }
- }
- if (hasOwnProperty(item, "output")) {
- if (item.output === null) {
- assert.strictEqual(
- result.output,
- item.code,
- "Expected no autofixes to be suggested",
- );
- } else {
- assert.strictEqual(
- result.output,
- item.output,
- "Output is incorrect.",
- );
- assert.notStrictEqual(
- item.code,
- item.output,
- "Test property 'output' matches 'code'. If no autofix is expected, then omit the 'output' property or set it to null.",
- );
- }
- } else {
- assert.strictEqual(
- result.output,
- item.code,
- "The rule fixed the code. Please add 'output' property.",
- );
- }
- assertASTDidntChange(result.beforeAST, result.afterAST);
- }
- /*
- * This creates a mocha test suite and pipes all supplied info through
- * one of the templates above.
- * The test suites for valid/invalid are created conditionally as
- * test runners (eg. vitest) fail for empty test suites.
- */
- this.constructor.describe(ruleName, () => {
- if (test.valid.length > 0) {
- this.constructor.describe("valid", () => {
- test.valid.forEach(valid => {
- this.constructor[valid.only ? "itOnly" : "it"](
- sanitize(
- typeof valid === "object"
- ? valid.name || valid.code
- : valid,
- ),
- () => {
- try {
- runHook(valid, "before");
- testValidTemplate(valid);
- } finally {
- runHook(valid, "after");
- }
- },
- );
- });
- });
- }
- if (test.invalid.length > 0) {
- this.constructor.describe("invalid", () => {
- test.invalid.forEach(invalid => {
- this.constructor[invalid.only ? "itOnly" : "it"](
- sanitize(invalid.name || invalid.code),
- () => {
- try {
- runHook(invalid, "before");
- testInvalidTemplate(invalid);
- } finally {
- runHook(invalid, "after");
- }
- },
- );
- });
- });
- }
- });
- }
- }
- RuleTester[DESCRIBE] = RuleTester[IT] = RuleTester[IT_ONLY] = null;
- module.exports = RuleTester;
|