eslint-plugin-jest 22.15.0 → 22.17.0

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

@@ -7,17 +7,17 @@ exports.default = void 0;
 
 var _experimentalUtils = require("@typescript-eslint/experimental-utils");
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
 
 function isIdentifierResolves(node) {
   return node.property.type === _experimentalUtils.AST_NODE_TYPES.Identifier && node.property.name === 'resolves';
 }
 
 function isExpectResolves(node) {
-  return (0, _tsUtils.isExpectCall)(node.object) && isIdentifierResolves(node);
+  return (0, _utils.isExpectCall)(node.object) && isIdentifierResolves(node);
 }
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {

package/lib/rules/no-export.js

@@ -7,9 +7,9 @@ exports.default = void 0;
 
 var _experimentalUtils = require("@typescript-eslint/experimental-utils");
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -42,7 +42,7 @@ var _default = (0, _tsUtils.createRule)({
       },
 
       CallExpression(node) {
-        if ((0, _tsUtils.isTestCase)(node)) {
+        if ((0, _utils.isTestCase)(node)) {
           hasTestCase = true;
         }
       },

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

@@ -7,19 +7,19 @@ exports.default = void 0;
 
 var _experimentalUtils = require("@typescript-eslint/experimental-utils");
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
 
 const testFunctions = new Set(['describe', 'it', 'test']);
 
-const matchesTestFunction = object => object && 'name' in object && (object.name in _tsUtils.TestCaseName || object.name in _tsUtils.DescribeAlias);
+const matchesTestFunction = object => object && 'name' in object && (object.name in _utils.TestCaseName || object.name in _utils.DescribeAlias);
 
-const isCallToFocusedTestFunction = object => object && object.name[0] === 'f' && testFunctions.has(object.name.substring(1));
+const isCallToFocusedTestFunction = object => object && object.name.startsWith('f') && testFunctions.has(object.name.substring(1));
 
 const isPropertyNamedOnly = property => property && ('name' in property && property.name === 'only' || 'value' in property && property.value === 'only');
 
 const isCallToTestOnlyFunction = callee => matchesTestFunction(callee.object) && isPropertyNamedOnly(callee.property);
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {

package/lib/rules/no-hooks.js

@@ -5,9 +5,9 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -46,7 +46,7 @@ var _default = (0, _tsUtils.createRule)({
 
     return {
       CallExpression(node) {
-        if ((0, _tsUtils.isHook)(node) && !isWhitelisted(node)) {
+        if ((0, _utils.isHook)(node) && !isWhitelisted(node)) {
           context.report({
             node,
             messageId: 'unexpectedHook',

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

@@ -5,7 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
 
 function _slicedToArray(arr, i) { return _arrayWithHoles(arr) || _iterableToArrayLimit(arr, i) || _nonIterableRest(); }
 
@@ -20,7 +20,7 @@ const newDescribeContext = () => ({
   testTitles: []
 });
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -43,20 +43,20 @@ var _default = (0, _tsUtils.createRule)({
       CallExpression(node) {
         const currentLayer = contexts[contexts.length - 1];
 
-        if ((0, _tsUtils.isDescribe)(node)) {
+        if ((0, _utils.isDescribe)(node)) {
           contexts.push(newDescribeContext());
         }
 
         const _node$arguments = _slicedToArray(node.arguments, 1),
-              firstArgument = _node$arguments[0];
+              argument = _node$arguments[0];
 
-        if (!(0, _tsUtils.isStringNode)(firstArgument) || (0, _tsUtils.isTemplateLiteral)(firstArgument) && (0, _tsUtils.hasExpressions)(firstArgument)) {
+        if (!argument || !(0, _utils.isStringNode)(argument) || (0, _utils.isTemplateLiteral)(argument) && argument.expressions.length > 0) {
           return;
         }
 
-        const title = (0, _tsUtils.getStringValue)(firstArgument);
+        const title = (0, _utils.getAccessorValue)(argument);
 
-        if ((0, _tsUtils.isTestCase)(node)) {
+        if ((0, _utils.isTestCase)(node)) {
           if (currentLayer.testTitles.includes(title)) {
             context.report({
               messageId: 'multipleTestTitle',
@@ -67,7 +67,7 @@ var _default = (0, _tsUtils.createRule)({
           currentLayer.testTitles.push(title);
         }
 
-        if (!(0, _tsUtils.isDescribe)(node)) {
+        if (!(0, _utils.isDescribe)(node)) {
           return;
         }
 
@@ -82,7 +82,7 @@ var _default = (0, _tsUtils.createRule)({
       },
 
       'CallExpression:exit'(node) {
-        if ((0, _tsUtils.isDescribe)(node)) {
+        if ((0, _utils.isDescribe)(node)) {
           contexts.pop();
         }
       }

package/lib/rules/no-if.js

@@ -5,15 +5,15 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
 
 var _experimentalUtils = require("@typescript-eslint/experimental-utils");
 
-const testCaseNames = new Set([...Object.keys(_tsUtils.TestCaseName), 'it.only', 'it.skip', 'test.only', 'test.skip']);
+const testCaseNames = new Set([...Object.keys(_utils.TestCaseName), 'it.only', 'it.skip', 'test.only', 'test.skip']);
 
-const isTestArrowFunction = node => node.parent !== undefined && node.parent.type === _experimentalUtils.AST_NODE_TYPES.CallExpression && testCaseNames.has((0, _tsUtils.getNodeName)(node.parent.callee));
+const isTestArrowFunction = node => node.parent !== undefined && node.parent.type === _experimentalUtils.AST_NODE_TYPES.CallExpression && testCaseNames.has((0, _utils.getNodeName)(node.parent.callee));
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -49,7 +49,7 @@ var _default = (0, _tsUtils.createRule)({
 
     return {
       CallExpression(node) {
-        stack.push((0, _tsUtils.isTestCase)(node));
+        stack.push((0, _utils.isTestCase)(node));
       },
 
       FunctionExpression() {

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

@@ -7,9 +7,9 @@ exports.default = void 0;
 
 var _experimentalUtils = require("@typescript-eslint/experimental-utils");
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -34,14 +34,14 @@ var _default = (0, _tsUtils.createRule)({
     return {
       CallExpression(node) {
         const callee = node.callee;
-        const calleeName = (0, _tsUtils.getNodeName)(callee);
+        const calleeName = (0, _utils.getNodeName)(callee);
 
         if (!calleeName) {
           return;
         }
 
         if (calleeName === 'spyOn' || calleeName === 'spyOnProperty' || calleeName === 'fail' || calleeName === 'pending') {
-          if ((0, _tsUtils.scopeHasLocalReference)(context.getScope(), calleeName)) {
+          if ((0, _utils.scopeHasLocalReference)(context.getScope(), calleeName)) {
             // It's a local variable, not a jasmine global.
             return;
           }

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

@@ -5,9 +5,9 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     type: 'problem',

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

@@ -5,17 +5,20 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
+var _experimentalUtils = require("@typescript-eslint/experimental-utils");
+
 var _path = require("path");
 
-var _util = require("./util");
+var _utils = require("./utils");
 
-const reportOnViolation = (context, node) => {
-  const lineLimit = context.options[0] && Number.isFinite(context.options[0].maxSize) ? context.options[0].maxSize : 50;
+const reportOnViolation = (context, node, {
+  maxSize: lineLimit = 50,
+  whitelistedSnapshots = {}
+}) => {
   const startLine = node.loc.start.line;
   const endLine = node.loc.end.line;
   const lineCount = endLine - startLine;
-  const whitelistedSnapshots = context.options && context.options[0] && context.options[0].whitelistedSnapshots;
-  const allPathsAreAbsolute = Object.keys(whitelistedSnapshots || {}).every(_path.isAbsolute);
+  const allPathsAreAbsolute = Object.keys(whitelistedSnapshots).every(_path.isAbsolute);
 
   if (!allPathsAreAbsolute) {
     throw new Error('All paths for whitelistedSnapshots must be absolute. You can use JS config and `path.resolve`');
@@ -23,18 +26,18 @@ const reportOnViolation = (context, node) => {
 
   let isWhitelisted = false;
 
-  if (whitelistedSnapshots) {
+  if (whitelistedSnapshots && node.type === _experimentalUtils.AST_NODE_TYPES.ExpressionStatement && 'left' in node.expression && (0, _utils.isExpectMember)(node.expression.left)) {
     const fileName = context.getFilename();
     const whitelistedSnapshotsInFile = whitelistedSnapshots[fileName];
 
     if (whitelistedSnapshotsInFile) {
-      const snapshotName = (0, _util.getStringValue)(node.expression.left.property);
+      const snapshotName = (0, _utils.getAccessorValue)(node.expression.left.property);
       isWhitelisted = whitelistedSnapshotsInFile.some(name => {
-        if (name.test && typeof name.test === 'function') {
+        if (name instanceof RegExp) {
           return name.test(snapshotName);
         }
 
-        return name === snapshotName;
+        return snapshotName;
       });
     }
   }
@@ -51,15 +54,19 @@ const reportOnViolation = (context, node) => {
   }
 };
 
-var _default = {
+var _default = (0, _utils.createRule)({
+  name: __filename,
   meta: {
     docs: {
-      url: (0, _util.getDocsUrl)(__filename)
+      category: 'Best Practices',
+      description: 'disallow large snapshots',
+      recommended: false
     },
     messages: {
       noSnapshot: '`{{ lineCount }}`s should begin with lowercase',
       tooLongSnapshots: 'Expected Jest snapshot to be smaller than {{ lineLimit }} lines but was {{ lineCount }} lines long'
     },
+    type: 'suggestion',
     schema: [{
       type: 'object',
       properties: {
@@ -78,22 +85,21 @@ var _default = {
       additionalProperties: false
     }]
   },
+  defaultOptions: [{}],
 
-  create(context) {
+  create(context, [options]) {
     if (context.getFilename().endsWith('.snap')) {
       return {
         ExpressionStatement(node) {
-          reportOnViolation(context, node);
+          reportOnViolation(context, node, options);
         }
 
       };
     } else if (context.getFilename().endsWith('.js')) {
       return {
         CallExpression(node) {
-          const propertyName = node.callee.property && node.callee.property.name;
-
-          if (propertyName === 'toMatchInlineSnapshot' || propertyName === 'toThrowErrorMatchingInlineSnapshot') {
-            reportOnViolation(context, node);
+          if ('property' in node.callee && ((0, _utils.isSupportedAccessor)(node.callee.property, 'toMatchInlineSnapshot') || (0, _utils.isSupportedAccessor)(node.callee.property, 'toThrowErrorMatchingInlineSnapshot'))) {
+            reportOnViolation(context, node, options);
           }
         }
 
@@ -103,5 +109,6 @@ var _default = {
     return {};
   }
 
-};
+});
+
 exports.default = _default;

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

@@ -7,21 +7,29 @@ exports.default = void 0;
 
 var _path = require("path");
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
+
+function _slicedToArray(arr, i) { return _arrayWithHoles(arr) || _iterableToArrayLimit(arr, i) || _nonIterableRest(); }
+
+function _nonIterableRest() { throw new TypeError("Invalid attempt to destructure non-iterable instance"); }
+
+function _iterableToArrayLimit(arr, i) { var _arr = []; var _n = true; var _d = false; var _e = undefined; try { for (var _i = arr[Symbol.iterator](), _s; !(_n = (_s = _i.next()).done); _n = true) { _arr.push(_s.value); if (i && _arr.length === i) break; } } catch (err) { _d = true; _e = err; } finally { try { if (!_n && _i["return"] != null) _i["return"](); } finally { if (_d) throw _e; } } return _arr; }
+
+function _arrayWithHoles(arr) { if (Array.isArray(arr)) return arr; }
 
 const mocksDirName = '__mocks__';
 
 const isMockPath = path => path.split(_path.posix.sep).includes(mocksDirName);
 
-const isMockImportLiteral = expression => expression !== undefined && (0, _tsUtils.isLiteralNode)(expression) && typeof expression.value === 'string' && isMockPath(expression.value);
+const isMockImportLiteral = expression => (0, _utils.isStringNode)(expression) && isMockPath((0, _utils.getStringValue)(expression));
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     type: 'problem',
     docs: {
-      description: 'When using `jest.mock`, your tests (just like the code being tested) should import from `./x`, not `./__mocks__/x`. Not following this rule can lead to confusion, because you will have multiple instances of the mocked module',
       category: 'Best Practices',
+      description: 'When using `jest.mock`, your tests (just like the code being tested) should import from `./x`, not `./__mocks__/x`. Not following this rule can lead to confusion, because you will have multiple instances of the mocked module',
       recommended: 'error'
     },
     messages: {
@@ -34,7 +42,7 @@ var _default = (0, _tsUtils.createRule)({
   create(context) {
     return {
       ImportDeclaration(node) {
-        if (isMockImportLiteral(node.source)) {
+        if (node.source && isMockImportLiteral(node.source)) {
           context.report({
             node,
             messageId: 'noManualImport'
@@ -43,9 +51,12 @@ var _default = (0, _tsUtils.createRule)({
       },
 
       'CallExpression[callee.name="require"]'(node) {
-        if (isMockImportLiteral(node.arguments[0])) {
+        const _node$arguments = _slicedToArray(node.arguments, 1),
+              arg = _node$arguments[0];
+
+        if (arg && isMockImportLiteral(arg)) {
           context.report({
-            node: node.arguments[0],
+            node: arg,
             messageId: 'noManualImport'
           });
         }

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

@@ -7,7 +7,7 @@ exports.default = void 0;
 
 var _experimentalUtils = require("@typescript-eslint/experimental-utils");
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
 
 const getBlockType = stmt => {
   const func = stmt.parent;
@@ -22,7 +22,7 @@ const getBlockType = stmt => {
     return 'function';
   }
 
-  if ((0, _tsUtils.isFunction)(func) && func.parent) {
+  if ((0, _utils.isFunction)(func) && func.parent) {
     const expr = func.parent; // arrowfunction or function expr
 
     if (expr.type === _experimentalUtils.AST_NODE_TYPES.VariableDeclarator) {
@@ -30,7 +30,7 @@ const getBlockType = stmt => {
     } // if it's not a variable, it will be callExpr, we only care about describe
 
 
-    if (expr.type === _experimentalUtils.AST_NODE_TYPES.CallExpression && (0, _tsUtils.isDescribe)(expr)) {
+    if (expr.type === _experimentalUtils.AST_NODE_TYPES.CallExpression && (0, _utils.isDescribe)(expr)) {
       return 'describe';
     }
   }
@@ -39,14 +39,14 @@ const getBlockType = stmt => {
 };
 
 const isEach = node => {
-  if (node && node.callee && node.callee.type === _experimentalUtils.AST_NODE_TYPES.CallExpression && node.callee.callee && node.callee.callee.type === _experimentalUtils.AST_NODE_TYPES.MemberExpression && node.callee.callee.property && node.callee.callee.property.type === _experimentalUtils.AST_NODE_TYPES.Identifier && node.callee.callee.property.name === 'each' && node.callee.callee.object && node.callee.callee.object.type === _experimentalUtils.AST_NODE_TYPES.Identifier && _tsUtils.TestCaseName.hasOwnProperty(node.callee.callee.object.name)) {
+  if (node && node.callee && node.callee.type === _experimentalUtils.AST_NODE_TYPES.CallExpression && node.callee.callee && node.callee.callee.type === _experimentalUtils.AST_NODE_TYPES.MemberExpression && node.callee.callee.property && node.callee.callee.property.type === _experimentalUtils.AST_NODE_TYPES.Identifier && node.callee.callee.property.name === 'each' && node.callee.callee.object && node.callee.callee.object.type === _experimentalUtils.AST_NODE_TYPES.Identifier && _utils.TestCaseName.hasOwnProperty(node.callee.callee.object.name)) {
     return true;
   }
 
   return false;
 };
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -66,7 +66,7 @@ var _default = (0, _tsUtils.createRule)({
     const callStack = [];
     return {
       CallExpression(node) {
-        if ((0, _tsUtils.isExpectCall)(node)) {
+        if ((0, _utils.isExpectCall)(node)) {
           const parent = callStack[callStack.length - 1];
 
           if (!parent || parent === 'describe') {
@@ -79,7 +79,7 @@ var _default = (0, _tsUtils.createRule)({
           return;
         }
 
-        if ((0, _tsUtils.isTestCase)(node)) {
+        if ((0, _utils.isTestCase)(node)) {
           callStack.push('test');
         }
 
@@ -91,7 +91,7 @@ var _default = (0, _tsUtils.createRule)({
       'CallExpression:exit'(node) {
         const top = callStack[callStack.length - 1];
 
-        if (((0, _tsUtils.isTestCase)(node) && node.callee.type !== _experimentalUtils.AST_NODE_TYPES.MemberExpression || isEach(node)) && top === 'test' || node.callee.type === _experimentalUtils.AST_NODE_TYPES.TaggedTemplateExpression && top === 'template') {
+        if (((0, _utils.isTestCase)(node) && node.callee.type !== _experimentalUtils.AST_NODE_TYPES.MemberExpression || isEach(node)) && top === 'test' || node.callee.type === _experimentalUtils.AST_NODE_TYPES.TaggedTemplateExpression && top === 'template') {
           callStack.pop();
         }
       },

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

@@ -5,7 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
 
 function _slicedToArray(arr, i) { return _arrayWithHoles(arr) || _iterableToArrayLimit(arr, i) || _nonIterableRest(); }
 
@@ -15,7 +15,7 @@ function _iterableToArrayLimit(arr, i) { var _arr = []; var _n = true; var _d =
 
 function _arrayWithHoles(arr) { if (Array.isArray(arr)) return arr; }
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -35,14 +35,14 @@ var _default = (0, _tsUtils.createRule)({
   create(context) {
     return {
       CallExpression(node) {
-        if (!(0, _tsUtils.isTestCase)(node) || node.arguments.length !== 2) {
+        if (!(0, _utils.isTestCase)(node) || node.arguments.length !== 2) {
           return;
         }
 
         const _node$arguments = _slicedToArray(node.arguments, 2),
               callback = _node$arguments[1];
 
-        if (!(0, _tsUtils.isFunction)(callback) || callback.params.length !== 1) {
+        if (!(0, _utils.isFunction)(callback) || callback.params.length !== 1) {
           return;
         }
 
@@ -55,7 +55,7 @@ var _default = (0, _tsUtils.createRule)({
 
           fix(fixer) {
             const body = callback.body;
-            /* istanbul ignore if */
+            /* istanbul ignore if https://github.com/typescript-eslint/typescript-eslint/issues/734 */
 
             if (!body) {
               throw new Error(`Unexpected null when attempting to fix ${context.getFilename()} - please file a github issue at https://github.com/jest-community/eslint-plugin-jest`);

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

@@ -5,9 +5,9 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -27,8 +27,8 @@ var _default = (0, _tsUtils.createRule)({
   create(context) {
     return {
       CallExpression(node) {
-        const nodeName = (0, _tsUtils.getNodeName)(node.callee);
-        if (!nodeName || !(0, _tsUtils.isDescribe)(node) && !(0, _tsUtils.isTestCase)(node)) return;
+        const nodeName = (0, _utils.getNodeName)(node.callee);
+        if (!nodeName || !(0, _utils.isDescribe)(node) && !(0, _utils.isTestCase)(node)) return;
         const preferredNodeName = getPreferredNodeName(nodeName);
         if (!preferredNodeName) return;
         context.report({

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

@@ -5,7 +5,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _tsUtils = require("./tsUtils");
+var _utils = require("./utils");
 
 function _slicedToArray(arr, i) { return _arrayWithHoles(arr) || _iterableToArrayLimit(arr, i) || _nonIterableRest(); }
 
@@ -22,14 +22,14 @@ const getBody = args => {
   const _args = _slicedToArray(args, 2),
         secondArg = _args[1];
 
-  if (secondArg && (0, _tsUtils.isFunction)(secondArg) && secondArg.body && secondArg.body.type === BLOCK_STATEMENT) {
+  if (secondArg && (0, _utils.isFunction)(secondArg) && secondArg.body && secondArg.body.type === BLOCK_STATEMENT) {
     return secondArg.body.body;
   }
 
   return [];
 };
 
-var _default = (0, _tsUtils.createRule)({
+var _default = (0, _utils.createRule)({
   name: __filename,
   meta: {
     docs: {
@@ -48,7 +48,7 @@ var _default = (0, _tsUtils.createRule)({
   create(context) {
     return {
       CallExpression(node) {
-        if (!(0, _tsUtils.isTestCase)(node)) return;
+        if (!(0, _utils.isTestCase)(node)) return;
         const body = getBody(node.arguments);
         const returnStmt = body.find(t => t.type === RETURN_STATEMENT);
         if (!returnStmt) return;

package/lib/rules/no-truthy-falsy.js

@@ -5,41 +5,53 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.default = void 0;
 
-var _util = require("./util");
+var _utils = require("./utils");
 
-var _default = {
+// todo: refactor into "ban-matchers"
+var _default = (0, _utils.createRule)({
+  name: __filename,
   meta: {
     docs: {
-      url: (0, _util.getDocsUrl)(__filename)
+      category: 'Best Practices',
+      description: 'Disallow using `toBeTruthy()` & `toBeFalsy()`',
+      recommended: false
     },
     messages: {
-      avoidMessage: 'Avoid {{methodName}}'
+      avoidMessage: 'Avoid {{ methodName }}'
     },
+    type: 'suggestion',
     schema: []
   },
+  defaultOptions: [],
 
   create(context) {
     return {
       CallExpression(node) {
-        if ((0, _util.expectCaseWithParent)(node) || (0, _util.expectNotCase)(node) || (0, _util.expectResolvesCase)(node) || (0, _util.expectRejectsCase)(node)) {
-          const targetNode = node.parent.parent.type === 'MemberExpression' ? node.parent : node;
-          const methodNode = (0, _util.method)(targetNode);
-          const methodName = methodNode.name;
-
-          if (methodName === 'toBeTruthy' || methodName === 'toBeFalsy') {
-            context.report({
-              data: {
-                methodName
-              },
-              messageId: 'avoidMessage',
-              node: methodNode
-            });
-          }
+        if (!(0, _utils.isExpectCall)(node)) {
+          return;
         }
+
+        const _parseExpectCall = (0, _utils.parseExpectCall)(node),
+              matcher = _parseExpectCall.matcher;
+
+        if (!matcher || !['toBeTruthy', 'toBeFalsy'].includes(matcher.name)) {
+          return;
+        }
+
+        context.report({
+          data: {
+            methodName: matcher.name
+          },
+          // todo: rename to 'matcherName'
+          messageId: 'avoidMessage',
+          // todo: rename to 'avoidMatcher'
+          node: matcher.node.property
+        });
       }
 
     };
   }
 
-};
+});
+
 exports.default = _default;