eslint-plugin-jest 27.9.0 → 28.0.0-next.2

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

@@ -9,14 +9,13 @@ var _utils2 = require("./utils");
 const isFirstStatement = node => {
   let parent = node;
   while (parent) {
-    var _parent$parent, _parent$parent2;
-    if (((_parent$parent = parent.parent) === null || _parent$parent === void 0 ? void 0 : _parent$parent.type) === _utils.AST_NODE_TYPES.BlockStatement) {
+    if (parent.parent?.type === _utils.AST_NODE_TYPES.BlockStatement) {
       return parent.parent.body[0] === parent;
     }
 
     // if we've hit an arrow function, then it must have a single expression
     // as its body, as otherwise we would have hit the block statement already
-    if (((_parent$parent2 = parent.parent) === null || _parent$parent2 === void 0 ? void 0 : _parent$parent2.type) === _utils.AST_NODE_TYPES.ArrowFunctionExpression) {
+    if (parent.parent?.type === _utils.AST_NODE_TYPES.ArrowFunctionExpression) {
       return true;
     }
     parent = parent.parent;
@@ -150,13 +149,12 @@ var _default = exports.default = (0, _utils2.createRule)({
       'ForOfStatement:exit': exitForLoop,
       CallExpression(node) {
         const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) === 'test') {
+        if (jestFnCall?.type === 'test') {
           inTestCaseCall = true;
           return;
         }
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) === 'expect' && inTestCaseCall) {
-          var _jestFnCall$head$node;
-          if (expressionDepth === 1 && isFirstStatement(node) && ((_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' && inTestCaseCall) {
+          if (expressionDepth === 1 && isFirstStatement(node) && jestFnCall.head.node.parent?.type === _utils.AST_NODE_TYPES.MemberExpression && jestFnCall.members.length === 1 && ['assertions', 'hasAssertions'].includes((0, _utils2.getAccessorValue)(jestFnCall.members[0]))) {
             checkExpectHasAssertions(jestFnCall, node);
             hasExpectAssertionsAsFirstStatement = true;
           }

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

@@ -25,17 +25,17 @@ var _default = exports.default = (0, _utils2.createRule)({
   create: context => ({
     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;
       }
       const {
         parent
       } = jestFnCall.head.node;
-      if ((parent === null || parent === void 0 ? void 0 : parent.type) !== _utils.AST_NODE_TYPES.CallExpression) {
+      if (parent?.type !== _utils.AST_NODE_TYPES.CallExpression) {
         return;
       }
       const [awaitNode] = parent.arguments;
-      if ((awaitNode === null || awaitNode === void 0 ? void 0 : awaitNode.type) === _utils.AST_NODE_TYPES.AwaitExpression) {
+      if (awaitNode?.type === _utils.AST_NODE_TYPES.AwaitExpression) {
         context.report({
           node: awaitNode,
           messageId: 'expectResolves',

package/lib/rules/prefer-hooks-in-order.js

@@ -31,7 +31,7 @@ var _default = exports.default = (0, _utils.createRule)({
           return;
         }
         const jestFnCall = (0, _utils.parseJestFnCall)(node, context);
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'hook') {
+        if (jestFnCall?.type !== 'hook') {
           // Reset the previousHookIndex when encountering something different from a hook
           previousHookIndex = -1;
           return;

package/lib/rules/prefer-mock-promise-shorthand.js

@@ -10,11 +10,10 @@ const withOnce = (name, addOnce) => {
   return `${name}${addOnce ? 'Once' : ''}`;
 };
 const findSingleReturnArgumentNode = fnNode => {
-  var _fnNode$body$body$;
   if (fnNode.body.type !== _utils.AST_NODE_TYPES.BlockStatement) {
     return fnNode.body;
   }
-  if (((_fnNode$body$body$ = fnNode.body.body[0]) === null || _fnNode$body$body$ === void 0 ? void 0 : _fnNode$body$body$.type) === _utils.AST_NODE_TYPES.ReturnStatement) {
+  if (fnNode.body.body[0]?.type === _utils.AST_NODE_TYPES.ReturnStatement) {
     return fnNode.body.body[0].argument;
   }
   return null;
@@ -37,7 +36,7 @@ var _default = exports.default = (0, _utils2.createRule)({
   defaultOptions: [],
   create(context) {
     const report = (property, isOnce, outerArgNode, innerArgNode = outerArgNode) => {
-      if ((innerArgNode === null || innerArgNode === void 0 ? void 0 : innerArgNode.type) !== _utils.AST_NODE_TYPES.CallExpression) {
+      if (innerArgNode?.type !== _utils.AST_NODE_TYPES.CallExpression) {
         return;
       }
       const argName = (0, _utils2.getNodeName)(innerArgNode);

package/lib/rules/prefer-snapshot-hint.js

@@ -95,8 +95,8 @@ var _default = exports.default = (0, _utils.createRule)({
       },
       CallExpression(node) {
         const jestFnCall = (0, _utils.parseJestFnCall)(node, context);
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'expect') {
-          if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) === 'describe' || (jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) === 'test') {
+        if (jestFnCall?.type !== 'expect') {
+          if (jestFnCall?.type === 'describe' || jestFnCall?.type === 'test') {
             depths.push(expressionDepth);
             expressionDepth = 0;
           }

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

@@ -29,8 +29,7 @@ const getJestFnCall = node => {
   return getJestFnCall(obj);
 };
 const getAutoFixMockImplementation = (jestFnCall, context) => {
-  var _jestFnCall$parent;
-  const hasMockImplementationAlready = ((_jestFnCall$parent = jestFnCall.parent) === null || _jestFnCall$parent === void 0 ? void 0 : _jestFnCall$parent.type) === _utils.AST_NODE_TYPES.MemberExpression && jestFnCall.parent.property.type === _utils.AST_NODE_TYPES.Identifier && jestFnCall.parent.property.name === 'mockImplementation';
+  const hasMockImplementationAlready = jestFnCall.parent?.type === _utils.AST_NODE_TYPES.MemberExpression && jestFnCall.parent.property.type === _utils.AST_NODE_TYPES.Identifier && jestFnCall.parent.property.name === 'mockImplementation';
   if (hasMockImplementationAlready) {
     return '';
   }

package/lib/rules/prefer-strict-equal.js

@@ -26,7 +26,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/prefer-to-be.js

@@ -30,9 +30,8 @@ const reportPreferToBe = (context, whatToBe, expectFnCall, func, modifierNode) =
   context.report({
     messageId: `useToBe${whatToBe}`,
     fix(fixer) {
-      var _expectFnCall$args;
       const fixes = [(0, _utils2.replaceAccessorFixer)(fixer, expectFnCall.matcher, `toBe${whatToBe}`)];
-      if ((_expectFnCall$args = expectFnCall.args) !== null && _expectFnCall$args !== void 0 && _expectFnCall$args.length && whatToBe !== '') {
+      if (expectFnCall.args?.length && whatToBe !== '') {
         fixes.push((0, _utils2.removeExtraArgumentsFixer)(fixer, context, func, 0));
       }
       if (modifierNode) {
@@ -67,7 +66,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;
         }
         const matcherName = (0, _utils2.getAccessorValue)(jestFnCall.matcher);

package/lib/rules/prefer-to-contain.js

@@ -37,13 +37,13 @@ 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' || jestFnCall.args.length === 0) {
+        if (jestFnCall?.type !== 'expect' || jestFnCall.args.length === 0) {
           return;
         }
         const {
           parent: expect
         } = jestFnCall.head.node;
-        if ((expect === null || expect === void 0 ? void 0 : expect.type) !== _utils.AST_NODE_TYPES.CallExpression) {
+        if (expect?.type !== _utils.AST_NODE_TYPES.CallExpression) {
           return;
         }
         const {

package/lib/rules/prefer-to-have-length.js

@@ -26,20 +26,20 @@ 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;
         }
         const {
           parent: expect
         } = jestFnCall.head.node;
-        if ((expect === null || expect === void 0 ? void 0 : expect.type) !== _utils.AST_NODE_TYPES.CallExpression) {
+        if (expect?.type !== _utils.AST_NODE_TYPES.CallExpression) {
           return;
         }
         const [argument] = expect.arguments;
         const {
           matcher
         } = jestFnCall;
-        if (!_utils2.EqualityMatcher.hasOwnProperty((0, _utils2.getAccessorValue)(matcher)) || (argument === null || argument === void 0 ? void 0 : argument.type) !== _utils.AST_NODE_TYPES.MemberExpression || !(0, _utils2.isSupportedAccessor)(argument.property, 'length')) {
+        if (!_utils2.EqualityMatcher.hasOwnProperty((0, _utils2.getAccessorValue)(matcher)) || argument?.type !== _utils.AST_NODE_TYPES.MemberExpression || !(0, _utils2.isSupportedAccessor)(argument.property, 'length')) {
           return;
         }
         context.report({

package/lib/rules/prefer-todo.js

@@ -51,7 +51,7 @@ var _default = exports.default = (0, _utils2.createRule)({
       CallExpression(node) {
         const [title, callback] = node.arguments;
         const jestFnCall = (0, _utils2.parseJestFnCall)(node, context);
-        if (!title || (jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'test' || !isTargetedTestCase(jestFnCall) || !(0, _utils2.isStringNode)(title)) {
+        if (!title || jestFnCall?.type !== 'test' || !isTargetedTestCase(jestFnCall) || !(0, _utils2.isStringNode)(title)) {
           return;
         }
         if (callback && isEmptyFunction(callback)) {

package/lib/rules/require-hook.js

@@ -7,11 +7,10 @@ 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.'));
+  return !!(0, _utils2.getNodeName)(node)?.startsWith('jest.');
 };
 const isNullOrUndefined = node => {
   return node.type === _utils.AST_NODE_TYPES.Literal && node.value === null || (0, _utils2.isIdentifier)(node, 'undefined');

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

@@ -24,7 +24,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/unbound-method.js

@@ -34,27 +34,26 @@ var _default = exports.default = (0, _utils2.createRule)({
     },
     schema: [],
     type: 'problem',
-    ...(baseRule === null || baseRule === void 0 ? void 0 : baseRule.meta),
+    ...baseRule?.meta,
     docs: {
       category: 'Best Practices',
       description: 'Enforce unbound methods are called with their expected scope',
       requiresTypeChecking: true,
-      ...(baseRule === null || baseRule === void 0 ? void 0 : baseRule.meta.docs),
+      ...baseRule?.meta.docs,
       recommended: false
     }
   },
   create(context) {
-    const baseSelectors = baseRule === null || baseRule === void 0 ? void 0 : baseRule.create(context);
+    const baseSelectors = baseRule?.create(context);
     if (!baseSelectors) {
       return {};
     }
     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) {
+        if (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') {
+          if (jestFnCall?.type === 'expect') {
             const {
               matcher
             } = jestFnCall;
@@ -63,7 +62,7 @@ var _default = exports.default = (0, _utils2.createRule)({
             }
           }
         }
-        (_baseSelectors$Member = baseSelectors.MemberExpression) === null || _baseSelectors$Member === void 0 || _baseSelectors$Member.call(baseSelectors, node);
+        baseSelectors.MemberExpression?.(node);
       }
     };
   }

package/lib/rules/utils/misc.js

@@ -103,7 +103,7 @@ const getTestCallExpressionsFromDeclaredVariables = (declaredVariables, context)
     references
   }) => acc.concat(references.map(({
     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']))), []);
+  }) => identifier.parent).filter(node => node?.type === _utils.AST_NODE_TYPES.CallExpression && (0, _parseJestFnCall.isTypeOfJestFnCall)(node, context, ['test']))), []);
 };
 
 /**

package/lib/rules/utils/parseJestFnCall.js

@@ -51,8 +51,7 @@ const determineJestFnType = name => {
 };
 const ValidJestFnCallChains = ['afterAll', 'afterEach', 'beforeAll', 'beforeEach', 'describe', 'describe.each', 'describe.only', 'describe.only.each', 'describe.skip', 'describe.skip.each', 'fdescribe', 'fdescribe.each', 'xdescribe', 'xdescribe.each', 'it', 'it.concurrent', 'it.concurrent.failing', 'it.concurrent.each', 'it.concurrent.failing.each', 'it.concurrent.failing.only.each', 'it.concurrent.failing.skip.each', 'it.concurrent.only.each', 'it.concurrent.skip.each', 'it.each', 'it.failing', 'it.failing.each', 'it.only', 'it.only.each', 'it.only.failing', 'it.only.failing.each', 'it.skip', 'it.skip.each', 'it.skip.failing', 'it.skip.failing.each', 'it.todo', 'fit', 'fit.each', 'fit.failing', 'fit.failing.each', 'xit', 'xit.each', 'xit.failing', 'xit.failing.each', 'test', 'test.concurrent', 'test.concurrent.failing', 'test.concurrent.each', 'test.concurrent.failing.each', 'test.concurrent.failing.only.each', 'test.concurrent.failing.skip.each', 'test.concurrent.only.each', 'test.concurrent.skip.each', 'test.each', 'test.failing', 'test.failing.each', 'test.only', 'test.only.each', 'test.only.failing', 'test.only.failing.each', 'test.skip', 'test.skip.each', 'test.skip.failing', 'test.skip.failing.each', 'test.todo', 'xtest', 'xtest.each', 'xtest.failing', 'xtest.failing.each'];
 const resolvePossibleAliasedGlobal = (global, context) => {
-  var _context$settings$jes;
-  const globalAliases = ((_context$settings$jes = context.settings.jest) === null || _context$settings$jes === void 0 ? void 0 : _context$settings$jes.globalAliases) ?? {};
+  const globalAliases = context.settings.jest?.globalAliases ?? {};
   const alias = Object.entries(globalAliases).find(([, aliases]) => aliases.includes(global));
   if (alias) {
     return alias[0];
@@ -79,9 +78,8 @@ const parseJestFnCallWithReason = (node, context) => {
 };
 exports.parseJestFnCallWithReason = parseJestFnCallWithReason;
 const parseJestFnCallWithReasonInner = (node, context) => {
-  var _node$parent2, _node$parent3;
   const chain = getNodeChain(node);
-  if (!(chain !== null && chain !== void 0 && chain.length)) {
+  if (!chain?.length) {
     return null;
   }
   const [first, ...rest] = chain;
@@ -127,8 +125,7 @@ const parseJestFnCallWithReasonInner = (node, context) => {
       return null;
     }
     if (result === 'matcher-not-found') {
-      var _node$parent;
-      if (((_node$parent = node.parent) === null || _node$parent === void 0 ? void 0 : _node$parent.type) === _utils.AST_NODE_TYPES.MemberExpression) {
+      if (node.parent?.type === _utils.AST_NODE_TYPES.MemberExpression) {
         return 'matcher-not-called';
       }
     }
@@ -136,17 +133,14 @@ const parseJestFnCallWithReasonInner = (node, context) => {
   }
 
   // check that every link in the chain except the last is a member expression
-  if (chain.slice(0, chain.length - 1).some(nod => {
-    var _nod$parent;
-    return ((_nod$parent = nod.parent) === null || _nod$parent === void 0 ? void 0 : _nod$parent.type) !== _utils.AST_NODE_TYPES.MemberExpression;
-  })) {
+  if (chain.slice(0, chain.length - 1).some(nod => nod.parent?.type !== _utils.AST_NODE_TYPES.MemberExpression)) {
     return null;
   }
 
   // ensure that we're at the "top" of the function call chain otherwise when
   // parsing e.g. x().y.z(), we'll incorrectly find & parse "x()" even though
   // the full chain is not a valid jest function call chain
-  if (((_node$parent2 = node.parent) === null || _node$parent2 === void 0 ? void 0 : _node$parent2.type) === _utils.AST_NODE_TYPES.CallExpression || ((_node$parent3 = node.parent) === null || _node$parent3 === void 0 ? void 0 : _node$parent3.type) === _utils.AST_NODE_TYPES.MemberExpression) {
+  if (node.parent?.type === _utils.AST_NODE_TYPES.CallExpression || node.parent?.type === _utils.AST_NODE_TYPES.MemberExpression) {
     return null;
   }
   return {
@@ -157,10 +151,9 @@ const parseJestFnCallWithReasonInner = (node, context) => {
 const findModifiersAndMatcher = members => {
   const modifiers = [];
   for (const member of members) {
-    var _member$parent, _member$parent$parent;
     // check if the member is being called, which means it is the matcher
     // (and also the end of the entire "expect" call chain)
-    if (((_member$parent = member.parent) === null || _member$parent === void 0 ? void 0 : _member$parent.type) === _utils.AST_NODE_TYPES.MemberExpression && ((_member$parent$parent = member.parent.parent) === null || _member$parent$parent === void 0 ? void 0 : _member$parent$parent.type) === _utils.AST_NODE_TYPES.CallExpression) {
+    if (member.parent?.type === _utils.AST_NODE_TYPES.MemberExpression && member.parent.parent?.type === _utils.AST_NODE_TYPES.CallExpression) {
       return {
         matcher: member,
         args: member.parent.parent.arguments,
@@ -245,7 +238,6 @@ const findImportSourceNode = node => {
   return null;
 };
 const describeVariableDefAsImport = def => {
-  var _def$name$parent;
   // make sure that we've actually being assigned a value
   if (!def.node.init) {
     return null;
@@ -254,7 +246,7 @@ const describeVariableDefAsImport = def => {
   if (!sourceNode || !(0, _utils2.isStringNode)(sourceNode)) {
     return null;
   }
-  if (((_def$name$parent = def.name.parent) === null || _def$name$parent === void 0 ? void 0 : _def$name$parent.type) !== _utils.AST_NODE_TYPES.Property) {
+  if (def.name.parent?.type !== _utils.AST_NODE_TYPES.Property) {
     return null;
   }
   if (!(0, _utils2.isSupportedAccessor)(def.name.parent.key)) {
@@ -293,7 +285,7 @@ const resolveScope = (scope, identifier) => {
     if (ref && ref.defs.length > 0) {
       const def = ref.defs[ref.defs.length - 1];
       const importDetails = describePossibleImportDef(def);
-      if ((importDetails === null || importDetails === void 0 ? void 0 : importDetails.local) === identifier) {
+      if (importDetails?.local === identifier) {
         return importDetails;
       }
       return 'local';

package/lib/rules/valid-describe-callback.js

@@ -37,7 +37,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') {
+        if (jestFnCall?.type !== 'describe') {
           return;
         }
         if (node.arguments.length < 1) {

package/lib/rules/valid-expect-in-promise.js

@@ -24,7 +24,7 @@ const isPromiseChainCall = node => {
 };
 const isTestCaseCallWithCallbackArg = (node, context) => {
   const jestCallFn = (0, _utils2.parseJestFnCall)(node, context);
-  if ((jestCallFn === null || jestCallFn === void 0 ? void 0 : jestCallFn.type) !== 'test') {
+  if (jestCallFn?.type !== 'test') {
     return false;
   }
   const isJestEach = jestCallFn.members.some(s => (0, _utils2.getAccessorValue)(s) === 'each');
@@ -67,10 +67,10 @@ const isPromiseMethodThatUsesValue = (node, identifier) => {
  */
 const isValueAwaitedInElements = (name, elements) => {
   for (const element of elements) {
-    if ((element === null || element === void 0 ? void 0 : element.type) === _utils.AST_NODE_TYPES.AwaitExpression && (0, _utils2.isIdentifier)(element.argument, name)) {
+    if (element?.type === _utils.AST_NODE_TYPES.AwaitExpression && (0, _utils2.isIdentifier)(element.argument, name)) {
       return true;
     }
-    if ((element === null || element === void 0 ? void 0 : element.type) === _utils.AST_NODE_TYPES.ArrayExpression && isValueAwaitedInElements(name, element.elements)) {
+    if (element?.type === _utils.AST_NODE_TYPES.ArrayExpression && isValueAwaitedInElements(name, element.elements)) {
       return true;
     }
   }
@@ -138,7 +138,7 @@ const isValueAwaitedOrReturned = (identifier, body, context) => {
         }
         const leftMostCall = getLeftMostCallExpression(node.expression);
         const jestFnCall = (0, _utils2.parseJestFnCall)(node.expression, context);
-        if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) === 'expect' && leftMostCall.arguments.length > 0 && (0, _utils2.isIdentifier)(leftMostCall.arguments[0], name)) {
+        if (jestFnCall?.type === 'expect' && leftMostCall.arguments.length > 0 && (0, _utils2.isIdentifier)(leftMostCall.arguments[0], name)) {
           if (jestFnCall.members.some(m => {
             const v = (0, _utils2.getAccessorValue)(m);
             return v === _utils2.ModifierName.resolves || v === _utils2.ModifierName.rejects;
@@ -154,10 +154,9 @@ const isValueAwaitedOrReturned = (identifier, body, context) => {
       // (re)assignment changes the runtime value, so if we've not found an
       // await or return already we act as if we've reached the end of the body
       if (node.expression.type === _utils.AST_NODE_TYPES.AssignmentExpression) {
-        var _getNodeName;
         // unless we're assigning to the same identifier, in which case
         // we might be chaining off the existing promise value
-        if ((0, _utils2.isIdentifier)(node.expression.left, name) && (_getNodeName = (0, _utils2.getNodeName)(node.expression.right)) !== null && _getNodeName !== void 0 && _getNodeName.startsWith(`${name}.`) && isPromiseChainCall(node.expression.right)) {
+        if ((0, _utils2.isIdentifier)(node.expression.left, name) && (0, _utils2.getNodeName)(node.expression.right)?.startsWith(`${name}.`) && isPromiseChainCall(node.expression.right)) {
           continue;
         }
         break;
@@ -185,9 +184,8 @@ const isDirectlyWithinTestCaseCall = (node, context) => {
   let parent = node;
   while (parent) {
     if ((0, _utils2.isFunction)(parent)) {
-      var _parent;
       parent = parent.parent;
-      return ((_parent = parent) === null || _parent === void 0 ? void 0 : _parent.type) === _utils.AST_NODE_TYPES.CallExpression && (0, _utils2.isTypeOfJestFnCall)(parent, context, ['test']);
+      return parent?.type === _utils.AST_NODE_TYPES.CallExpression && (0, _utils2.isTypeOfJestFnCall)(parent, context, ['test']);
     }
     parent = parent.parent;
   }

package/lib/rules/valid-expect.js

@@ -27,13 +27,9 @@ const getPromiseCallExpressionNode = node => {
   }
   return null;
 };
-const findPromiseCallExpressionNode = node => {
-  var _node$parent;
-  return (_node$parent = node.parent) !== null && _node$parent !== void 0 && _node$parent.parent && [_utils.AST_NODE_TYPES.CallExpression, _utils.AST_NODE_TYPES.ArrayExpression].includes(node.parent.type) ? getPromiseCallExpressionNode(node.parent) : null;
-};
+const findPromiseCallExpressionNode = node => node.parent?.parent && [_utils.AST_NODE_TYPES.CallExpression, _utils.AST_NODE_TYPES.ArrayExpression].includes(node.parent.type) ? getPromiseCallExpressionNode(node.parent) : null;
 const getParentIfThenified = node => {
-  var _node$parent2;
-  const grandParentNode = (_node$parent2 = node.parent) === null || _node$parent2 === void 0 ? void 0 : _node$parent2.parent;
+  const grandParentNode = node.parent?.parent;
   if (grandParentNode && grandParentNode.type === _utils.AST_NODE_TYPES.CallExpression && grandParentNode.callee.type === _utils.AST_NODE_TYPES.MemberExpression && (0, _utils2.isSupportedAccessor)(grandParentNode.callee.property) && ['then', 'catch'].includes((0, _utils2.getAccessorValue)(grandParentNode.callee.property)) && grandParentNode.parent) {
     // Just in case `then`s are chained look one above.
     return getParentIfThenified(grandParentNode);
@@ -139,8 +135,7 @@ var _default = exports.default = (0, _utils2.createRule)({
       CallExpression(node) {
         const jestFnCall = (0, _utils2.parseJestFnCallWithReason)(node, context);
         if (typeof jestFnCall === 'string') {
-          var _node$parent3;
-          const reportingNode = ((_node$parent3 = node.parent) === null || _node$parent3 === void 0 ? void 0 : _node$parent3.type) === _utils.AST_NODE_TYPES.MemberExpression ? findTopMostMemberExpression(node.parent).property : node;
+          const reportingNode = node.parent?.type === _utils.AST_NODE_TYPES.MemberExpression ? findTopMostMemberExpression(node.parent).property : node;
           if (jestFnCall === 'matcher-not-found') {
             context.report({
               messageId: 'matcherNotFound',
@@ -162,13 +157,13 @@ var _default = exports.default = (0, _utils2.createRule)({
             return;
           }
           return;
-        } else if ((jestFnCall === null || jestFnCall === void 0 ? void 0 : jestFnCall.type) !== 'expect') {
+        } else if (jestFnCall?.type !== 'expect') {
           return;
         }
         const {
           parent: expect
         } = jestFnCall.head.node;
-        if ((expect === null || expect === void 0 ? void 0 : expect.type) !== _utils.AST_NODE_TYPES.CallExpression) {
+        if (expect?.type !== _utils.AST_NODE_TYPES.CallExpression) {
           return;
         }
         if (expect.arguments.length < minArgs) {
@@ -222,7 +217,7 @@ var _default = exports.default = (0, _utils2.createRule)({
         } = jestFnCall;
         const parentNode = matcher.parent.parent;
         const shouldBeAwaited = jestFnCall.modifiers.some(nod => (0, _utils2.getAccessorValue)(nod) !== 'not') || asyncMatchers.includes((0, _utils2.getAccessorValue)(matcher));
-        if (!(parentNode !== null && parentNode !== void 0 && parentNode.parent) || !shouldBeAwaited) {
+        if (!parentNode?.parent || !shouldBeAwaited) {
           return;
         }
         /**

package/lib/rules/valid-title.js

@@ -128,7 +128,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;
         }
         const [argument] = node.arguments;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-jest",
-  "version": "27.9.0",
+  "version": "28.0.0-next.2",
   "description": "ESLint rules for Jest",
   "keywords": [
     "eslint",
@@ -21,7 +21,7 @@
   ],
   "scripts": {
     "build": "babel --extensions .js,.ts src --out-dir lib --copy-files && rimraf --glob lib/__tests__ 'lib/**/__tests__'",
-    "_postinstall": "is-ci || husky install",
+    "_postinstall": "is-ci || husky",
     "lint": "eslint . --ignore-pattern '!.eslintrc.js' --ext js,ts",
     "prepack": "rimraf lib && yarn build",
     "prepublishOnly": "pinst --disable",
@@ -126,7 +126,7 @@
     "eslint-plugin-prettier": "^5.0.0",
     "eslint-remote-tester": "^3.0.0",
     "eslint-remote-tester-repositories": "~1.0.0",
-    "husky": "^8.0.1",
+    "husky": "^9.0.1",
     "is-ci": "^3.0.0",
     "jest": "^29.0.0",
     "jest-runner-eslint": "^2.0.0",
@@ -135,7 +135,7 @@
     "pinst": "^3.0.0",
     "prettier": "^3.0.0",
     "rimraf": "^5.0.0",
-    "semantic-release": "^22.0.0",
+    "semantic-release": "^23.0.0",
     "semver": "^7.3.5",
     "strip-ansi": "^6.0.0",
     "ts-node": "^10.2.1",
@@ -154,14 +154,14 @@
       "optional": true
     }
   },
-  "packageManager": "yarn@3.8.0",
+  "packageManager": "yarn@3.8.1",
   "engines": {
-    "node": "^14.15.0 || ^16.10.0 || >=18.0.0"
+    "node": "^16.10.0 || >=18.0.0"
   },
   "publishConfig": {
     "provenance": true
   },
   "resolutions": {
-    "typescript": "~5.2.2"
+    "@typescript-eslint/typescript-estree@5.62.0": "patch:@typescript-eslint/typescript-estree@npm:^5.62.0#./.yarn/patches/@typescript-eslint-typescript-estree-npm-5.62.0-5d1ea132a9.patch"
   }
 }

package/docs/rules/no-if.md

@@ -1,58 +0,0 @@
-# Disallow conditional logic (`no-if`)
-
-❌ This rule is deprecated. It was replaced by
-[`jest/no-conditional-in-test`](no-conditional-in-test.md).
-
-<!-- end auto-generated rule header -->
-
-Conditional logic in tests is usually an indication that a test is attempting to
-cover too much, and not testing the logic it intends to. Each branch of code
-executing within an if statement will usually be better served by a test devoted
-to it.
-
-Conditionals are often used to satisfy the typescript type checker. In these
-cases, using the non-null assertion operator (!) would be best.
-
-## Rule details
-
-This rule prevents the use of if/ else statements and conditional (ternary)
-operations in tests.
-
-The following patterns are considered warnings:
-
-```js
-it('foo', () => {
-  if ('bar') {
-    // an if statement here is invalid
-    // you are probably testing too much
-  }
-});
-
-it('foo', () => {
-  const bar = foo ? 'bar' : null;
-});
-```
-
-These patterns would not be considered warnings:
-
-```js
-it('foo', () => {
-  // only test the 'foo' case
-});
-
-it('bar', () => {
-  // test the 'bar' case separately
-});
-
-it('foo', () => {
-  function foo(bar) {
-    // nested functions are valid
-    return foo ? bar : null;
-  }
-});
-```
-
-## When Not To Use It
-
-If you do not wish to prevent the use of if statements in tests, you can safely
-disable this rule.