forked from testing-library/eslint-plugin-testing-library
-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathno-await-sync-events.ts
127 lines (112 loc) · 3.57 KB
/
no-await-sync-events.ts
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
import { ASTUtils, TSESTree } from '@typescript-eslint/utils';
import { createTestingLibraryRule } from '../create-testing-library-rule';
import {
getDeepestIdentifierNode,
getPropertyIdentifierNode,
isLiteral,
isObjectExpression,
isProperty,
} from '../node-utils';
const USER_EVENT_ASYNC_EXCEPTIONS: string[] = ['type', 'keyboard'];
const VALID_EVENT_MODULES = ['fire-event', 'user-event'] as const;
export const RULE_NAME = 'no-await-sync-events';
export type MessageIds = 'noAwaitSyncEvents';
type Options = [
{ eventModules?: readonly typeof VALID_EVENT_MODULES[number][] }
];
export default createTestingLibraryRule<Options, MessageIds>({
name: RULE_NAME,
meta: {
type: 'problem',
docs: {
description: 'Disallow unnecessary `await` for sync events',
recommendedConfig: {
dom: false,
angular: false,
react: false,
vue: false,
marko: false,
},
},
messages: {
noAwaitSyncEvents:
'`{{ name }}` is sync and does not need `await` operator',
},
schema: [
{
type: 'object',
properties: {
eventModules: {
type: 'array',
minItems: 1,
items: {
enum: VALID_EVENT_MODULES,
},
},
},
additionalProperties: false,
},
],
},
defaultOptions: [{ eventModules: VALID_EVENT_MODULES }],
create(context, [options], helpers) {
const { eventModules = VALID_EVENT_MODULES } = options;
// userEvent.type() and userEvent.keyboard() are exceptions, which returns a
// Promise. But it is only necessary to wait when delay option other than 0
// is specified. So this rule has a special exception for the case await:
// - userEvent.type(element, 'abc', {delay: 1234})
// - userEvent.keyboard('abc', {delay: 1234})
return {
'AwaitExpression > CallExpression'(node: TSESTree.CallExpression) {
const simulateEventFunctionIdentifier = getDeepestIdentifierNode(node);
if (!simulateEventFunctionIdentifier) {
return;
}
const isUserEventMethod = helpers.isUserEventMethod(
simulateEventFunctionIdentifier
);
const isFireEventMethod = helpers.isFireEventMethod(
simulateEventFunctionIdentifier
);
const isSimulateEventMethod = isUserEventMethod || isFireEventMethod;
if (!isSimulateEventMethod) {
return;
}
if (isFireEventMethod && !eventModules.includes('fire-event')) {
return;
}
if (isUserEventMethod && !eventModules.includes('user-event')) {
return;
}
const lastArg = node.arguments[node.arguments.length - 1];
const hasDelay =
isObjectExpression(lastArg) &&
lastArg.properties.some(
(property) =>
isProperty(property) &&
ASTUtils.isIdentifier(property.key) &&
property.key.name === 'delay' &&
isLiteral(property.value) &&
!!property.value.value &&
property.value.value > 0
);
const simulateEventFunctionName = simulateEventFunctionIdentifier.name;
if (
USER_EVENT_ASYNC_EXCEPTIONS.includes(simulateEventFunctionName) &&
hasDelay
) {
return;
}
context.report({
node,
messageId: 'noAwaitSyncEvents',
data: {
name: `${
getPropertyIdentifierNode(node)?.name
}.${simulateEventFunctionName}`,
},
});
},
};
},
});