eslint-plugin-jest 28.0.0-next.1 → 28.0.0-next.3

package/lib/index.js

@@ -8,8 +8,6 @@ var snapshotProcessor = _interopRequireWildcard(require("./processors/snapshot-p
 function _getRequireWildcardCache(e) { if ("function" != typeof WeakMap) return null; var r = new WeakMap(), t = new WeakMap(); return (_getRequireWildcardCache = function (e) { return e ? t : r; })(e); }
 function _interopRequireWildcard(e, r) { if (!r && e && e.__esModule) return e; if (null === e || "object" != typeof e && "function" != typeof e) return { default: e }; var t = _getRequireWildcardCache(r); if (t && t.has(e)) return t.get(e); var n = { __proto__: null }, a = Object.defineProperty && Object.getOwnPropertyDescriptor; for (var u in e) if ("default" !== u && Object.prototype.hasOwnProperty.call(e, u)) { var i = a ? Object.getOwnPropertyDescriptor(e, u) : null; i && (i.get || i.set) ? Object.defineProperty(n, u, i) : n[u] = e[u]; } return n.default = e, t && t.set(e, n), n; }
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
-// v5 of `@typescript-eslint/experimental-utils` removed this
-
 // copied from https://github.com/babel/babel/blob/d8da63c929f2d28c401571e2a43166678c555bc4/packages/babel-helpers/src/helpers.js#L602-L606
 /* istanbul ignore next */
 const interopRequireDefault = obj => obj && obj.__esModule ? obj : {

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

@@ -11,9 +11,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Enforce `test` and `it` usage conventions',
-      recommended: false
+      description: 'Enforce `test` and `it` usage conventions'
     },
     fixable: 'code',
     messages: {
@@ -24,9 +22,11 @@ var _default = exports.default = (0, _utils2.createRule)({
       type: 'object',
       properties: {
         fn: {
+          type: 'string',
           enum: [_utils2.TestCaseName.it, _utils2.TestCaseName.test]
         },
         withinDescribe: {
+          type: 'string',
           enum: [_utils2.TestCaseName.it, _utils2.TestCaseName.test]
         }
       },

package/lib/rules/expect-expect.js

@@ -30,9 +30,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Enforce assertion to be made in a test body',
-      recommended: 'warn'
+      description: 'Enforce assertion to be made in a test body'
     },
     messages: {
       noAssertions: 'Test has no assertions'

package/lib/rules/max-expects.js

@@ -10,9 +10,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Enforces a maximum number assertion calls in a test body',
-      recommended: false
+      description: 'Enforces a maximum number assertion calls in a test body'
     },
     messages: {
       exceededMaxAssertion: 'Too many assertion calls ({{ count }}) - maximum allowed is {{ max }}'
@@ -37,8 +35,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   }]) {
     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']);
+      const isTestFn = node.parent?.type !== _utils.AST_NODE_TYPES.CallExpression || (0, _utils2.isTypeOfJestFnCall)(node.parent, context, ['test']);
       if (isTestFn) {
         count = 0;
       }
@@ -49,9 +46,8 @@ var _default = exports.default = (0, _utils2.createRule)({
       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) {
+        if (jestFnCall?.type !== 'expect' || jestFnCall.head.node.parent?.type === _utils.AST_NODE_TYPES.MemberExpression) {
           return;
         }
         count += 1;

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

@@ -9,9 +9,7 @@ var _default = exports.default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Enforces a maximum depth to nested describe calls',
-      recommended: false
+      description: 'Enforces a maximum depth to nested describe calls'
     },
     messages: {
       exceededMaxDepth: 'Too many nested describe calls ({{ depth }}) - maximum allowed is {{ max }}'

package/lib/rules/no-alias-methods.js

@@ -9,9 +9,7 @@ var _default = exports.default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow alias methods',
-      recommended: 'error'
+      description: 'Disallow alias methods'
     },
     messages: {
       replaceAlias: `Replace {{ alias }}() with its canonical name of {{ canonical }}()`
@@ -39,7 +37,7 @@ var _default = exports.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') {
+        if (jestFnCall?.type !== 'expect') {
           return;
         }
         const {

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

@@ -12,9 +12,7 @@ var _default = exports.default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow commented out tests',
-      recommended: 'warn'
+      description: 'Disallow commented out tests'
     },
     messages: {
       commentedTests: 'Some tests seem to be commented'

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

@@ -11,9 +11,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      description: 'Disallow calling `expect` conditionally',
-      category: 'Best Practices',
-      recommended: 'error'
+      description: 'Disallow calling `expect` conditionally'
     },
     messages: {
       conditionalExpect: 'Avoid calling `expect` conditionally`'

package/lib/rules/no-conditional-in-test.js

@@ -9,9 +9,7 @@ var _default = exports.default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
-      description: 'Disallow conditional logic in tests',
-      category: 'Best Practices',
-      recommended: false
+      description: 'Disallow conditional logic in tests'
     },
     messages: {
       conditionalInTest: 'Avoid having conditionals in tests'

package/lib/rules/no-confusing-set-timeout.js

@@ -12,9 +12,7 @@ var _default = exports.default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow confusing usages of jest.setTimeout',
-      recommended: false
+      description: 'Disallow confusing usages of jest.setTimeout'
     },
     messages: {
       globalSetTimeout: '`jest.setTimeout` should be call in `global` scope',

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

@@ -17,9 +17,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow use of deprecated functions',
-      recommended: 'error'
+      description: 'Disallow use of deprecated functions'
     },
     messages: {
       deprecatedFunction: '`{{ deprecation }}` has been deprecated in favor of `{{ replacement }}`'
@@ -30,8 +28,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   },
   defaultOptions: [],
   create(context) {
-    var _context$settings;
-    const jestVersion = parseJestVersion(((_context$settings = context.settings) === null || _context$settings === void 0 || (_context$settings = _context$settings.jest) === null || _context$settings === void 0 ? void 0 : _context$settings.version) || (0, _utils2.detectJestVersion)());
+    const jestVersion = parseJestVersion(context.settings?.jest?.version || (0, _utils2.detectJestVersion)());
     const deprecations = {
       ...(jestVersion >= 15 && {
         'jest.resetModuleRegistry': 'jest.resetModules'

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

@@ -9,9 +9,7 @@ var _default = exports.default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow disabled tests',
-      recommended: 'warn'
+      description: 'Disallow disabled tests'
     },
     messages: {
       missingFunction: 'Test is missing function argument',

package/lib/rules/no-done-callback.js

@@ -11,10 +11,10 @@ const findCallbackArg = (node, isJestEach, context) => {
     return node.arguments[1];
   }
   const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
-  if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) === 'hook' && node.arguments.length >= 1) {
+  if (jestFnCall?.type === 'hook' && node.arguments.length >= 1) {
     return node.arguments[0];
   }
-  if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) === 'test' && node.arguments.length >= 2) {
+  if (jestFnCall?.type === 'test' && node.arguments.length >= 2) {
     return node.arguments[1];
   }
   return null;
@@ -23,9 +23,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow using a callback in asynchronous tests and hooks',
-      recommended: 'error'
+      description: 'Disallow using a callback in asynchronous tests and hooks'
     },
     messages: {
       noDoneCallback: 'Return a Promise instead of relying on callback parameter',
@@ -40,9 +38,8 @@ var _default = exports.default = (0, _utils2.createRule)({
   create(context) {
     return {
       CallExpression(node) {
-        var _getNodeName;
         // done is the second argument for it.each, not the first
-        const isJestEach = ((_getNodeName = (0, _utils2.getNodeName)(node.callee)) === null || _getNodeName === void 0 ? void 0 : _getNodeName.endsWith('.each')) ?? false;
+        const isJestEach = (0, _utils2.getNodeName)(node.callee)?.endsWith('.each') ?? false;
         if (isJestEach && node.callee.type !== _utils.AST_NODE_TYPES.TaggedTemplateExpression) {
           // isJestEach but not a TaggedTemplateExpression, so this must be
           // the `jest.each([])()` syntax which this rule doesn't support due
@@ -78,7 +75,6 @@ var _default = exports.default = (0, _utils2.createRule)({
               callback: argument.name
             },
             fix(fixer) {
-              var _tokenAfterLastParam;
               const {
                 body,
                 params
@@ -90,7 +86,7 @@ var _default = exports.default = (0, _utils2.createRule)({
               const lastParam = params[params.length - 1];
               const tokenBeforeFirstParam = sourceCode.getTokenBefore(firstParam);
               let tokenAfterLastParam = sourceCode.getTokenAfter(lastParam);
-              if (((_tokenAfterLastParam = tokenAfterLastParam) === null || _tokenAfterLastParam === void 0 ? void 0 : _tokenAfterLastParam.value) === ',') {
+              if (tokenAfterLastParam?.value === ',') {
                 tokenAfterLastParam = sourceCode.getTokenAfter(tokenAfterLastParam);
               }
 

package/lib/rules/no-duplicate-hooks.js

@@ -9,9 +9,7 @@ var _default = exports.default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow duplicate setup and teardown hooks',
-      recommended: false
+      description: 'Disallow duplicate setup and teardown hooks'
     },
     messages: {
       noDuplicateHook: 'Duplicate {{hook}} in describe block'
@@ -24,16 +22,15 @@ var _default = exports.default = (0, _utils.createRule)({
     const hookContexts = [{}];
     return {
       CallExpression(node) {
-        var _jestFnCall$name;
         const jestFnCall = (0, _utils.parseJestFnCall)(node, context);
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) === 'describe') {
+        if (jestFnCall?.type === 'describe') {
           hookContexts.push({});
         }
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'hook') {
+        if (jestFnCall?.type !== 'hook') {
           return;
         }
         const currentLayer = hookContexts[hookContexts.length - 1];
-        currentLayer[_jestFnCall$name = jestFnCall.name] || (currentLayer[_jestFnCall$name] = 0);
+        currentLayer[jestFnCall.name] ||= 0;
         currentLayer[jestFnCall.name] += 1;
         if (currentLayer[jestFnCall.name] > 1) {
           context.report({

package/lib/rules/no-export.js

@@ -10,9 +10,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow using `exports` in files containing tests',
-      recommended: 'error'
+      description: 'Disallow using `exports` in files containing tests'
     },
     messages: {
       unexpectedExport: `Do not export from a test file`

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

@@ -10,9 +10,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow focused tests',
-      recommended: 'error'
+      description: 'Disallow focused tests'
     },
     messages: {
       focusedTest: 'Unexpected focused test',
@@ -27,7 +25,7 @@ var _default = exports.default = (0, _utils2.createRule)({
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'test' && (jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'describe') {
+        if (jestFnCall?.type !== 'test' && jestFnCall?.type !== 'describe') {
           return;
         }
         if (jestFnCall.name.startsWith('f')) {

package/lib/rules/no-hooks.js

@@ -9,9 +9,7 @@ var _default = exports.default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow setup and teardown hooks',
-      recommended: false
+      description: 'Disallow setup and teardown hooks'
     },
     messages: {
       unexpectedHook: "Unexpected '{{ hookName }}' hook"
@@ -21,6 +19,7 @@ var _default = exports.default = (0, _utils.createRule)({
       properties: {
         allow: {
           type: 'array',
+          // @ts-expect-error https://github.com/eslint/eslint/discussions/17573
           contains: ['beforeAll', 'beforeEach', 'afterAll', 'afterEach']
         }
       },
@@ -37,7 +36,7 @@ var _default = exports.default = (0, _utils.createRule)({
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils.parseJestFnCall)(node, context);
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) === 'hook' && !allow.includes(jestFnCall.name)) {
+        if (jestFnCall?.type === 'hook' && !allow.includes(jestFnCall.name)) {
           context.report({
             node,
             messageId: 'unexpectedHook',

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

@@ -13,9 +13,7 @@ var _default = exports.default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow identical titles',
-      recommended: 'error'
+      description: 'Disallow identical titles'
     },
     messages: {
       multipleTestTitle: 'Test title is used multiple times in the same describe block',

package/lib/rules/no-interpolation-in-snapshots.js

@@ -10,9 +10,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow string interpolation inside snapshots',
-      recommended: 'error'
+      description: 'Disallow string interpolation inside snapshots'
     },
     messages: {
       noInterpolation: 'Do not use string interpolation inside of snapshots'
@@ -25,7 +23,7 @@ var _default = exports.default = (0, _utils2.createRule)({
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'expect') {
+        if (jestFnCall?.type !== 'expect') {
           return;
         }
         if (['toMatchInlineSnapshot', 'toThrowErrorMatchingInlineSnapshot'].includes((0, _utils2.getAccessorValue)(jestFnCall.matcher))) {

package/lib/rules/no-jasmine-globals.js

@@ -10,9 +10,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow Jasmine globals',
-      recommended: 'error'
+      description: 'Disallow Jasmine globals'
     },
     messages: {
       illegalGlobal: 'Illegal usage of global `{{ global }}`, prefer `{{ replacement }}`',

package/lib/rules/no-large-snapshots.js

@@ -47,9 +47,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow large snapshots',
-      recommended: false
+      description: 'Disallow large snapshots'
     },
     messages: {
       noSnapshot: '`{{ lineCount }}`s should begin with lowercase',
@@ -87,7 +85,7 @@ var _default = exports.default = (0, _utils2.createRule)({
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'expect') {
+        if (jestFnCall?.type !== 'expect') {
           return;
         }
         if (['toMatchInlineSnapshot', 'toThrowErrorMatchingInlineSnapshot'].includes((0, _utils2.getAccessorValue)(jestFnCall.matcher)) && jestFnCall.args.length) {

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

@@ -14,9 +14,7 @@ var _default = exports.default = (0, _utils.createRule)({
   meta: {
     type: 'problem',
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow manually importing from `__mocks__`',
-      recommended: 'error'
+      description: 'Disallow manually importing from `__mocks__`'
     },
     messages: {
       noManualImport: `Mocks should not be manually imported from a ${mocksDirName} directory. Instead use \`jest.mock\` and import from the original module path`

package/lib/rules/no-restricted-jest-methods.js

@@ -13,9 +13,7 @@ var _default = exports.default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow specific `jest.` methods',
-      recommended: false
+      description: 'Disallow specific `jest.` methods'
     },
     type: 'suggestion',
     schema: [{
@@ -31,7 +29,7 @@ var _default = exports.default = (0, _utils.createRule)({
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils.parseJestFnCall)(node, context);
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'jest' || jestFnCall.members.length === 0) {
+        if (jestFnCall?.type !== 'jest' || jestFnCall.members.length === 0) {
           return;
         }
         const method = (0, _utils.getAccessorValue)(jestFnCall.members[0]);

package/lib/rules/no-restricted-matchers.js

@@ -15,9 +15,7 @@ var _default = exports.default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow specific matchers & modifiers',
-      recommended: false
+      description: 'Disallow specific matchers & modifiers'
     },
     type: 'suggestion',
     schema: [{
@@ -36,7 +34,7 @@ var _default = exports.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') {
+        if (jestFnCall?.type !== 'expect') {
           return;
         }
         const chain = jestFnCall.members.map(nod => (0, _utils.getAccessorValue)(nod)).join('.');

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

@@ -37,15 +37,14 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow using `expect` outside of `it` or `test` blocks',
-      recommended: 'error'
+      description: 'Disallow using `expect` outside of `it` or `test` blocks'
     },
     messages: {
       unexpectedExpect: 'Expect must be inside of a test block'
     },
     type: 'suggestion',
     schema: [{
+      type: 'object',
       properties: {
         additionalTestBlockFunctions: {
           type: 'array',
@@ -68,9 +67,8 @@ var _default = exports.default = (0, _utils2.createRule)({
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) === 'expect') {
-          var _jestFnCall$head$node;
-          if (((_jestFnCall$head$node = jestFnCall.head.node.parent) === null || _jestFnCall$head$node === void 0 ? void 0 : _jestFnCall$head$node.type) === _utils.AST_NODE_TYPES.MemberExpression && jestFnCall.members.length === 1 && !['assertions', 'hasAssertions'].includes((0, _utils2.getAccessorValue)(jestFnCall.members[0]))) {
+        if (jestFnCall?.type === 'expect') {
+          if (jestFnCall.head.node.parent?.type === _utils.AST_NODE_TYPES.MemberExpression && jestFnCall.members.length === 1 && !['assertions', 'hasAssertions'].includes((0, _utils2.getAccessorValue)(jestFnCall.members[0]))) {
             return;
           }
           const parent = callStack[callStack.length - 1];
@@ -82,7 +80,7 @@ var _default = exports.default = (0, _utils2.createRule)({
           }
           return;
         }
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) === 'test' || isCustomTestBlockFunction(node)) {
+        if (jestFnCall?.type === 'test' || isCustomTestBlockFunction(node)) {
           callStack.push('test');
         }
         if (node.callee.type === _utils.AST_NODE_TYPES.TaggedTemplateExpression) {
@@ -107,8 +105,7 @@ var _default = exports.default = (0, _utils2.createRule)({
         }
       },
       ArrowFunctionExpression(node) {
-        var _node$parent;
-        if (((_node$parent = node.parent) === null || _node$parent === void 0 ? void 0 : _node$parent.type) !== _utils.AST_NODE_TYPES.CallExpression) {
+        if (node.parent?.type !== _utils.AST_NODE_TYPES.CallExpression) {
           callStack.push('arrow');
         }
       },

package/lib/rules/no-test-prefixes.js

@@ -10,9 +10,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Require using `.only` and `.skip` over `f` and `x`',
-      recommended: 'error'
+      description: 'Require using `.only` and `.skip` over `f` and `x`'
     },
     messages: {
       usePreferredName: 'Use "{{ preferredNodeName }}" instead'
@@ -26,7 +24,7 @@ var _default = exports.default = (0, _utils2.createRule)({
     return {
       CallExpression(node) {
         const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'describe' && (jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'test') {
+        if (jestFnCall?.type !== 'describe' && jestFnCall?.type !== 'test') {
           return;
         }
         if (jestFnCall.name[0] !== 'f' && jestFnCall.name[0] !== 'x') {

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

@@ -17,9 +17,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow explicitly returning from tests',
-      recommended: false
+      description: 'Disallow explicitly returning from tests'
     },
     messages: {
       noReturnValue: 'Jest tests should not return a value'

package/lib/rules/no-untyped-mock-factory.js

@@ -16,9 +16,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Disallow using `jest.mock()` factories without an explicit type parameter',
-      recommended: false
+      description: 'Disallow using `jest.mock()` factories without an explicit type parameter'
     },
     messages: {
       addTypeParameterToModuleMock: 'Add a type parameter to the mock factory such as `typeof import({{ moduleName }})`'
@@ -31,10 +29,15 @@ var _default = exports.default = (0, _utils2.createRule)({
   create(context) {
     return {
       CallExpression(node) {
-        const {
+        let {
           callee,
-          typeParameters
+          typeArguments
         } = node;
+
+        /* istanbul ignore next */
+        if (!('typeArguments' in node)) {
+          typeArguments = node.typeParameters;
+        }
         if (callee.type !== _utils.AST_NODE_TYPES.MemberExpression) {
           return;
         }
@@ -43,7 +46,7 @@ var _default = exports.default = (0, _utils2.createRule)({
         } = callee;
         if (node.arguments.length === 2 && (0, _utils2.isTypeOfJestFnCall)(node, context, ['jest']) && (0, _utils2.isSupportedAccessor)(property) && ['mock', 'doMock'].includes((0, _utils2.getAccessorValue)(property))) {
           const [nameNode, factoryNode] = node.arguments;
-          const hasTypeParameter = typeParameters !== undefined && typeParameters.params.length > 0;
+          const hasTypeParameter = typeArguments !== undefined && typeArguments.params.length > 0;
           const hasReturnType = (0, _utils2.isFunction)(factoryNode) && factoryNode.returnType !== undefined;
           if (hasTypeParameter || hasReturnType) {
             return;
@@ -52,7 +55,7 @@ var _default = exports.default = (0, _utils2.createRule)({
           context.report({
             messageId: 'addTypeParameterToModuleMock',
             data: {
-              moduleName: (moduleName === null || moduleName === void 0 ? void 0 : moduleName.raw) ?? './module-name'
+              moduleName: moduleName?.raw ?? './module-name'
             },
             node,
             fix(fixer) {

package/lib/rules/prefer-called-with.js

@@ -9,9 +9,7 @@ var _default = exports.default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
-      category: 'Best Practices',
-      description: 'Suggest using `toBeCalledWith()` or `toHaveBeenCalledWith()`',
-      recommended: false
+      description: 'Suggest using `toBeCalledWith()` or `toHaveBeenCalledWith()`'
     },
     messages: {
       preferCalledWith: 'Prefer {{ matcherName }}With(/* expected args */)'
@@ -24,7 +22,7 @@ var _default = exports.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') {
+        if (jestFnCall?.type !== 'expect') {
           return;
         }
         if (jestFnCall.modifiers.some(nod => (0, _utils.getAccessorValue)(nod) === 'not')) {