eslint-plugin-jest 26.8.7 → 27.1.6

package/lib/rules/require-hook.js

@@ -4,49 +4,37 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
-
 var _utils = require("@typescript-eslint/utils");
-
 var _utils2 = require("./utils");
-
 const isJestFnCall = (node, context) => {
   var _getNodeName;
-
   if ((0, _utils2.parseJestFnCall)(node, context)) {
     return true;
   }
-
   return !!((_getNodeName = (0, _utils2.getNodeName)(node)) !== null && _getNodeName !== void 0 && _getNodeName.startsWith('jest.'));
 };
-
 const isNullOrUndefined = node => {
   return node.type === _utils.AST_NODE_TYPES.Literal && node.value === null || (0, _utils2.isIdentifier)(node, 'undefined');
 };
-
 const shouldBeInHook = (node, context, allowedFunctionCalls = []) => {
   switch (node.type) {
     case _utils.AST_NODE_TYPES.ExpressionStatement:
       return shouldBeInHook(node.expression, context, allowedFunctionCalls);
-
     case _utils.AST_NODE_TYPES.CallExpression:
       return !(isJestFnCall(node, context) || allowedFunctionCalls.includes((0, _utils2.getNodeName)(node)));
-
     case _utils.AST_NODE_TYPES.VariableDeclaration:
       {
         if (node.kind === 'const') {
           return false;
         }
-
         return node.declarations.some(({
           init
         }) => init !== null && !isNullOrUndefined(init));
       }
-
     default:
       return false;
   }
 };
-
 var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
@@ -75,14 +63,10 @@ var _default = (0, _utils2.createRule)({
   defaultOptions: [{
     allowedFunctionCalls: []
   }],
-
   create(context) {
-    var _context$options$;
-
     const {
       allowedFunctionCalls
-    } = (_context$options$ = context.options[0]) !== null && _context$options$ !== void 0 ? _context$options$ : {};
-
+    } = context.options[0] ?? {};
     const checkBlockBody = body => {
       for (const statement of body) {
         if (shouldBeInHook(statement, context, allowedFunctionCalls)) {
@@ -93,29 +77,21 @@ var _default = (0, _utils2.createRule)({
         }
       }
     };
-
     return {
       Program(program) {
         checkBlockBody(program.body);
       },
-
       CallExpression(node) {
         if (!(0, _utils2.isTypeOfJestFnCall)(node, context, ['describe']) || node.arguments.length < 2) {
           return;
         }
-
         const [, testFn] = node.arguments;
-
         if (!(0, _utils2.isFunction)(testFn) || testFn.body.type !== _utils.AST_NODE_TYPES.BlockStatement) {
           return;
         }
-
         checkBlockBody(testFn.body.body);
       }
-
     };
   }
-
 });
-
 exports.default = _default;

package/lib/rules/require-to-throw-message.js

@@ -4,9 +4,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
-
 var _utils = require("./utils");
-
 var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
@@ -22,21 +20,17 @@ var _default = (0, _utils.createRule)({
     schema: []
   },
   defaultOptions: [],
-
   create(context) {
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils.parseJestFnCall)(node, context);
-
         if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'expect') {
           return;
         }
-
         const {
           matcher
         } = jestFnCall;
         const matcherName = (0, _utils.getAccessorValue)(matcher);
-
         if (jestFnCall.args.length === 0 && ['toThrow', 'toThrowError'].includes(matcherName) && !jestFnCall.modifiers.some(nod => (0, _utils.getAccessorValue)(nod) === 'not')) {
           // Look for `toThrow` calls with no arguments.
           context.report({
@@ -48,10 +42,7 @@ var _default = (0, _utils.createRule)({
           });
         }
       }
-
     };
   }
-
 });
-
 exports.default = _default;

package/lib/rules/require-top-level-describe.js

@@ -4,15 +4,12 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
-
 var _utils = require("./utils");
-
 const messages = {
   tooManyDescribes: 'There should not be more than {{ max }} describe{{ s }} at the top level',
   unexpectedTestCase: 'All test cases must be wrapped in a describe block.',
   unexpectedHook: 'All hooks must be wrapped in a describe block.'
 };
-
 var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
@@ -35,29 +32,22 @@ var _default = (0, _utils.createRule)({
     }]
   },
   defaultOptions: [{}],
-
   create(context) {
-    var _context$options$;
-
     const {
       maxNumberOfTopLevelDescribes = Infinity
-    } = (_context$options$ = context.options[0]) !== null && _context$options$ !== void 0 ? _context$options$ : {};
+    } = context.options[0] ?? {};
     let numberOfTopLevelDescribeBlocks = 0;
     let numberOfDescribeBlocks = 0;
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils.parseJestFnCall)(node, context);
-
         if (!jestFnCall) {
           return;
         }
-
         if (jestFnCall.type === 'describe') {
           numberOfDescribeBlocks++;
-
           if (numberOfDescribeBlocks === 1) {
             numberOfTopLevelDescribeBlocks++;
-
             if (numberOfTopLevelDescribeBlocks > maxNumberOfTopLevelDescribes) {
               context.report({
                 node,
@@ -69,10 +59,8 @@ var _default = (0, _utils.createRule)({
               });
             }
           }
-
           return;
         }
-
         if (numberOfDescribeBlocks === 0) {
           if (jestFnCall.type === 'test') {
             context.report({
@@ -81,7 +69,6 @@ var _default = (0, _utils.createRule)({
             });
             return;
           }
-
           if (jestFnCall.type === 'hook') {
             context.report({
               node,
@@ -91,16 +78,12 @@ var _default = (0, _utils.createRule)({
           }
         }
       },
-
       'CallExpression:exit'(node) {
         if ((0, _utils.isTypeOfJestFnCall)(node, context, ['describe'])) {
           numberOfDescribeBlocks--;
         }
       }
-
     };
   }
-
 });
-
 exports.default = _default;

package/lib/rules/unbound-method.js

@@ -4,40 +4,23 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
-
 var _utils = require("@typescript-eslint/utils");
-
 var _utils2 = require("./utils");
-
 const toThrowMatchers = ['toThrow', 'toThrowError', 'toThrowErrorMatchingSnapshot', 'toThrowErrorMatchingInlineSnapshot'];
-
 const baseRule = (() => {
   try {
     // eslint-disable-next-line @typescript-eslint/no-require-imports
     const TSESLintPlugin = require('@typescript-eslint/eslint-plugin');
-
     return TSESLintPlugin.rules['unbound-method'];
   } catch (e) {
     const error = e;
-
     if (error.code === 'MODULE_NOT_FOUND') {
       return null;
     }
-
     throw error;
   }
 })();
-
-const tryCreateBaseRule = context => {
-  try {
-    return baseRule === null || baseRule === void 0 ? void 0 : baseRule.create(context);
-  } catch {
-    return null;
-  }
-};
-
 const DEFAULT_MESSAGE = 'This rule requires `@typescript-eslint/eslint-plugin`';
-
 var _default = (0, _utils2.createRule)({
   defaultOptions: [{
     ignoreStatic: false
@@ -60,39 +43,29 @@ var _default = (0, _utils2.createRule)({
       recommended: false
     }
   },
-
   create(context) {
-    const baseSelectors = tryCreateBaseRule(context);
-
+    const baseSelectors = baseRule === null || baseRule === void 0 ? void 0 : baseRule.create(context);
     if (!baseSelectors) {
       return {};
     }
-
-    return { ...baseSelectors,
-
+    return {
+      ...baseSelectors,
       MemberExpression(node) {
         var _node$parent, _baseSelectors$Member;
-
         if (((_node$parent = node.parent) === null || _node$parent === void 0 ? void 0 : _node$parent.type) === _utils.AST_NODE_TYPES.CallExpression) {
           const jestFnCall = (0, _utils2.parseJestFnCall)((0, _utils2.findTopMostCallExpression)(node.parent), context);
-
           if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) === 'expect') {
             const {
               matcher
             } = jestFnCall;
-
             if (!toThrowMatchers.includes((0, _utils2.getAccessorValue)(matcher))) {
               return;
             }
           }
         }
-
         (_baseSelectors$Member = baseSelectors.MemberExpression) === null || _baseSelectors$Member === void 0 ? void 0 : _baseSelectors$Member.call(baseSelectors, node);
       }
-
     };
   }
-
 });
-
 exports.default = _default;

package/lib/rules/utils/accessors.js

@@ -4,9 +4,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.isSupportedAccessor = exports.isStringNode = exports.isIdentifier = exports.getStringValue = exports.getAccessorValue = void 0;
-
 var _utils = require("@typescript-eslint/utils");
-
 /**
  * Checks if the given `node` is a `StringLiteral`.
  *
@@ -21,7 +19,6 @@ var _utils = require("@typescript-eslint/utils");
  * @template V
  */
 const isStringLiteral = (node, value) => node.type === _utils.AST_NODE_TYPES.Literal && typeof node.value === 'string' && (value === undefined || node.value === value);
-
 /**
  * Checks if the given `node` is a `TemplateLiteral`.
  *
@@ -37,9 +34,9 @@ const isStringLiteral = (node, value) => node.type === _utils.AST_NODE_TYPES.Lit
  *
  * @template V
  */
-const isTemplateLiteral = (node, value) => node.type === _utils.AST_NODE_TYPES.TemplateLiteral && node.quasis.length === 1 && ( // bail out if not simple
+const isTemplateLiteral = (node, value) => node.type === _utils.AST_NODE_TYPES.TemplateLiteral && node.quasis.length === 1 && (
+// bail out if not simple
 value === undefined || node.quasis[0].value.raw === value);
-
 /**
  * Checks if the given `node` is a {@link StringNode}.
  *
@@ -51,6 +48,7 @@ value === undefined || node.quasis[0].value.raw === value);
  * @template V
  */
 const isStringNode = (node, specifics) => isStringLiteral(node, specifics) || isTemplateLiteral(node, specifics);
+
 /**
  * Gets the value of the given `StringNode`.
  *
@@ -63,18 +61,13 @@ const isStringNode = (node, specifics) => isStringLiteral(node, specifics) || is
  *
  * @template S
  */
-
-
 exports.isStringNode = isStringNode;
-
 const getStringValue = node => isTemplateLiteral(node) ? node.quasis[0].value.raw : node.value;
+
 /**
  * An `Identifier` with a known `name` value - i.e `expect`.
  */
-
-
 exports.getStringValue = getStringValue;
-
 /**
  * Checks if the given `node` is an `Identifier`.
  *
@@ -89,6 +82,7 @@ exports.getStringValue = getStringValue;
  * @template V
  */
 const isIdentifier = (node, name) => node.type === _utils.AST_NODE_TYPES.Identifier && (name === undefined || node.name === name);
+
 /**
  * Checks if the given `node` is a "supported accessor".
  *
@@ -111,11 +105,9 @@ const isIdentifier = (node, name) => node.type === _utils.AST_NODE_TYPES.Identif
  *
  * @template V
  */
-
-
 exports.isIdentifier = isIdentifier;
-
 const isSupportedAccessor = (node, value) => isIdentifier(node, value) || isStringNode(node, value);
+
 /**
  * Gets the value of the given `AccessorNode`,
  * account for the different node types.
@@ -126,10 +118,6 @@ const isSupportedAccessor = (node, value) => isIdentifier(node, value) || isStri
  *
  * @template S
  */
-
-
 exports.isSupportedAccessor = isSupportedAccessor;
-
 const getAccessorValue = accessor => accessor.type === _utils.AST_NODE_TYPES.Identifier ? accessor.name : getStringValue(accessor);
-
 exports.getAccessorValue = getAccessorValue;

package/lib/rules/utils/detectJestVersion.js

@@ -5,25 +5,20 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.detectJestVersion = void 0;
 let cachedJestVersion = null;
-
 const detectJestVersion = () => {
   if (cachedJestVersion) {
     return cachedJestVersion;
   }
-
   try {
     const jestPath = require.resolve('jest/package.json');
-
-    const jestPackageJson = // eslint-disable-next-line @typescript-eslint/no-require-imports
+    const jestPackageJson =
+    // eslint-disable-next-line @typescript-eslint/no-require-imports
     require(jestPath);
-
     if (jestPackageJson.version) {
       const [majorVersion] = jestPackageJson.version.split('.');
       return cachedJestVersion = parseInt(majorVersion, 10);
     }
   } catch {}
-
   throw new Error('Unable to detect Jest version - please ensure jest package is installed, or otherwise set version explicitly');
 };
-
 exports.detectJestVersion = detectJestVersion;

package/lib/rules/utils/followTypeAssertionChain.js

@@ -4,11 +4,7 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.followTypeAssertionChain = void 0;
-
 var _utils = require("@typescript-eslint/utils");
-
 const isTypeCastExpression = node => node.type === _utils.AST_NODE_TYPES.TSAsExpression || node.type === _utils.AST_NODE_TYPES.TSTypeAssertion;
-
 const followTypeAssertionChain = expression => isTypeCastExpression(expression) ? followTypeAssertionChain(expression.expression) : expression;
-
 exports.followTypeAssertionChain = followTypeAssertionChain;

package/lib/rules/utils/index.js

@@ -3,9 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-
 var _accessors = require("./accessors");
-
 Object.keys(_accessors).forEach(function (key) {
   if (key === "default" || key === "__esModule") return;
   if (key in exports && exports[key] === _accessors[key]) return;
@@ -16,9 +14,7 @@ Object.keys(_accessors).forEach(function (key) {
     }
   });
 });
-
 var _detectJestVersion = require("./detectJestVersion");
-
 Object.keys(_detectJestVersion).forEach(function (key) {
   if (key === "default" || key === "__esModule") return;
   if (key in exports && exports[key] === _detectJestVersion[key]) return;
@@ -29,9 +25,7 @@ Object.keys(_detectJestVersion).forEach(function (key) {
     }
   });
 });
-
 var _followTypeAssertionChain = require("./followTypeAssertionChain");
-
 Object.keys(_followTypeAssertionChain).forEach(function (key) {
   if (key === "default" || key === "__esModule") return;
   if (key in exports && exports[key] === _followTypeAssertionChain[key]) return;
@@ -42,9 +36,7 @@ Object.keys(_followTypeAssertionChain).forEach(function (key) {
     }
   });
 });
-
 var _misc = require("./misc");
-
 Object.keys(_misc).forEach(function (key) {
   if (key === "default" || key === "__esModule") return;
   if (key in exports && exports[key] === _misc[key]) return;
@@ -55,9 +47,7 @@ Object.keys(_misc).forEach(function (key) {
     }
   });
 });
-
 var _parseJestFnCall = require("./parseJestFnCall");
-
 Object.keys(_parseJestFnCall).forEach(function (key) {
   if (key === "default" || key === "__esModule") return;
   if (key in exports && exports[key] === _parseJestFnCall[key]) return;

package/lib/rules/utils/misc.js

@@ -5,33 +5,23 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.getFirstMatcherArg = exports.findTopMostCallExpression = exports.createRule = exports.TestCaseName = exports.ModifierName = exports.HookName = exports.EqualityMatcher = exports.DescribeAlias = void 0;
 exports.getNodeName = getNodeName;
-exports.replaceAccessorFixer = exports.isFunction = exports.isBooleanLiteral = exports.hasOnlyOneArgument = exports.getTestCallExpressionsFromDeclaredVariables = void 0;
-
+exports.replaceAccessorFixer = exports.removeExtraArgumentsFixer = exports.isFunction = exports.isBooleanLiteral = exports.hasOnlyOneArgument = exports.getTestCallExpressionsFromDeclaredVariables = void 0;
 var _path = require("path");
-
 var _utils = require("@typescript-eslint/utils");
-
 var _package = require("../../../package.json");
-
 var _accessors = require("./accessors");
-
 var _followTypeAssertionChain = require("./followTypeAssertionChain");
-
 var _parseJestFnCall = require("./parseJestFnCall");
-
 const REPO_URL = 'https://github.com/jest-community/eslint-plugin-jest';
-
 const createRule = _utils.ESLintUtils.RuleCreator(name => {
   const ruleName = (0, _path.parse)(name).name;
   return `${REPO_URL}/blob/v${_package.version}/docs/rules/${ruleName}.md`;
 });
+
 /**
  * Represents a `MemberExpression` with a "known" `property`.
  */
-
-
 exports.createRule = createRule;
-
 /**
  * Guards that the given `call` has only one `argument`.
  *
@@ -40,20 +30,16 @@ exports.createRule = createRule;
  * @return {call is CallExpressionWithSingleArgument}
  */
 const hasOnlyOneArgument = call => call.arguments.length === 1;
-
 exports.hasOnlyOneArgument = hasOnlyOneArgument;
 let DescribeAlias;
 exports.DescribeAlias = DescribeAlias;
-
 (function (DescribeAlias) {
   DescribeAlias["describe"] = "describe";
   DescribeAlias["fdescribe"] = "fdescribe";
   DescribeAlias["xdescribe"] = "xdescribe";
 })(DescribeAlias || (exports.DescribeAlias = DescribeAlias = {}));
-
 let TestCaseName;
 exports.TestCaseName = TestCaseName;
-
 (function (TestCaseName) {
   TestCaseName["fit"] = "fit";
   TestCaseName["it"] = "it";
@@ -61,61 +47,46 @@ exports.TestCaseName = TestCaseName;
   TestCaseName["xit"] = "xit";
   TestCaseName["xtest"] = "xtest";
 })(TestCaseName || (exports.TestCaseName = TestCaseName = {}));
-
 let HookName;
 exports.HookName = HookName;
-
 (function (HookName) {
   HookName["beforeAll"] = "beforeAll";
   HookName["beforeEach"] = "beforeEach";
   HookName["afterAll"] = "afterAll";
   HookName["afterEach"] = "afterEach";
 })(HookName || (exports.HookName = HookName = {}));
-
 let ModifierName;
 exports.ModifierName = ModifierName;
-
 (function (ModifierName) {
   ModifierName["not"] = "not";
   ModifierName["rejects"] = "rejects";
   ModifierName["resolves"] = "resolves";
 })(ModifierName || (exports.ModifierName = ModifierName = {}));
-
 let EqualityMatcher;
 exports.EqualityMatcher = EqualityMatcher;
-
 (function (EqualityMatcher) {
   EqualityMatcher["toBe"] = "toBe";
   EqualityMatcher["toEqual"] = "toEqual";
   EqualityMatcher["toStrictEqual"] = "toStrictEqual";
 })(EqualityMatcher || (exports.EqualityMatcher = EqualityMatcher = {}));
-
 const joinNames = (a, b) => a && b ? `${a}.${b}` : null;
-
 function getNodeName(node) {
   if ((0, _accessors.isSupportedAccessor)(node)) {
     return (0, _accessors.getAccessorValue)(node);
   }
-
   switch (node.type) {
     case _utils.AST_NODE_TYPES.TaggedTemplateExpression:
       return getNodeName(node.tag);
-
     case _utils.AST_NODE_TYPES.MemberExpression:
       return joinNames(getNodeName(node.object), getNodeName(node.property));
-
     case _utils.AST_NODE_TYPES.NewExpression:
     case _utils.AST_NODE_TYPES.CallExpression:
       return getNodeName(node.callee);
   }
-
   return null;
 }
-
 const isFunction = node => node.type === _utils.AST_NODE_TYPES.FunctionExpression || node.type === _utils.AST_NODE_TYPES.ArrowFunctionExpression;
-
 exports.isFunction = isFunction;
-
 const getTestCallExpressionsFromDeclaredVariables = (declaredVariables, context) => {
   return declaredVariables.reduce((acc, {
     references
@@ -123,59 +94,55 @@ const getTestCallExpressionsFromDeclaredVariables = (declaredVariables, context)
     identifier
   }) => identifier.parent).filter(node => (node === null || node === void 0 ? void 0 : node.type) === _utils.AST_NODE_TYPES.CallExpression && (0, _parseJestFnCall.isTypeOfJestFnCall)(node, context, ['test']))), []);
 };
+
 /**
  * Replaces an accessor node with the given `text`, surrounding it in quotes if required.
  *
  * This ensures that fixes produce valid code when replacing both dot-based and
  * bracket-based property accessors.
  */
-
-
 exports.getTestCallExpressionsFromDeclaredVariables = getTestCallExpressionsFromDeclaredVariables;
-
 const replaceAccessorFixer = (fixer, node, text) => {
   return fixer.replaceText(node, node.type === _utils.AST_NODE_TYPES.Identifier ? text : `'${text}'`);
 };
-
 exports.replaceAccessorFixer = replaceAccessorFixer;
-
+const removeExtraArgumentsFixer = (fixer, context, func, from) => {
+  const firstArg = func.arguments[from];
+  const lastArg = func.arguments[func.arguments.length - 1];
+  const sourceCode = context.getSourceCode();
+  let tokenAfterLastParam = sourceCode.getTokenAfter(lastArg);
+  if (tokenAfterLastParam.value === ',') {
+    tokenAfterLastParam = sourceCode.getTokenAfter(tokenAfterLastParam);
+  }
+  return fixer.removeRange([firstArg.range[0], tokenAfterLastParam.range[0]]);
+};
+exports.removeExtraArgumentsFixer = removeExtraArgumentsFixer;
 const findTopMostCallExpression = node => {
   let topMostCallExpression = node;
   let {
     parent
   } = node;
-
   while (parent) {
     if (parent.type === _utils.AST_NODE_TYPES.CallExpression) {
       topMostCallExpression = parent;
       parent = parent.parent;
       continue;
     }
-
     if (parent.type !== _utils.AST_NODE_TYPES.MemberExpression) {
       break;
     }
-
     parent = parent.parent;
   }
-
   return topMostCallExpression;
 };
-
 exports.findTopMostCallExpression = findTopMostCallExpression;
-
 const isBooleanLiteral = node => node.type === _utils.AST_NODE_TYPES.Literal && typeof node.value === 'boolean';
-
 exports.isBooleanLiteral = isBooleanLiteral;
-
 const getFirstMatcherArg = expectFnCall => {
   const [firstArg] = expectFnCall.args;
-
   if (firstArg.type === _utils.AST_NODE_TYPES.SpreadElement) {
     return firstArg;
   }
-
   return (0, _followTypeAssertionChain.followTypeAssertionChain)(firstArg);
 };
-
 exports.getFirstMatcherArg = getFirstMatcherArg;