eslint-plugin-jest 24.4.2 → 26.1.0

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

@@ -5,13 +5,13 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _experimentalUtils = require("@typescript-eslint/experimental-utils");
+var _utils = require("@typescript-eslint/utils");
 
-var _utils = require("./utils");
+var _utils2 = require("./utils");
 
-const isCatchCall = node => node.callee.type === _experimentalUtils.AST_NODE_TYPES.MemberExpression && (0, _utils.isSupportedAccessor)(node.callee.property, 'catch');
+const isCatchCall = node => node.callee.type === _utils.AST_NODE_TYPES.MemberExpression && (0, _utils2.isSupportedAccessor)(node.callee.property, 'catch');
 
-var _default = (0, _utils.createRule)({
+var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -39,7 +39,7 @@ var _default = (0, _utils.createRule)({
     return {
       FunctionDeclaration(node) {
         const declaredVariables = context.getDeclaredVariables(node);
-        const testCallExpressions = (0, _utils.getTestCallExpressionsFromDeclaredVariables)(declaredVariables);
+        const testCallExpressions = (0, _utils2.getTestCallExpressionsFromDeclaredVariables)(declaredVariables);
 
         if (testCallExpressions.length > 0) {
           inTestCase = true;
@@ -47,7 +47,7 @@ var _default = (0, _utils.createRule)({
       },
 
       CallExpression(node) {
-        if ((0, _utils.isTestCaseCall)(node)) {
+        if ((0, _utils2.isTestCaseCall)(node)) {
           inTestCase = true;
         }
 
@@ -55,14 +55,14 @@ var _default = (0, _utils.createRule)({
           inPromiseCatch = true;
         }
 
-        if (inTestCase && (0, _utils.isExpectCall)(node) && conditionalDepth > 0) {
+        if (inTestCase && (0, _utils2.isExpectCall)(node) && conditionalDepth > 0) {
           context.report({
             messageId: 'conditionalExpect',
             node
           });
         }
 
-        if (inPromiseCatch && (0, _utils.isExpectCall)(node)) {
+        if (inPromiseCatch && (0, _utils2.isExpectCall)(node)) {
           context.report({
             messageId: 'conditionalExpect',
             node
@@ -71,7 +71,7 @@ var _default = (0, _utils.createRule)({
       },
 
       'CallExpression:exit'(node) {
-        if ((0, _utils.isTestCaseCall)(node)) {
+        if ((0, _utils2.isTestCaseCall)(node)) {
           inTestCase = false;
         }
 

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

@@ -0,0 +1,60 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.default = void 0;
+
+var _utils = require("./utils");
+
+var _default = (0, _utils.createRule)({
+  name: __filename,
+  meta: {
+    docs: {
+      description: 'Disallow conditional logic in tests',
+      category: 'Best Practices',
+      recommended: false
+    },
+    messages: {
+      conditionalInTest: 'Avoid having conditionals in tests'
+    },
+    type: 'problem',
+    schema: []
+  },
+  defaultOptions: [],
+
+  create(context) {
+    let inTestCase = false;
+
+    const maybeReportConditional = node => {
+      if (inTestCase) {
+        context.report({
+          messageId: 'conditionalInTest',
+          node
+        });
+      }
+    };
+
+    return {
+      CallExpression(node) {
+        if ((0, _utils.isTestCaseCall)(node)) {
+          inTestCase = true;
+        }
+      },
+
+      'CallExpression:exit'(node) {
+        if ((0, _utils.isTestCaseCall)(node)) {
+          inTestCase = false;
+        }
+      },
+
+      IfStatement: maybeReportConditional,
+      SwitchStatement: maybeReportConditional,
+      ConditionalExpression: maybeReportConditional,
+      LogicalExpression: maybeReportConditional
+    };
+  }
+
+});
+
+exports.default = _default;

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

@@ -3,42 +3,24 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.default = exports._clearCachedJestVersion = void 0;
+exports.default = void 0;
 
-var _experimentalUtils = require("@typescript-eslint/experimental-utils");
+var _utils = require("@typescript-eslint/utils");
 
-var _utils = require("./utils");
+var _detectJestVersion = require("./detectJestVersion");
 
-let cachedJestVersion = null;
-/** @internal */
+var _utils2 = require("./utils");
 
-const _clearCachedJestVersion = () => cachedJestVersion = null;
-
-exports._clearCachedJestVersion = _clearCachedJestVersion;
-
-const detectJestVersion = () => {
-  if (cachedJestVersion) {
-    return cachedJestVersion;
+const parseJestVersion = rawVersion => {
+  if (typeof rawVersion === 'number') {
+    return rawVersion;
   }
 
-  try {
-    const jestPath = require.resolve('jest/package.json', {
-      paths: [process.cwd()]
-    });
-
-    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');
+  const [majorVersion] = rawVersion.split('.');
+  return parseInt(majorVersion, 10);
 };
 
-var _default = (0, _utils.createRule)({
+var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -58,7 +40,7 @@ var _default = (0, _utils.createRule)({
   create(context) {
     var _context$settings, _context$settings$jes;
 
-    const jestVersion = ((_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) || detectJestVersion();
+    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, _detectJestVersion.detectJestVersion)());
     const deprecations = { ...(jestVersion >= 15 && {
         'jest.resetModuleRegistry': 'jest.resetModules'
       }),
@@ -78,11 +60,11 @@ var _default = (0, _utils.createRule)({
     };
     return {
       CallExpression(node) {
-        if (node.callee.type !== _experimentalUtils.AST_NODE_TYPES.MemberExpression) {
+        if (node.callee.type !== _utils.AST_NODE_TYPES.MemberExpression) {
           return;
         }
 
-        const deprecation = (0, _utils.getNodeName)(node);
+        const deprecation = (0, _utils2.getNodeName)(node);
 
         if (!deprecation || !(deprecation in deprecations)) {
           return;
@@ -103,7 +85,7 @@ var _default = (0, _utils.createRule)({
           fix(fixer) {
             let [name, func] = replacement.split('.');
 
-            if (callee.property.type === _experimentalUtils.AST_NODE_TYPES.Literal) {
+            if (callee.property.type === _utils.AST_NODE_TYPES.Literal) {
               func = `'${func}'`;
             }
 

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

@@ -5,27 +5,27 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _experimentalUtils = require("@typescript-eslint/experimental-utils");
+var _utils = require("@typescript-eslint/utils");
 
-var _utils = require("./utils");
+var _utils2 = require("./utils");
 
 const findCallbackArg = (node, isJestEach) => {
   if (isJestEach) {
     return node.arguments[1];
   }
 
-  if ((0, _utils.isHook)(node) && node.arguments.length >= 1) {
+  if ((0, _utils2.isHook)(node) && node.arguments.length >= 1) {
     return node.arguments[0];
   }
 
-  if ((0, _utils.isTestCaseCall)(node) && node.arguments.length >= 2) {
+  if ((0, _utils2.isTestCaseCall)(node) && node.arguments.length >= 2) {
     return node.arguments[1];
   }
 
   return null;
 };
 
-var _default = (0, _utils.createRule)({
+var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -51,9 +51,9 @@ var _default = (0, _utils.createRule)({
         var _getNodeName$endsWith, _getNodeName;
 
         // done is the second argument for it.each, not the first
-        const isJestEach = (_getNodeName$endsWith = (_getNodeName = (0, _utils.getNodeName)(node.callee)) === null || _getNodeName === void 0 ? void 0 : _getNodeName.endsWith('.each')) !== null && _getNodeName$endsWith !== void 0 ? _getNodeName$endsWith : false;
+        const isJestEach = (_getNodeName$endsWith = (_getNodeName = (0, _utils2.getNodeName)(node.callee)) === null || _getNodeName === void 0 ? void 0 : _getNodeName.endsWith('.each')) !== null && _getNodeName$endsWith !== void 0 ? _getNodeName$endsWith : false;
 
-        if (isJestEach && node.callee.type !== _experimentalUtils.AST_NODE_TYPES.TaggedTemplateExpression) {
+        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
           // to its complexity (see jest-community/eslint-plugin-jest#710)
@@ -63,13 +63,13 @@ var _default = (0, _utils.createRule)({
         const callback = findCallbackArg(node, isJestEach);
         const callbackArgIndex = Number(isJestEach);
 
-        if (!callback || !(0, _utils.isFunction)(callback) || callback.params.length !== 1 + callbackArgIndex) {
+        if (!callback || !(0, _utils2.isFunction)(callback) || callback.params.length !== 1 + callbackArgIndex) {
           return;
         }
 
         const argument = callback.params[callbackArgIndex];
 
-        if (argument.type !== _experimentalUtils.AST_NODE_TYPES.Identifier) {
+        if (argument.type !== _utils.AST_NODE_TYPES.Identifier) {
           context.report({
             node: argument,
             messageId: 'noDoneCallback'
@@ -126,7 +126,7 @@ var _default = (0, _utils.createRule)({
               let afterReplacement = ')';
               let replaceBefore = true;
 
-              if (body.type === _experimentalUtils.AST_NODE_TYPES.BlockStatement) {
+              if (body.type === _utils.AST_NODE_TYPES.BlockStatement) {
                 const keyword = 'return';
                 beforeReplacement = `${keyword} ${beforeReplacement}{`;
                 afterReplacement += '}';

package/lib/rules/no-export.js

@@ -5,11 +5,11 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _experimentalUtils = require("@typescript-eslint/experimental-utils");
+var _utils = require("@typescript-eslint/utils");
 
-var _utils = require("./utils");
+var _utils2 = require("./utils");
 
-var _default = (0, _utils.createRule)({
+var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -41,7 +41,7 @@ var _default = (0, _utils.createRule)({
       },
 
       CallExpression(node) {
-        if ((0, _utils.isTestCaseCall)(node)) {
+        if ((0, _utils2.isTestCaseCall)(node)) {
           hasTestCase = true;
         }
       },
@@ -56,14 +56,14 @@ var _default = (0, _utils.createRule)({
           property
         } = node;
 
-        if (object.type === _experimentalUtils.AST_NODE_TYPES.MemberExpression) {
+        if (object.type === _utils.AST_NODE_TYPES.MemberExpression) {
           ({
             object,
             property
           } = object);
         }
 
-        if ('name' in object && object.name === 'module' && property.type === _experimentalUtils.AST_NODE_TYPES.Identifier && /^exports?$/u.test(property.name)) {
+        if ('name' in object && object.name === 'module' && property.type === _utils.AST_NODE_TYPES.Identifier && /^exports?$/u.test(property.name)) {
           exportNodes.push(node);
         }
       }

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

@@ -5,21 +5,21 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _experimentalUtils = require("@typescript-eslint/experimental-utils");
+var _utils = require("@typescript-eslint/utils");
 
-var _utils = require("./utils");
+var _utils2 = require("./utils");
 
 const findOnlyNode = node => {
-  const callee = node.callee.type === _experimentalUtils.AST_NODE_TYPES.TaggedTemplateExpression ? node.callee.tag : node.callee.type === _experimentalUtils.AST_NODE_TYPES.CallExpression ? node.callee.callee : node.callee;
+  const callee = 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 (callee.type === _experimentalUtils.AST_NODE_TYPES.MemberExpression) {
-    if (callee.object.type === _experimentalUtils.AST_NODE_TYPES.MemberExpression) {
-      if ((0, _utils.isSupportedAccessor)(callee.object.property, 'only')) {
+  if (callee.type === _utils.AST_NODE_TYPES.MemberExpression) {
+    if (callee.object.type === _utils.AST_NODE_TYPES.MemberExpression) {
+      if ((0, _utils2.isSupportedAccessor)(callee.object.property, 'only')) {
         return callee.object.property;
       }
     }
 
-    if ((0, _utils.isSupportedAccessor)(callee.property, 'only')) {
+    if ((0, _utils2.isSupportedAccessor)(callee.property, 'only')) {
       return callee.property;
     }
   }
@@ -27,7 +27,7 @@ const findOnlyNode = node => {
   return null;
 };
 
-var _default = (0, _utils.createRule)({
+var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -47,11 +47,11 @@ var _default = (0, _utils.createRule)({
   defaultOptions: [],
   create: context => ({
     CallExpression(node) {
-      if (!(0, _utils.isDescribeCall)(node) && !(0, _utils.isTestCaseCall)(node)) {
+      if (!(0, _utils2.isDescribeCall)(node) && !(0, _utils2.isTestCaseCall)(node)) {
         return;
       }
 
-      if ((0, _utils.getNodeName)(node).startsWith('f')) {
+      if ((0, _utils2.getNodeName)(node).startsWith('f')) {
         context.report({
           messageId: 'focusedTest',
           node,
@@ -74,7 +74,7 @@ var _default = (0, _utils.createRule)({
         node: onlyNode,
         suggest: [{
           messageId: 'suggestRemoveFocus',
-          fix: fixer => fixer.removeRange([onlyNode.range[0] - 1, onlyNode.range[1] + Number(onlyNode.type !== _experimentalUtils.AST_NODE_TYPES.Identifier)])
+          fix: fixer => fixer.removeRange([onlyNode.range[0] - 1, onlyNode.range[1] + Number(onlyNode.type !== _utils.AST_NODE_TYPES.Identifier)])
         }]
       });
     }

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

@@ -5,8 +5,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _experimentalUtils = require("@typescript-eslint/experimental-utils");
-
 var _utils = require("./utils");
 
 const newDescribeContext = () => ({
@@ -35,13 +33,15 @@ var _default = (0, _utils.createRule)({
     const contexts = [newDescribeContext()];
     return {
       CallExpression(node) {
+        var _getNodeName;
+
         const currentLayer = contexts[contexts.length - 1];
 
         if ((0, _utils.isDescribeCall)(node)) {
           contexts.push(newDescribeContext());
         }
 
-        if (node.callee.type === _experimentalUtils.AST_NODE_TYPES.TaggedTemplateExpression) {
+        if ((_getNodeName = (0, _utils.getNodeName)(node.callee)) !== null && _getNodeName !== void 0 && _getNodeName.endsWith('.each')) {
           return;
         }
 

package/lib/rules/no-if.js

@@ -5,21 +5,21 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _experimentalUtils = require("@typescript-eslint/experimental-utils");
+var _utils = require("@typescript-eslint/utils");
 
-var _utils = require("./utils");
+var _utils2 = require("./utils");
 
-const testCaseNames = new Set([...Object.keys(_utils.TestCaseName), 'it.only', 'it.concurrent.only', 'it.skip', 'it.concurrent.skip', 'test.only', 'test.concurrent.only', 'test.skip', 'test.concurrent.skip', 'fit.concurrent']);
+const testCaseNames = new Set([...Object.keys(_utils2.TestCaseName), 'it.only', 'it.concurrent.only', 'it.skip', 'it.concurrent.skip', 'test.only', 'test.concurrent.only', 'test.skip', 'test.concurrent.skip', 'fit.concurrent']);
 
-const isTestFunctionExpression = node => node.parent !== undefined && node.parent.type === _experimentalUtils.AST_NODE_TYPES.CallExpression && testCaseNames.has((0, _utils.getNodeName)(node.parent.callee));
+const isTestFunctionExpression = node => node.parent !== undefined && node.parent.type === _utils.AST_NODE_TYPES.CallExpression && testCaseNames.has((0, _utils2.getNodeName)(node.parent.callee));
 
 const conditionName = {
-  [_experimentalUtils.AST_NODE_TYPES.ConditionalExpression]: 'conditional',
-  [_experimentalUtils.AST_NODE_TYPES.SwitchStatement]: 'switch',
-  [_experimentalUtils.AST_NODE_TYPES.IfStatement]: 'if'
+  [_utils.AST_NODE_TYPES.ConditionalExpression]: 'conditional',
+  [_utils.AST_NODE_TYPES.SwitchStatement]: 'switch',
+  [_utils.AST_NODE_TYPES.IfStatement]: 'if'
 };
 
-var _default = (0, _utils.createRule)({
+var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -30,6 +30,8 @@ var _default = (0, _utils.createRule)({
     messages: {
       conditionalInTest: 'Test should not contain {{ condition }} statements.'
     },
+    deprecated: true,
+    replacedBy: ['no-conditional-in-test'],
     schema: [],
     type: 'suggestion'
   },
@@ -56,10 +58,10 @@ var _default = (0, _utils.createRule)({
 
     return {
       CallExpression(node) {
-        if ((0, _utils.isTestCaseCall)(node)) {
+        if ((0, _utils2.isTestCaseCall)(node)) {
           stack.push(true);
 
-          if ((0, _utils.getNodeName)(node).endsWith('each')) {
+          if ((0, _utils2.getNodeName)(node).endsWith('each')) {
             stack.push(true);
           }
         }
@@ -71,7 +73,7 @@ var _default = (0, _utils.createRule)({
 
       FunctionDeclaration(node) {
         const declaredVariables = context.getDeclaredVariables(node);
-        const testCallExpressions = (0, _utils.getTestCallExpressionsFromDeclaredVariables)(declaredVariables);
+        const testCallExpressions = (0, _utils2.getTestCallExpressionsFromDeclaredVariables)(declaredVariables);
         stack.push(testCallExpressions.length > 0);
       },
 

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

@@ -5,11 +5,11 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _experimentalUtils = require("@typescript-eslint/experimental-utils");
+var _utils = require("@typescript-eslint/utils");
 
-var _utils = require("./utils");
+var _utils2 = require("./utils");
 
-var _default = (0, _utils.createRule)({
+var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -28,13 +28,13 @@ var _default = (0, _utils.createRule)({
   create(context) {
     return {
       CallExpression(node) {
-        if (!(0, _utils.isExpectCall)(node)) {
+        if (!(0, _utils2.isExpectCall)(node)) {
           return;
         }
 
         const {
           matcher
-        } = (0, _utils.parseExpectCall)(node);
+        } = (0, _utils2.parseExpectCall)(node);
 
         if (!matcher) {
           return;
@@ -45,7 +45,7 @@ var _default = (0, _utils.createRule)({
 
           // Check all since the optional 'propertyMatchers' argument might be present
           (_matcher$arguments = matcher.arguments) === null || _matcher$arguments === void 0 ? void 0 : _matcher$arguments.forEach(argument => {
-            if (argument.type === _experimentalUtils.AST_NODE_TYPES.TemplateLiteral && argument.expressions.length > 0) {
+            if (argument.type === _utils.AST_NODE_TYPES.TemplateLiteral && argument.expressions.length > 0) {
               context.report({
                 messageId: 'noInterpolation',
                 node: argument

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

@@ -5,11 +5,11 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _experimentalUtils = require("@typescript-eslint/experimental-utils");
+var _utils = require("@typescript-eslint/utils");
 
-var _utils = require("./utils");
+var _utils2 = require("./utils");
 
-var _default = (0, _utils.createRule)({
+var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -36,14 +36,14 @@ var _default = (0, _utils.createRule)({
         const {
           callee
         } = node;
-        const calleeName = (0, _utils.getNodeName)(callee);
+        const calleeName = (0, _utils2.getNodeName)(callee);
 
         if (!calleeName) {
           return;
         }
 
         if (calleeName === 'spyOn' || calleeName === 'spyOnProperty' || calleeName === 'fail' || calleeName === 'pending') {
-          if ((0, _utils.scopeHasLocalReference)(context.getScope(), calleeName)) {
+          if ((0, _utils2.scopeHasLocalReference)(context.getScope(), calleeName)) {
             // It's a local variable, not a jasmine global.
             return;
           }
@@ -79,7 +79,7 @@ var _default = (0, _utils.createRule)({
           return;
         }
 
-        if (callee.type === _experimentalUtils.AST_NODE_TYPES.MemberExpression && calleeName.startsWith('jasmine.')) {
+        if (callee.type === _utils.AST_NODE_TYPES.MemberExpression && calleeName.startsWith('jasmine.')) {
           const functionName = calleeName.replace('jasmine.', '');
 
           if (functionName === 'any' || functionName === 'anything' || functionName === 'arrayContaining' || functionName === 'objectContaining' || functionName === 'stringMatching') {
@@ -127,19 +127,19 @@ var _default = (0, _utils.createRule)({
       },
 
       MemberExpression(node) {
-        if ((0, _utils.isSupportedAccessor)(node.object, 'jasmine')) {
+        if ((0, _utils2.isSupportedAccessor)(node.object, 'jasmine')) {
           const {
             parent,
             property
           } = node;
 
-          if (parent && parent.type === _experimentalUtils.AST_NODE_TYPES.AssignmentExpression) {
-            if ((0, _utils.isSupportedAccessor)(property, 'DEFAULT_TIMEOUT_INTERVAL')) {
+          if (parent && parent.type === _utils.AST_NODE_TYPES.AssignmentExpression) {
+            if ((0, _utils2.isSupportedAccessor)(property, 'DEFAULT_TIMEOUT_INTERVAL')) {
               const {
                 right
               } = parent;
 
-              if (right.type === _experimentalUtils.AST_NODE_TYPES.Literal) {
+              if (right.type === _utils.AST_NODE_TYPES.Literal) {
                 context.report({
                   fix: fixer => [fixer.replaceText(parent, `jest.setTimeout(${right.value})`)],
                   node,

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

@@ -7,9 +7,9 @@ exports.default = void 0;
 
 var _path = require("path");
 
-var _experimentalUtils = require("@typescript-eslint/experimental-utils");
+var _utils = require("@typescript-eslint/utils");
 
-var _utils = require("./utils");
+var _utils2 = require("./utils");
 
 const reportOnViolation = (context, node, {
   maxSize: lineLimit = 50,
@@ -26,12 +26,12 @@ const reportOnViolation = (context, node, {
 
   let isAllowed = false;
 
-  if (node.type === _experimentalUtils.AST_NODE_TYPES.ExpressionStatement && 'left' in node.expression && (0, _utils.isExpectMember)(node.expression.left)) {
+  if (node.type === _utils.AST_NODE_TYPES.ExpressionStatement && 'left' in node.expression && (0, _utils2.isExpectMember)(node.expression.left)) {
     const fileName = context.getFilename();
     const allowedSnapshotsInFile = allowedSnapshots[fileName];
 
     if (allowedSnapshotsInFile) {
-      const snapshotName = (0, _utils.getAccessorValue)(node.expression.left.property);
+      const snapshotName = (0, _utils2.getAccessorValue)(node.expression.left.property);
       isAllowed = allowedSnapshotsInFile.some(name => {
         if (name instanceof RegExp) {
           return name.test(snapshotName);
@@ -54,7 +54,7 @@ const reportOnViolation = (context, node, {
   }
 };
 
-var _default = (0, _utils.createRule)({
+var _default = (0, _utils2.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -100,24 +100,24 @@ var _default = (0, _utils.createRule)({
 
     return {
       CallExpression(node) {
-        var _matcher$node$parent;
+        var _matcher$arguments;
 
-        if (!(0, _utils.isExpectCall)(node)) {
+        if (!(0, _utils2.isExpectCall)(node)) {
           return;
         }
 
         const {
           matcher
-        } = (0, _utils.parseExpectCall)(node);
+        } = (0, _utils2.parseExpectCall)(node);
 
-        if ((matcher === null || matcher === void 0 ? void 0 : (_matcher$node$parent = matcher.node.parent) === null || _matcher$node$parent === void 0 ? void 0 : _matcher$node$parent.type) !== _experimentalUtils.AST_NODE_TYPES.CallExpression) {
+        if ((matcher === null || matcher === void 0 ? void 0 : matcher.node.parent.type) !== _utils.AST_NODE_TYPES.CallExpression) {
           return;
         }
 
-        if (['toMatchInlineSnapshot', 'toThrowErrorMatchingInlineSnapshot'].includes(matcher.name)) {
+        if (['toMatchInlineSnapshot', 'toThrowErrorMatchingInlineSnapshot'].includes(matcher.name) && (_matcher$arguments = matcher.arguments) !== null && _matcher$arguments !== void 0 && _matcher$arguments.length) {
           var _options$inlineMaxSiz;
 
-          reportOnViolation(context, matcher.node.parent, { ...options,
+          reportOnViolation(context, matcher.arguments[0], { ...options,
             maxSize: (_options$inlineMaxSiz = options.inlineMaxSize) !== null && _options$inlineMaxSiz !== void 0 ? _options$inlineMaxSiz : options.maxSize
           });
         }