12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034103510361037103810391040104110421043104410451046104710481049105010511052105310541055105610571058105910601061106210631064106510661067106810691070107110721073107410751076107710781079108010811082108310841085108610871088108910901091109210931094109510961097109810991100110111021103110411051106110711081109111011111112111311141115111611171118111911201121112211231124112511261127112811291130113111321133113411351136113711381139114011411142114311441145114611471148114911501151115211531154115511561157115811591160116111621163116411651166116711681169117011711172117311741175117611771178117911801181118211831184118511861187118811891190119111921193119411951196119711981199120012011202120312041205120612071208120912101211121212131214121512161217121812191220122112221223122412251226122712281229123012311232123312341235123612371238123912401241124212431244124512461247124812491250125112521253125412551256125712581259126012611262126312641265126612671268126912701271127212731274127512761277127812791280128112821283128412851286128712881289129012911292129312941295129612971298129913001301130213031304130513061307 |
- /**
- * @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"),
- { getRuleOptionsSchema } = require("../config/flat-config-helpers"),
- { 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 } = 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
- //------------------------------------------------------------------------------
- /** @typedef {import("../shared/types").Parser} Parser */
- /** @typedef {import("../shared/types").LanguageOptions} LanguageOptions */
- /** @typedef {import("../shared/types").Rule} Rule */
- /**
- * 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.
- * @returns {void}
- */
- function freezeDeeply(x) {
- if (typeof x === "object" && x !== null) {
- if (Array.isArray(x)) {
- x.forEach(freezeDeeply);
- } else {
- for (const key in x) {
- if (key !== "parent" && hasOwnProperty(x, key)) {
- freezeDeeply(x[key]);
- }
- }
- }
- 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 {Rule} 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 = [
- { files: ["**"] }, // Make sure the default config matches for all files
- {
- 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
- },
- ...defaultConfig.slice(1)
- ];
- /**
- * 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;
- }
- 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 = 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}`);
- }
- }
- // 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}`);
- }
- if (hasOwnProperty(error, "line")) {
- assert.strictEqual(message.line, error.line, `Error line should be ${error.line}`);
- }
- if (hasOwnProperty(error, "column")) {
- assert.strictEqual(message.column, error.column, `Error column should be ${error.column}`);
- }
- if (hasOwnProperty(error, "endLine")) {
- assert.strictEqual(message.endLine, error.endLine, `Error endLine should be ${error.endLine}`);
- }
- if (hasOwnProperty(error, "endColumn")) {
- assert.strictEqual(message.endColumn, error.endColumn, `Error endColumn should be ${error.endColumn}`);
- }
- 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;
|