Skip to content

Commit bbb173c

Browse files
committed
feat(require-hook): allow calls on jest global
1 parent 85899dc commit bbb173c

File tree

2 files changed

+10
-6
lines changed

2 files changed

+10
-6
lines changed

src/rules/__tests__/require-hook.test.ts

-5
Original file line numberDiff line numberDiff line change
@@ -223,11 +223,6 @@ ruleTester.run('require-hook', rule, {
223223
`,
224224
parserOptions: { sourceType: 'module' },
225225
errors: [
226-
{
227-
messageId: 'useHook',
228-
line: 4,
229-
column: 1,
230-
},
231226
{
232227
messageId: 'useHook',
233228
line: 16,

src/rules/require-hook.ts

+10-1
Original file line numberDiff line numberDiff line change
@@ -4,18 +4,27 @@ import {
44
} from '@typescript-eslint/experimental-utils';
55
import {
66
createRule,
7+
getNodeName,
78
isDescribeCall,
89
isFunction,
910
isHook,
1011
isTestCaseCall,
1112
} from './utils';
1213

14+
const isJestFnCall = (node: TSESTree.CallExpression): boolean => {
15+
if (isDescribeCall(node) || isTestCaseCall(node) || isHook(node)) {
16+
return true;
17+
}
18+
19+
return !!getNodeName(node)?.startsWith('jest.');
20+
};
21+
1322
const shouldBeInHook = (node: TSESTree.Node): boolean => {
1423
switch (node.type) {
1524
case AST_NODE_TYPES.ExpressionStatement:
1625
return shouldBeInHook(node.expression);
1726
case AST_NODE_TYPES.CallExpression:
18-
return !(isDescribeCall(node) || isTestCaseCall(node) || isHook(node));
27+
return !isJestFnCall(node);
1928

2029
default:
2130
return false;

0 commit comments

Comments
 (0)