eslint-plugin-jest 25.2.4 → 25.3.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/CHANGELOG.md +7 -0
- package/docs/rules/require-hook.md +37 -0
- package/lib/rules/require-hook.js +25 -6
- package/package.json +4 -4
package/CHANGELOG.md
CHANGED
|
@@ -1,3 +1,10 @@
|
|
|
1
|
+
# [25.3.0](https://github.com/jest-community/eslint-plugin-jest/compare/v25.2.4...v25.3.0) (2021-11-23)
|
|
2
|
+
|
|
3
|
+
|
|
4
|
+
### Features
|
|
5
|
+
|
|
6
|
+
* **require-hook:** add `allowedFunctionCalls` setting ([#983](https://github.com/jest-community/eslint-plugin-jest/issues/983)) ([9d9336a](https://github.com/jest-community/eslint-plugin-jest/commit/9d9336a7624c53c0bb3ee899b8cc336a0b3349cb))
|
|
7
|
+
|
|
1
8
|
## [25.2.4](https://github.com/jest-community/eslint-plugin-jest/compare/v25.2.3...v25.2.4) (2021-11-08)
|
|
2
9
|
|
|
3
10
|
|
|
@@ -148,3 +148,40 @@ afterEach(() => {
|
|
|
148
148
|
clearCityDatabase();
|
|
149
149
|
});
|
|
150
150
|
```
|
|
151
|
+
|
|
152
|
+
## Options
|
|
153
|
+
|
|
154
|
+
If there are methods that you want to call outside of hooks and tests, you can
|
|
155
|
+
mark them as allowed using the `allowedFunctionCalls` option.
|
|
156
|
+
|
|
157
|
+
```json
|
|
158
|
+
{
|
|
159
|
+
"jest/require-hook": [
|
|
160
|
+
"error",
|
|
161
|
+
{
|
|
162
|
+
"allowedFunctionCalls": ["enableAutoDestroy"]
|
|
163
|
+
}
|
|
164
|
+
]
|
|
165
|
+
}
|
|
166
|
+
```
|
|
167
|
+
|
|
168
|
+
Examples of **correct** code when using
|
|
169
|
+
`{ "allowedFunctionCalls": ["enableAutoDestroy"] }` option:
|
|
170
|
+
|
|
171
|
+
```js
|
|
172
|
+
/* eslint jest/require-hook: ["error", { "allowedFunctionCalls": ["enableAutoDestroy"] }] */
|
|
173
|
+
|
|
174
|
+
import { enableAutoDestroy, mount } from '@vue/test-utils';
|
|
175
|
+
import { initDatabase, tearDownDatabase } from './databaseUtils';
|
|
176
|
+
|
|
177
|
+
enableAutoDestroy(afterEach);
|
|
178
|
+
|
|
179
|
+
beforeEach(initDatabase);
|
|
180
|
+
afterEach(tearDownDatabase);
|
|
181
|
+
|
|
182
|
+
describe('Foo', () => {
|
|
183
|
+
test('always returns 42', () => {
|
|
184
|
+
expect(global.getAnswer()).toBe(42);
|
|
185
|
+
});
|
|
186
|
+
});
|
|
187
|
+
```
|
|
@@ -23,13 +23,13 @@ const isNullOrUndefined = node => {
|
|
|
23
23
|
return node.type === _experimentalUtils.AST_NODE_TYPES.Literal && node.value === null || (0, _utils.isIdentifier)(node, 'undefined');
|
|
24
24
|
};
|
|
25
25
|
|
|
26
|
-
const shouldBeInHook = node => {
|
|
26
|
+
const shouldBeInHook = (node, allowedFunctionCalls = []) => {
|
|
27
27
|
switch (node.type) {
|
|
28
28
|
case _experimentalUtils.AST_NODE_TYPES.ExpressionStatement:
|
|
29
|
-
return shouldBeInHook(node.expression);
|
|
29
|
+
return shouldBeInHook(node.expression, allowedFunctionCalls);
|
|
30
30
|
|
|
31
31
|
case _experimentalUtils.AST_NODE_TYPES.CallExpression:
|
|
32
|
-
return !isJestFnCall(node);
|
|
32
|
+
return !(isJestFnCall(node) || allowedFunctionCalls.includes((0, _utils.getNodeName)(node)));
|
|
33
33
|
|
|
34
34
|
case _experimentalUtils.AST_NODE_TYPES.VariableDeclaration:
|
|
35
35
|
{
|
|
@@ -59,14 +59,33 @@ var _default = (0, _utils.createRule)({
|
|
|
59
59
|
useHook: 'This should be done within a hook'
|
|
60
60
|
},
|
|
61
61
|
type: 'suggestion',
|
|
62
|
-
schema: [
|
|
62
|
+
schema: [{
|
|
63
|
+
type: 'object',
|
|
64
|
+
properties: {
|
|
65
|
+
allowedFunctionCalls: {
|
|
66
|
+
type: 'array',
|
|
67
|
+
items: {
|
|
68
|
+
type: 'string'
|
|
69
|
+
}
|
|
70
|
+
}
|
|
71
|
+
},
|
|
72
|
+
additionalProperties: false
|
|
73
|
+
}]
|
|
63
74
|
},
|
|
64
|
-
defaultOptions: [
|
|
75
|
+
defaultOptions: [{
|
|
76
|
+
allowedFunctionCalls: []
|
|
77
|
+
}],
|
|
65
78
|
|
|
66
79
|
create(context) {
|
|
80
|
+
var _context$options$;
|
|
81
|
+
|
|
82
|
+
const {
|
|
83
|
+
allowedFunctionCalls
|
|
84
|
+
} = (_context$options$ = context.options[0]) !== null && _context$options$ !== void 0 ? _context$options$ : {};
|
|
85
|
+
|
|
67
86
|
const checkBlockBody = body => {
|
|
68
87
|
for (const statement of body) {
|
|
69
|
-
if (shouldBeInHook(statement)) {
|
|
88
|
+
if (shouldBeInHook(statement, allowedFunctionCalls)) {
|
|
70
89
|
context.report({
|
|
71
90
|
node: statement,
|
|
72
91
|
messageId: 'useHook'
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "eslint-plugin-jest",
|
|
3
|
-
"version": "25.
|
|
3
|
+
"version": "25.3.0",
|
|
4
4
|
"description": "Eslint rules for Jest",
|
|
5
5
|
"keywords": [
|
|
6
6
|
"eslint",
|
|
@@ -89,8 +89,8 @@
|
|
|
89
89
|
"@babel/core": "^7.4.4",
|
|
90
90
|
"@babel/preset-env": "^7.4.4",
|
|
91
91
|
"@babel/preset-typescript": "^7.3.3",
|
|
92
|
-
"@commitlint/cli": "^
|
|
93
|
-
"@commitlint/config-conventional": "^
|
|
92
|
+
"@commitlint/cli": "^15.0.0",
|
|
93
|
+
"@commitlint/config-conventional": "^15.0.0",
|
|
94
94
|
"@schemastore/package": "^0.0.6",
|
|
95
95
|
"@semantic-release/changelog": "^6.0.0",
|
|
96
96
|
"@semantic-release/git": "^10.0.0",
|
|
@@ -115,7 +115,7 @@
|
|
|
115
115
|
"is-ci": "^3.0.0",
|
|
116
116
|
"jest": "^27.0.0",
|
|
117
117
|
"jest-runner-eslint": "^1.0.0",
|
|
118
|
-
"lint-staged": "^
|
|
118
|
+
"lint-staged": "^12.0.0",
|
|
119
119
|
"pinst": "^2.0.0",
|
|
120
120
|
"prettier": "^2.0.5",
|
|
121
121
|
"rimraf": "^3.0.0",
|