eslint-plugin-jest 27.4.0 → 27.4.2

package/lib/rules/max-expects.js

@@ -15,7 +15,7 @@ var _default = (0, _utils2.createRule)({
       recommended: false
     },
     messages: {
-      exceededMaxAssertion: 'Too many assertion calls ({{ count }}). Maximum allowed is {{ max }}.'
+      exceededMaxAssertion: 'Too many assertion calls ({{ count }}) - maximum allowed is {{ max }}'
     },
     type: 'suggestion',
     schema: [{

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

@@ -4,9 +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)({
+var _utils = require("./utils");
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -15,7 +14,7 @@ var _default = (0, _utils2.createRule)({
       recommended: false
     },
     messages: {
-      exceededMaxDepth: 'Too many nested describe calls ({{ depth }}). Maximum allowed is {{ max }}.'
+      exceededMaxDepth: 'Too many nested describe calls ({{ depth }}) - maximum allowed is {{ max }}'
     },
     type: 'suggestion',
     schema: [{
@@ -35,39 +34,28 @@ var _default = (0, _utils2.createRule)({
   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,
-          messageId: 'exceededMaxDepth',
-          data: {
-            depth: describeCallbackStack.length,
-            max
+    const describes = [];
+    return {
+      CallExpression(node) {
+        if ((0, _utils.isTypeOfJestFnCall)(node, context, ['describe'])) {
+          describes.unshift(node);
+          if (describes.length > max) {
+            context.report({
+              node,
+              messageId: 'exceededMaxDepth',
+              data: {
+                depth: describes.length,
+                max
+              }
+            });
           }
-        });
-      }
-    }
-    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();
+        }
+      },
+      'CallExpression:exit'(node) {
+        if (describes[0] === node) {
+          describes.shift();
+        }
       }
-    }
-    return {
-      FunctionExpression: pushDescribeCallback,
-      'FunctionExpression:exit': popDescribeCallback,
-      ArrowFunctionExpression: pushDescribeCallback,
-      'ArrowFunctionExpression:exit': popDescribeCallback
     };
   }
 });

package/lib/rules/no-export.js

@@ -15,7 +15,7 @@ var _default = (0, _utils2.createRule)({
       recommended: 'error'
     },
     messages: {
-      unexpectedExport: `Do not export from a test file.`
+      unexpectedExport: `Do not export from a test file`
     },
     type: 'suggestion',
     schema: []

package/lib/rules/no-focused-tests.js

@@ -15,8 +15,8 @@ var _default = (0, _utils2.createRule)({
       recommended: 'error'
     },
     messages: {
-      focusedTest: 'Unexpected focused test.',
-      suggestRemoveFocus: 'Remove focus from test.'
+      focusedTest: 'Unexpected focused test',
+      suggestRemoveFocus: 'Remove focus from test'
     },
     schema: [],
     type: 'suggestion',
@@ -33,7 +33,7 @@ var _default = (0, _utils2.createRule)({
         if (jestFnCall.name.startsWith('f')) {
           context.report({
             messageId: 'focusedTest',
-            node,
+            node: jestFnCall.head.node,
             suggest: [{
               messageId: 'suggestRemoveFocus',
               fix(fixer) {

package/lib/rules/no-identical-title.js

@@ -18,8 +18,8 @@ var _default = (0, _utils.createRule)({
       recommended: 'error'
     },
     messages: {
-      multipleTestTitle: 'Test title is used multiple times in the same describe block.',
-      multipleDescribeTitle: 'Describe block title is used multiple times in the same describe block.'
+      multipleTestTitle: 'Test title is used multiple times in the same describe block',
+      multipleDescribeTitle: 'Describe block title is used multiple times in the same describe block'
     },
     schema: [],
     type: 'suggestion'

package/lib/rules/no-if.js

@@ -22,7 +22,7 @@ var _default = (0, _utils2.createRule)({
       recommended: false
     },
     messages: {
-      conditionalInTest: 'Test should not contain {{ condition }} statements.'
+      conditionalInTest: 'Test should not contain {{ condition }} statements'
     },
     deprecated: true,
     replacedBy: ['no-conditional-in-test'],

package/lib/rules/no-mocks-import.js

@@ -19,7 +19,7 @@ var _default = (0, _utils.createRule)({
       recommended: 'error'
     },
     messages: {
-      noManualImport: `Mocks should not be manually imported from a ${mocksDirName} directory. Instead use \`jest.mock\` and import from the original module path.`
+      noManualImport: `Mocks should not be manually imported from a ${mocksDirName} directory. Instead use \`jest.mock\` and import from the original module path`
     },
     schema: []
   },

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

@@ -42,7 +42,7 @@ var _default = (0, _utils2.createRule)({
       recommended: 'error'
     },
     messages: {
-      unexpectedExpect: 'Expect must be inside of a test block.'
+      unexpectedExpect: 'Expect must be inside of a test block'
     },
     type: 'suggestion',
     schema: [{

package/lib/rules/no-test-return-statement.js

@@ -22,7 +22,7 @@ var _default = (0, _utils2.createRule)({
       recommended: false
     },
     messages: {
-      noReturnValue: 'Jest tests should not return a value.'
+      noReturnValue: 'Jest tests should not return a value'
     },
     schema: [],
     type: 'suggestion'

package/lib/rules/prefer-expect-resolves.js

@@ -16,7 +16,7 @@ var _default = (0, _utils2.createRule)({
     },
     fixable: 'code',
     messages: {
-      expectResolves: 'Use `await expect(...).resolves instead.'
+      expectResolves: 'Use `await expect(...).resolves instead'
     },
     schema: [],
     type: 'suggestion'

package/lib/rules/prefer-spy-on.js

@@ -47,7 +47,7 @@ var _default = (0, _utils2.createRule)({
       recommended: false
     },
     messages: {
-      useJestSpyOn: 'Use jest.spyOn() instead.'
+      useJestSpyOn: 'Use jest.spyOn() instead'
     },
     fixable: 'code',
     schema: [],

package/lib/rules/valid-expect.js

@@ -63,13 +63,13 @@ var _default = (0, _utils2.createRule)({
       recommended: 'error'
     },
     messages: {
-      tooManyArgs: 'Expect takes at most {{ amount }} argument{{ s }}.',
-      notEnoughArgs: 'Expect requires at least {{ amount }} argument{{ s }}.',
-      modifierUnknown: 'Expect has an unknown modifier.',
-      matcherNotFound: 'Expect must have a corresponding matcher call.',
-      matcherNotCalled: 'Matchers must be called to assert.',
-      asyncMustBeAwaited: 'Async assertions must be awaited{{ orReturned }}.',
-      promisesWithAsyncAssertionsMustBeAwaited: 'Promises which return async assertions must be awaited{{ orReturned }}.'
+      tooManyArgs: 'Expect takes at most {{ amount }} argument{{ s }}',
+      notEnoughArgs: 'Expect requires at least {{ amount }} argument{{ s }}',
+      modifierUnknown: 'Expect has an unknown modifier',
+      matcherNotFound: 'Expect must have a corresponding matcher call',
+      matcherNotCalled: 'Matchers must be called to assert',
+      asyncMustBeAwaited: 'Async assertions must be awaited{{ orReturned }}',
+      promisesWithAsyncAssertionsMustBeAwaited: 'Promises which return async assertions must be awaited{{ orReturned }}'
     },
     type: 'suggestion',
     schema: [{

package/lib/rules/valid-title.js

@@ -58,7 +58,7 @@ var _default = (0, _utils2.createRule)({
       emptyTitle: '{{ jestFunctionName }} should not have an empty title',
       duplicatePrefix: 'should not have duplicate prefix',
       accidentalSpace: 'should not have leading or trailing spaces',
-      disallowedWord: '"{{ word }}" is not allowed in test titles.',
+      disallowedWord: '"{{ word }}" is not allowed in test titles',
       mustNotMatch: '{{ jestFunctionName }} should not match {{ pattern }}',
       mustMatch: '{{ jestFunctionName }} should match {{ pattern }}',
       mustNotMatchCustom: '{{ message }}',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-jest",
-  "version": "27.4.0",
+  "version": "27.4.2",
   "description": "ESLint rules for Jest",
   "keywords": [
     "eslint",
@@ -135,7 +135,7 @@
     "pinst": "^3.0.0",
     "prettier": "^3.0.0",
     "rimraf": "^5.0.0",
-    "semantic-release": "^21.0.0",
+    "semantic-release": "^22.0.0",
     "semver": "^7.3.5",
     "ts-node": "^10.2.1",
     "typescript": "^5.0.4"