eslint-plugin-jest 27.1.2 → 27.1.4

package/lib/processors/snapshot-processor.js

@@ -4,14 +4,12 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.preprocess = exports.postprocess = void 0;
-
 // https://eslint.org/docs/developer-guide/working-with-plugins#processors-in-plugins
 // https://github.com/typescript-eslint/typescript-eslint/issues/808
-const preprocess = source => [source];
 
+const preprocess = source => [source];
 exports.preprocess = preprocess;
-
-const postprocess = messages => // snapshot files should only be linted with snapshot specific rules
+const postprocess = messages =>
+// snapshot files should only be linted with snapshot specific rules
 messages[0].filter(message => message.ruleId === 'jest/no-large-snapshots');
-
 exports.postprocess = postprocess;

package/lib/rules/consistent-test-it.js

@@ -4,19 +4,15 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
-
 var _utils = require("@typescript-eslint/utils");
-
 var _utils2 = require("./utils");
-
 const buildFixer = (callee, nodeName, preferredTestKeyword) => fixer => [fixer.replaceText(callee.type === _utils.AST_NODE_TYPES.MemberExpression ? callee.object : callee, getPreferredNodeName(nodeName, preferredTestKeyword))];
-
 var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
       category: 'Best Practices',
-      description: 'Have control over `test` and `it` usages',
+      description: 'Enforce `test` and `it` usage conventions',
       recommended: false
     },
     fixable: 'code',
@@ -42,7 +38,6 @@ var _default = (0, _utils2.createRule)({
     fn: _utils2.TestCaseName.test,
     withinDescribe: _utils2.TestCaseName.it
   }],
-
   create(context) {
     const configObj = context.options[0] || {};
     const testKeyword = configObj.fn || _utils2.TestCaseName.test;
@@ -51,18 +46,14 @@ var _default = (0, _utils2.createRule)({
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
-
         if (!jestFnCall) {
           return;
         }
-
         if (jestFnCall.type === 'describe') {
           describeNestingLevel++;
           return;
         }
-
         const funcNode = node.callee.type === _utils.AST_NODE_TYPES.TaggedTemplateExpression ? node.callee.tag : node.callee.type === _utils.AST_NODE_TYPES.CallExpression ? node.callee.callee : node.callee;
-
         if (jestFnCall.type === 'test' && describeNestingLevel === 0 && !jestFnCall.name.endsWith(testKeyword)) {
           const oppositeTestKeyword = getOppositeTestKeyword(testKeyword);
           context.report({
@@ -75,7 +66,6 @@ var _default = (0, _utils2.createRule)({
             fix: buildFixer(funcNode, jestFnCall.name, testKeyword)
           });
         }
-
         if (jestFnCall.type === 'test' && describeNestingLevel > 0 && !jestFnCall.name.endsWith(testKeywordWithinDescribe)) {
           const oppositeTestKeyword = getOppositeTestKeyword(testKeywordWithinDescribe);
           context.report({
@@ -89,32 +79,24 @@ var _default = (0, _utils2.createRule)({
           });
         }
       },
-
       'CallExpression:exit'(node) {
         if ((0, _utils2.isTypeOfJestFnCall)(node, context, ['describe'])) {
           describeNestingLevel--;
         }
       }
-
     };
   }
-
 });
-
 exports.default = _default;
-
 function getPreferredNodeName(nodeName, preferredTestKeyword) {
   if (nodeName === _utils2.TestCaseName.fit) {
     return 'test.only';
   }
-
   return nodeName.startsWith('f') || nodeName.startsWith('x') ? nodeName.charAt(0) + preferredTestKeyword : preferredTestKeyword;
 }
-
 function getOppositeTestKeyword(test) {
   if (test === _utils2.TestCaseName.test) {
     return _utils2.TestCaseName.it;
   }
-
   return _utils2.TestCaseName.test;
 }

package/lib/rules/expect-expect.js

@@ -4,11 +4,8 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
-
 var _utils = require("@typescript-eslint/utils");
-
 var _utils2 = require("./utils");
-
 /*
  * This implementation is adapted from eslint-plugin-jasmine.
  * MIT license, Remco Haszing.
@@ -27,7 +24,6 @@ function matchesAssertFunctionName(nodeName, patterns) {
     return x.replace(/\*/gu, '[a-z\\d]*');
   }).join('\\.')}(\\.|$)`, 'ui').test(nodeName));
 }
-
 var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
@@ -63,56 +59,45 @@ var _default = (0, _utils2.createRule)({
     assertFunctionNames: ['expect'],
     additionalTestBlockFunctions: []
   }],
-
   create(context, [{
     assertFunctionNames = ['expect'],
     additionalTestBlockFunctions = []
   }]) {
     const unchecked = [];
-
     function checkCallExpressionUsed(nodes) {
       for (const node of nodes) {
         const index = node.type === _utils.AST_NODE_TYPES.CallExpression ? unchecked.indexOf(node) : -1;
-
         if (node.type === _utils.AST_NODE_TYPES.FunctionDeclaration) {
           const declaredVariables = context.getDeclaredVariables(node);
           const testCallExpressions = (0, _utils2.getTestCallExpressionsFromDeclaredVariables)(declaredVariables, context);
           checkCallExpressionUsed(testCallExpressions);
         }
-
         if (index !== -1) {
           unchecked.splice(index, 1);
           break;
         }
       }
     }
-
     return {
       CallExpression(node) {
         const name = (0, _utils2.getNodeName)(node.callee) ?? '';
-
         if ((0, _utils2.isTypeOfJestFnCall)(node, context, ['test']) || additionalTestBlockFunctions.includes(name)) {
           if (node.callee.type === _utils.AST_NODE_TYPES.MemberExpression && (0, _utils2.isSupportedAccessor)(node.callee.property, 'todo')) {
             return;
           }
-
           unchecked.push(node);
         } else if (matchesAssertFunctionName(name, assertFunctionNames)) {
           // Return early in case of nested `it` statements.
           checkCallExpressionUsed(context.getAncestors());
         }
       },
-
       'Program:exit'() {
         unchecked.forEach(node => context.report({
           messageId: 'noAssertions',
           node
         }));
       }
-
     };
   }
-
 });
-
 exports.default = _default;

package/lib/rules/max-expects.js

@@ -4,11 +4,8 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
-
 var _utils = require("@typescript-eslint/utils");
-
 var _utils2 = require("./utils");
-
 var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
@@ -35,39 +32,29 @@ var _default = (0, _utils2.createRule)({
   defaultOptions: [{
     max: 5
   }],
-
   create(context, [{
     max
   }]) {
     let count = 0;
-
     const maybeResetCount = node => {
       var _node$parent;
-
       const isTestFn = ((_node$parent = node.parent) === null || _node$parent === void 0 ? void 0 : _node$parent.type) !== _utils.AST_NODE_TYPES.CallExpression || (0, _utils2.isTypeOfJestFnCall)(node.parent, context, ['test']);
-
       if (isTestFn) {
         count = 0;
       }
     };
-
     return {
       FunctionExpression: maybeResetCount,
       'FunctionExpression:exit': maybeResetCount,
       ArrowFunctionExpression: maybeResetCount,
       'ArrowFunctionExpression:exit': maybeResetCount,
-
       CallExpression(node) {
         var _jestFnCall$head$node;
-
         const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
-
         if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'expect' || ((_jestFnCall$head$node = jestFnCall.head.node.parent) === null || _jestFnCall$head$node === void 0 ? void 0 : _jestFnCall$head$node.type) === _utils.AST_NODE_TYPES.MemberExpression) {
           return;
         }
-
         count += 1;
-
         if (count > max) {
           context.report({
             node,
@@ -79,10 +66,7 @@ var _default = (0, _utils2.createRule)({
           });
         }
       }
-
     };
   }
-
 });
-
 exports.default = _default;

package/lib/rules/max-nested-describe.js

@@ -4,11 +4,8 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
-
 var _utils = require("@typescript-eslint/utils");
-
 var _utils2 = require("./utils");
-
 var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
@@ -35,23 +32,18 @@ var _default = (0, _utils2.createRule)({
   defaultOptions: [{
     max: 5
   }],
-
   create(context, [{
     max
   }]) {
     const describeCallbackStack = [];
-
     function pushDescribeCallback(node) {
       const {
         parent
       } = node;
-
       if ((parent === null || parent === void 0 ? void 0 : parent.type) !== _utils.AST_NODE_TYPES.CallExpression || !(0, _utils2.isTypeOfJestFnCall)(parent, context, ['describe'])) {
         return;
       }
-
       describeCallbackStack.push(0);
-
       if (describeCallbackStack.length > max) {
         context.report({
           node: parent,
@@ -63,17 +55,14 @@ var _default = (0, _utils2.createRule)({
         });
       }
     }
-
     function popDescribeCallback(node) {
       const {
         parent
       } = node;
-
       if ((parent === null || parent === void 0 ? void 0 : parent.type) === _utils.AST_NODE_TYPES.CallExpression && (0, _utils2.isTypeOfJestFnCall)(parent, context, ['describe'])) {
         describeCallbackStack.pop();
       }
     }
-
     return {
       FunctionExpression: pushDescribeCallback,
       'FunctionExpression:exit': popDescribeCallback,
@@ -81,7 +70,5 @@ var _default = (0, _utils2.createRule)({
       'ArrowFunctionExpression:exit': popDescribeCallback
     };
   }
-
 });
-
 exports.default = _default;

package/lib/rules/no-alias-methods.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: {
@@ -23,7 +21,6 @@ var _default = (0, _utils.createRule)({
     schema: []
   },
   defaultOptions: [],
-
   create(context) {
     // map of jest matcher aliases & their canonical names
     const methodNames = {
@@ -42,16 +39,13 @@ var _default = (0, _utils.createRule)({
     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 alias = (0, _utils.getAccessorValue)(matcher);
-
         if (alias in methodNames) {
           const canonical = methodNames[alias];
           context.report({
@@ -65,10 +59,7 @@ var _default = (0, _utils.createRule)({
           });
         }
       }
-
     };
   }
-
 });
-
 exports.default = _default;

package/lib/rules/no-commented-out-tests.js

@@ -4,13 +4,10 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
-
 var _utils = require("./utils");
-
 function hasTests(node) {
   return /^\s*[xf]?(test|it|describe)(\.\w+|\[['"]\w+['"]\])?\s*\(/mu.test(node.value);
 }
-
 var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
@@ -26,30 +23,23 @@ var _default = (0, _utils.createRule)({
     type: 'suggestion'
   },
   defaultOptions: [],
-
   create(context) {
     const sourceCode = context.getSourceCode();
-
     function checkNode(node) {
       if (!hasTests(node)) {
         return;
       }
-
       context.report({
         messageId: 'commentedTests',
         node
       });
     }
-
     return {
       Program() {
         const comments = sourceCode.getAllComments();
         comments.forEach(checkNode);
       }
-
     };
   }
-
 });
-
 exports.default = _default;

package/lib/rules/no-conditional-expect.js

@@ -4,18 +4,14 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
-
 var _utils = require("@typescript-eslint/utils");
-
 var _utils2 = require("./utils");
-
 const isCatchCall = node => node.callee.type === _utils.AST_NODE_TYPES.MemberExpression && (0, _utils2.isSupportedAccessor)(node.callee.property, 'catch');
-
 var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      description: 'Prevent calling `expect` conditionally',
+      description: 'Disallow calling `expect` conditionally',
       category: 'Best Practices',
       recommended: 'error'
     },
@@ -26,46 +22,36 @@ var _default = (0, _utils2.createRule)({
     schema: []
   },
   defaultOptions: [],
-
   create(context) {
     let conditionalDepth = 0;
     let inTestCase = false;
     let inPromiseCatch = false;
-
     const increaseConditionalDepth = () => inTestCase && conditionalDepth++;
-
     const decreaseConditionalDepth = () => inTestCase && conditionalDepth--;
-
     return {
       FunctionDeclaration(node) {
         const declaredVariables = context.getDeclaredVariables(node);
         const testCallExpressions = (0, _utils2.getTestCallExpressionsFromDeclaredVariables)(declaredVariables, context);
-
         if (testCallExpressions.length > 0) {
           inTestCase = true;
         }
       },
-
       CallExpression(node) {
         const {
           type: jestFnCallType
         } = (0, _utils2.parseJestFnCall)(node, context) ?? {};
-
         if (jestFnCallType === 'test') {
           inTestCase = true;
         }
-
         if (isCatchCall(node)) {
           inPromiseCatch = true;
         }
-
         if (inTestCase && jestFnCallType === 'expect' && conditionalDepth > 0) {
           context.report({
             messageId: 'conditionalExpect',
             node
           });
         }
-
         if (inPromiseCatch && jestFnCallType === 'expect') {
           context.report({
             messageId: 'conditionalExpect',
@@ -73,17 +59,14 @@ var _default = (0, _utils2.createRule)({
           });
         }
       },
-
       'CallExpression:exit'(node) {
         if ((0, _utils2.isTypeOfJestFnCall)(node, context, ['test'])) {
           inTestCase = false;
         }
-
         if (isCatchCall(node)) {
           inPromiseCatch = false;
         }
       },
-
       CatchClause: increaseConditionalDepth,
       'CatchClause:exit': decreaseConditionalDepth,
       IfStatement: increaseConditionalDepth,
@@ -96,7 +79,5 @@ var _default = (0, _utils2.createRule)({
       'LogicalExpression:exit': decreaseConditionalDepth
     };
   }
-
 });
-
 exports.default = _default;

package/lib/rules/no-conditional-in-test.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,10 +20,8 @@ var _default = (0, _utils.createRule)({
     schema: []
   },
   defaultOptions: [],
-
   create(context) {
     let inTestCase = false;
-
     const maybeReportConditional = node => {
       if (inTestCase) {
         context.report({
@@ -34,27 +30,22 @@ var _default = (0, _utils.createRule)({
         });
       }
     };
-
     return {
       CallExpression(node) {
         if ((0, _utils.isTypeOfJestFnCall)(node, context, ['test'])) {
           inTestCase = true;
         }
       },
-
       'CallExpression:exit'(node) {
         if ((0, _utils.isTypeOfJestFnCall)(node, context, ['test'])) {
           inTestCase = false;
         }
       },
-
       IfStatement: maybeReportConditional,
       SwitchStatement: maybeReportConditional,
       ConditionalExpression: maybeReportConditional,
       LogicalExpression: maybeReportConditional
     };
   }
-
 });
-
 exports.default = _default;

package/lib/rules/no-deprecated-functions.js

@@ -4,20 +4,15 @@ Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.default = void 0;
-
 var _utils = require("@typescript-eslint/utils");
-
 var _utils2 = require("./utils");
-
 const parseJestVersion = rawVersion => {
   if (typeof rawVersion === 'number') {
     return rawVersion;
   }
-
   const [majorVersion] = rawVersion.split('.');
   return parseInt(majorVersion, 10);
 };
-
 var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
@@ -34,12 +29,11 @@ var _default = (0, _utils2.createRule)({
     fixable: 'code'
   },
   defaultOptions: [],
-
   create(context) {
     var _context$settings, _context$settings$jes;
-
     const jestVersion = parseJestVersion(((_context$settings = context.settings) === null || _context$settings === void 0 ? void 0 : (_context$settings$jes = _context$settings.jest) === null || _context$settings$jes === void 0 ? void 0 : _context$settings$jes.version) || (0, _utils2.detectJestVersion)());
-    const deprecations = { ...(jestVersion >= 15 && {
+    const deprecations = {
+      ...(jestVersion >= 15 && {
         'jest.resetModuleRegistry': 'jest.resetModules'
       }),
       ...(jestVersion >= 17 && {
@@ -61,13 +55,10 @@ var _default = (0, _utils2.createRule)({
         if (node.callee.type !== _utils.AST_NODE_TYPES.MemberExpression) {
           return;
         }
-
         const deprecation = (0, _utils2.getNodeName)(node);
-
         if (!deprecation || !(deprecation in deprecations)) {
           return;
         }
-
         const replacement = deprecations[deprecation];
         const {
           callee
@@ -79,23 +70,16 @@ var _default = (0, _utils2.createRule)({
             replacement
           },
           node,
-
           fix(fixer) {
             let [name, func] = replacement.split('.');
-
             if (callee.property.type === _utils.AST_NODE_TYPES.Literal) {
               func = `'${func}'`;
             }
-
             return [fixer.replaceText(callee.object, name), fixer.replaceText(callee.property, func)];
           }
-
         });
       }
-
     };
   }
-
 });
-
 exports.default = _default;

package/lib/rules/no-disabled-tests.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: {
@@ -27,25 +25,20 @@ var _default = (0, _utils.createRule)({
     type: 'suggestion'
   },
   defaultOptions: [],
-
   create(context) {
     let suiteDepth = 0;
     let testDepth = 0;
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils.parseJestFnCall)(node, context);
-
         if (!jestFnCall) {
           return;
         }
-
         if (jestFnCall.type === 'describe') {
           suiteDepth++;
         }
-
         if (jestFnCall.type === 'test') {
           testDepth++;
-
           if (node.arguments.length < 2 && jestFnCall.members.every(s => (0, _utils.getAccessorValue)(s) !== 'todo')) {
             context.report({
               messageId: 'missingFunction',
@@ -53,8 +46,8 @@ var _default = (0, _utils.createRule)({
             });
           }
         }
-
-        if ( // the only jest functions that are with "x" are "xdescribe", "xtest", and "xit"
+        if (
+        // the only jest functions that are with "x" are "xdescribe", "xtest", and "xit"
         jestFnCall.name.startsWith('x') || jestFnCall.members.some(s => (0, _utils.getAccessorValue)(s) === 'skip')) {
           context.report({
             messageId: jestFnCall.type === 'describe' ? 'disabledSuite' : 'disabledTest',
@@ -62,28 +55,22 @@ var _default = (0, _utils.createRule)({
           });
         }
       },
-
       'CallExpression:exit'(node) {
         const jestFnCall = (0, _utils.parseJestFnCall)(node, context);
-
         if (!jestFnCall) {
           return;
         }
-
         if (jestFnCall.type === 'describe') {
           suiteDepth--;
         }
-
         if (jestFnCall.type === 'test') {
           testDepth--;
         }
       },
-
       'CallExpression[callee.name="pending"]'(node) {
         if ((0, _utils.scopeHasLocalReference)(context.getScope(), 'pending')) {
           return;
         }
-
         if (testDepth > 0) {
           context.report({
             messageId: 'pendingTest',
@@ -101,10 +88,7 @@ var _default = (0, _utils.createRule)({
           });
         }
       }
-
     };
   }
-
 });
-
 exports.default = _default;