claude-flow 2.0.0-alpha.7 → 2.0.0-alpha.70

package/src/ui/extension/node_modules/eslint/lib/rule-tester/rule-tester.js

@@ -3,9 +3,7 @@
  * @author Ilya Volodin
  */
 "use strict";
-
 /* globals describe, it -- Mocha globals */
-
 /*
  * This is a wrapper around mocha to allow for DRY unittests for eslint
  * Format:
@@ -35,11 +33,9 @@
  * {errorNodeType} - AST node type that is returned by they rule as
  *                   a cause of the failure.
  */
-
 //------------------------------------------------------------------------------
 // Requirements
 //------------------------------------------------------------------------------
-
 const
     assert = require("assert"),
     path = require("path"),
@@ -50,22 +46,15 @@ const
     { getRuleOptionsSchema, validate } = require("../shared/config-validator"),
     { Linter, SourceCodeFixer, interpolate } = require("../linter"),
     CodePath = require("../linter/code-path-analysis/code-path");
-
 const ajv = require("../shared/ajv")({ strictDefaults: true });
-
 const espreePath = require.resolve("espree");
 const parserSymbol = Symbol.for("eslint.RuleTester.parser");
-
 const { SourceCode } = require("../source-code");
-
 //------------------------------------------------------------------------------
 // Typedefs
 //------------------------------------------------------------------------------
-
 /** @typedef {import("../shared/types").Parser} Parser */
 /** @typedef {import("../shared/types").Rule} Rule */
-
-
 /**
  * A test case that is expected to pass lint.
  * @typedef {Object} ValidTestCase
@@ -80,7 +69,6 @@ const { SourceCode } = require("../source-code");
  * @property {{ [name: string]: boolean }} [env] Environments for the test case.
  * @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
@@ -97,7 +85,6 @@ const { SourceCode } = require("../source-code");
  * @property {{ [name: string]: boolean }} [env] Environments for 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
@@ -110,18 +97,15 @@ const { SourceCode } = require("../source-code");
  * @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: {} };
 let defaultConfig = { rules: {} };
-
 /*
  * List every parameters possible on a test case that are not related to eslint
  * configuration
@@ -135,7 +119,6 @@ const RuleTesterParameters = [
     "output",
     "only"
 ];
-
 /*
  * All allowed property names in error objects.
  */
@@ -151,7 +134,6 @@ const errorObjectParameters = new Set([
     "suggestions"
 ]);
 const friendlyErrorObjectParameterList = `[${[...errorObjectParameters].map(key => `'${key}'`).join(", ")}]`;
-
 /*
  * All allowed property names in suggestion objects.
  */
@@ -162,21 +144,17 @@ const suggestionObjectParameters = new Set([
     "output"
 ]);
 const friendlySuggestionObjectParameterList = `[${[...suggestionObjectParameters].map(key => `'${key}'`).join(", ")}]`;
-
 const forbiddenMethods = [
     "applyInlineConfig",
     "applyLanguageOptions",
     "finalize"
 ];
-
 const hasOwnProperty = Function.call.bind(Object.hasOwnProperty);
-
 const DEPRECATED_SOURCECODE_PASSTHROUGHS = {
     getSource: "getText",
     getSourceLines: "getLines",
     getAllComments: "getAllComments",
     getNodeByRangeIndex: "getNodeByRangeIndex",
-
     // getComments: "getComments", -- already handled by a separate error
     getCommentsBefore: "getCommentsBefore",
     getCommentsAfter: "getCommentsAfter",
@@ -193,13 +171,11 @@ const DEPRECATED_SOURCECODE_PASSTHROUGHS = {
     getTokensAfter: "getTokensAfter",
     getTokensBefore: "getTokensBefore",
     getTokensBetween: "getTokensBetween",
-
     getScope: "getScope",
     getAncestors: "getAncestors",
     getDeclaredVariables: "getDeclaredVariables",
     markVariableAsUsed: "markVariableAsUsed"
 };
-
 /**
  * Clones a given value deeply.
  * Note: This ignores `parent` property.
@@ -211,21 +187,16 @@ function cloneDeeplyExcludesParent(x) {
         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.
@@ -245,7 +216,6 @@ function freezeDeeply(x) {
         Object.freeze(x);
     }
 }
-
 /**
  * Replace control characters by `\u00xx` form.
  * @param {string} text The text to sanitize.
@@ -260,7 +230,6 @@ function sanitize(text) {
         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.
@@ -285,8 +254,6 @@ function defineStartEndAsError(objName, node) {
         }
     });
 }
-
-
 /**
  * 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.
@@ -298,7 +265,6 @@ function defineStartEndAsErrorInTree(ast, visitorKeys) {
     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.
@@ -306,30 +272,25 @@ function defineStartEndAsErrorInTree(ast, visitorKeys) {
  * @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 `SourceCode.prototype.getComments`.
  * @returns {void}
@@ -340,7 +301,6 @@ function getCommentsDeprecation() {
         "`SourceCode#getComments()` is deprecated and will be removed in a future major version. Use `getCommentsBefore()`, `getCommentsAfter()`, and `getCommentsInside()` instead."
     );
 }
-
 /**
  * Function to replace forbidden `SourceCode` methods.
  * @param {string} methodName The name of the method to forbid.
@@ -353,7 +313,6 @@ function throwForbiddenMethodError(methodName) {
         );
     };
 }
-
 /**
  * Emit a deprecation warning if function-style format is being used.
  * @param {string} ruleName Name of the rule.
@@ -368,7 +327,6 @@ function emitLegacyRuleAPIWarning(ruleName) {
         );
     }
 }
-
 /**
  * Emit a deprecation warning if rule has options but is missing the "meta.schema" property
  * @param {string} ruleName Name of the rule.
@@ -383,7 +341,6 @@ function emitMissingSchemaWarning(ruleName) {
         );
     }
 }
-
 /**
  * Emit a deprecation warning if a rule uses a deprecated `context` method.
  * @param {string} ruleName Name of the rule.
@@ -399,7 +356,6 @@ function emitDeprecatedContextMethodWarning(ruleName, methodName) {
         );
     }
 }
-
 /**
  * Emit a deprecation warning if rule uses CodePath#currentSegments.
  * @param {string} ruleName Name of the rule.
@@ -414,7 +370,6 @@ function emitCodePathCurrentSegmentsWarning(ruleName) {
         );
     }
 }
-
 /**
  * Emit a deprecation warning if `context.parserServices` is used.
  * @param {string} ruleName Name of the rule.
@@ -429,17 +384,13 @@ function emitParserServicesWarning(ruleName) {
         );
     }
 }
-
-
 //------------------------------------------------------------------------------
 // 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}
@@ -458,7 +409,6 @@ function itDefaultHandler(text, method) {
         throw err;
     }
 }
-
 /**
  * This is `describe` default handler if `describe` don't exist.
  * @this {Mocha}
@@ -469,18 +419,15 @@ function itDefaultHandler(text, 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.
@@ -492,7 +439,6 @@ class RuleTester {
             testerConfig,
             { rules: { "rule-tester/validate-ast": "error" } }
         );
-
         /**
          * Rule definitions to define before tests.
          * @type {Object}
@@ -500,7 +446,6 @@ class RuleTester {
         this.rules = {};
         this.linter = new Linter();
     }
-
     /**
      * Set the configuration to use for all future tests
      * @param {Object} config the configuration to use.
@@ -512,11 +457,9 @@ class RuleTester {
             throw new TypeError("RuleTester.setDefaultConfig: config must be an object");
         }
         defaultConfig = config;
-
         // Make sure the rules object exists since it is assumed to exist later
         defaultConfig.rules = defaultConfig.rules || {};
     }
-
     /**
      * Get the current configuration used for all tests
      * @returns {Object} the current configuration
@@ -524,7 +467,6 @@ class RuleTester {
     static getDefaultConfig() {
         return defaultConfig;
     }
-
     /**
      * Reset the configuration to the initial configuration of the tester removing
      * any changes made until now.
@@ -533,8 +475,6 @@ class RuleTester {
     static resetDefaultConfig() {
         defaultConfig = merge({}, testerDefaultConfig);
     }
-
-
     /*
      * 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
@@ -546,22 +486,18 @@ class RuleTester {
             (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.
@@ -571,10 +507,8 @@ class RuleTester {
         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];
@@ -585,7 +519,6 @@ class RuleTester {
         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" +
@@ -597,11 +530,9 @@ class RuleTester {
         }
         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;
     }
-
     /**
      * Define a rule for one particular run of tests.
      * @param {string} name The name of the rule to define.
@@ -614,7 +545,6 @@ class RuleTester {
         }
         this.rules[name] = rule;
     }
-
     /**
      * Adds a new rule test to execute.
      * @param {string} ruleName The name of the rule to run.
@@ -628,40 +558,32 @@ class RuleTester {
      * @returns {void}
      */
     run(ruleName, rule, test) {
-
         const testerConfig = this.testerConfig,
             requiredScenarios = ["valid", "invalid"],
             scenarioErrors = [],
             linter = this.linter;
-
         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"));
         }
-
         if (typeof rule === "function") {
             emitLegacyRuleAPIWarning(ruleName);
         }
-
         linter.defineRule(ruleName, Object.assign({}, rule, {
-
             // Create a wrapper rule that freezes the `context` properties.
-            create(context) {
+            create(_context) {
                 freezeDeeply(context.options);
                 freezeDeeply(context.settings);
                 freezeDeeply(context.parserOptions);
-
                 // wrap all deprecated methods
                 const newContext = Object.create(
                     context,
@@ -669,10 +591,8 @@ class RuleTester {
                         methodName,
                         {
                             value(...args) {
-
                                 // emit deprecation warning
                                 emitDeprecatedContextMethodWarning(ruleName, methodName);
-
                                 // call the original method
                                 return context[methodName].call(this, ...args);
                             },
@@ -680,25 +600,19 @@ class RuleTester {
                         }
                     ]))
                 );
-
                 // emit warning about context.parserServices
                 const parserServices = context.parserServices;
-
                 Object.defineProperty(newContext, "parserServices", {
                     get() {
                         emitParserServicesWarning(ruleName);
                         return parserServices;
                     }
                 });
-
                 Object.freeze(newContext);
-
                 return (typeof rule === "function" ? rule : rule.create)(newContext);
             }
         }));
-
         linter.defineRules(this.rules);
-
         /**
          * Run the rule for the given item
          * @param {string|Object} item Item to run the rule against
@@ -709,22 +623,18 @@ class RuleTester {
         function runRuleForItem(item) {
             let config = merge({}, testerConfig),
                 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.
@@ -734,11 +644,9 @@ class RuleTester {
                     itemConfig
                 );
             }
-
             if (item.filename) {
                 filename = item.filename;
             }
-
             if (hasOwnProperty(item, "options")) {
                 assert(Array.isArray(item.options), "options must be an array");
                 if (
@@ -754,9 +662,7 @@ class RuleTester {
             } else {
                 config.rules[ruleName] = 1;
             }
-
             const schema = getRuleOptionsSchema(rule);
-
             /*
              * Setup AST getters.
              * The goal is to check whether or not AST was modified when
@@ -774,28 +680,21 @@ class RuleTester {
                     };
                 }
             });
-
             if (typeof config.parser === "string") {
                 assert(path.isAbsolute(config.parser), "Parsers provided as strings to RuleTester must be absolute paths");
             } else {
                 config.parser = espreePath;
             }
-
             linter.defineParser(config.parser, wrapParser(require(config.parser)));
-
             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
@@ -808,14 +707,11 @@ class RuleTester {
                     throw new Error(`Schema for rule ${ruleName} is invalid: ${err.message}`);
                 }
             }
-
             validate(config, "rule-tester", id => (id === ruleName ? rule : null));
-
             // Verify the code.
             const { getComments, applyLanguageOptions, applyInlineConfig, finalize } = SourceCode.prototype;
             const originalCurrentSegments = Object.getOwnPropertyDescriptor(CodePath.prototype, "currentSegments");
-            let messages;
-
+            let messages; // TODO: Remove if unused
             try {
                 SourceCode.prototype.getComments = getCommentsDeprecation;
                 Object.defineProperty(CodePath.prototype, "currentSegments", {
@@ -824,11 +720,9 @@ class RuleTester {
                         return originalCurrentSegments.get.call(this);
                     }
                 });
-
                 forbiddenMethods.forEach(methodName => {
                     SourceCode.prototype[methodName] = throwForbiddenMethodError(methodName);
                 });
-
                 messages = linter.verify(code, config, filename);
             } finally {
                 SourceCode.prototype.getComments = getComments;
@@ -837,16 +731,12 @@ class RuleTester {
                 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, config, filename).find(m => m.fatal);
-
                 assert(!errorMessageInFix, [
                     "A fatal parsing error occurred in autofix.",
                     `Error: ${errorMessageInFix && errorMessageInFix.message}`,
@@ -856,7 +746,6 @@ class RuleTester {
             } else {
                 output = code;
             }
-
             return {
                 messages,
                 output,
@@ -864,7 +753,6 @@ class RuleTester {
                 afterAST: cloneDeeplyExcludesParent(afterAST)
             };
         }
-
         /**
          * Check if the AST was changed
          * @param {ASTNode} beforeAST AST node before running
@@ -877,7 +765,6 @@ class RuleTester {
                 assert.fail("Rule should not modify AST.");
             }
         }
-
         /**
          * Check if the template is valid or not
          * all valid cases go through this
@@ -887,22 +774,17 @@ class RuleTester {
          */
         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");
             }
-
             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
@@ -914,7 +796,6 @@ class RuleTester {
          */
         function assertMessageMatches(actual, expected) {
             if (expected instanceof RegExp) {
-
                 // assert.js doesn't have a built-in RegExp match function
                 assert.ok(
                     expected.test(actual),
@@ -924,7 +805,6 @@ class RuleTester {
                 assert.strictEqual(actual, expected);
             }
         }
-
         /**
          * Check if the template is invalid or not
          * all invalid cases go through this.
@@ -939,23 +819,17 @@ class RuleTester {
             }
             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");
             }
-
             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;
-
             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",
@@ -971,34 +845,26 @@ class RuleTester {
                         util.inspect(messages)
                     )
                 );
-
                 const hasMessageOfThisRule = messages.some(m => m.ruleId === ruleName);
-
                 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);
                     } 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'.");
@@ -1017,7 +883,6 @@ class RuleTester {
                                 `messageId '${message.messageId}' does not match expected messageId '${error.messageId}'.`
                             );
                             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.
@@ -1025,7 +890,6 @@ class RuleTester {
                                  */
                                 const unformattedOriginalMessage = rule.meta.messages[error.messageId];
                                 const rehydratedMessage = interpolate(unformattedOriginalMessage, error.data);
-
                                 assert.strictEqual(
                                     message.message,
                                     rehydratedMessage,
@@ -1033,34 +897,26 @@ class RuleTester {
                                 );
                             }
                         }
-
                         assert.ok(
-                            hasOwnProperty(error, "data") ? hasOwnProperty(error, "messageId") : true,
+                            hasOwnProperty(error, "data") ? hasOwnProperty(_error, "messageId") : true,
                             "Error must specify 'messageId' if 'data' is used."
                         );
-
                         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}`);
                         }
-
                         if (hasOwnProperty(error, "suggestions")) {
-
                             // Support asserting there are no suggestions
                             if (!error.suggestions || (Array.isArray(error.suggestions) && error.suggestions.length === 0)) {
                                 if (Array.isArray(message.suggestions) && message.suggestions.length > 0) {
@@ -1069,7 +925,6 @@ class RuleTester {
                             } else {
                                 assert.strictEqual(Array.isArray(message.suggestions), true, `Error should have an array of suggestions. Instead received "${message.suggestions}" on error with message: "${message.message}"`);
                                 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,
@@ -1081,10 +936,8 @@ class RuleTester {
                                             `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"),
@@ -1096,7 +949,6 @@ class RuleTester {
                                             `${suggestionPrefix} desc should be "${expectedSuggestion.desc}" but got "${actualSuggestion.desc}" instead.`
                                         );
                                     }
-
                                     if (hasOwnProperty(expectedSuggestion, "messageId")) {
                                         assert.ok(
                                             ruleHasMetaMessages,
@@ -1114,7 +966,6 @@ class RuleTester {
                                         if (hasOwnProperty(expectedSuggestion, "data")) {
                                             const unformattedMetaMessage = rule.meta.messages[expectedSuggestion.messageId];
                                             const rehydratedDesc = interpolate(unformattedMetaMessage, expectedSuggestion.data);
-
                                             assert.strictEqual(
                                                 actualSuggestion.desc,
                                                 rehydratedDesc,
@@ -1127,23 +978,19 @@ class RuleTester {
                                             `${suggestionPrefix} Test must specify 'messageId' if 'data' is used.`
                                         );
                                     }
-
                                     if (hasOwnProperty(expectedSuggestion, "output")) {
                                         const codeWithAppliedSuggestion = SourceCodeFixer.applyFixes(item.code, [actualSuggestion]).output;
-
                                         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}"`);
                                     }
                                 });
                             }
                         }
                     } 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(
@@ -1161,10 +1008,8 @@ class RuleTester {
                     "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.
@@ -1184,7 +1029,6 @@ class RuleTester {
                     });
                 });
             }
-
             if (test.invalid.length > 0) {
                 this.constructor.describe("invalid", () => {
                     test.invalid.forEach(invalid => {
@@ -1200,7 +1044,5 @@ class RuleTester {
         });
     }
 }
-
 RuleTester[DESCRIBE] = RuleTester[IT] = RuleTester[IT_ONLY] = null;
-
-module.exports = RuleTester;
+module.exports = RuleTester;