Skip to content

Commit ea893cc

Browse files
feat(prefer-query-by-disappearance): add to all configs by default (#487)
BREAKING CHANGE: `prefer-query-by-disappearance` is now enabled by default in all configs
1 parent 17d5376 commit ea893cc

File tree

7 files changed

+13
-5
lines changed

7 files changed

+13
-5
lines changed

README.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -209,7 +209,7 @@ To enable this configuration use the `extends` property in your
209209
| [`testing-library/prefer-explicit-assert`](./docs/rules/prefer-explicit-assert.md) | Suggest using explicit assertions rather than standalone queries | | |
210210
| [`testing-library/prefer-find-by`](./docs/rules/prefer-find-by.md) | Suggest using `find(All)By*` query instead of `waitFor` + `get(All)By*` to wait for elements | 🔧 | ![dom-badge][] ![angular-badge][] ![react-badge][] ![vue-badge][] |
211211
| [`testing-library/prefer-presence-queries`](./docs/rules/prefer-presence-queries.md) | Ensure appropriate `get*`/`query*` queries are used with their respective matchers | | ![dom-badge][] ![angular-badge][] ![react-badge][] ![vue-badge][] |
212-
| [`testing-library/prefer-query-by-disappearance`](./docs/rules/prefer-query-by-disappearance.md) | Suggest using `queryBy*` queries when waiting for disappearance | | |
212+
| [`testing-library/prefer-query-by-disappearance`](./docs/rules/prefer-query-by-disappearance.md) | Suggest using `queryBy*` queries when waiting for disappearance | | ![dom-badge][] ![angular-badge][] ![react-badge][] ![vue-badge][] |
213213
| [`testing-library/prefer-screen-queries`](./docs/rules/prefer-screen-queries.md) | Suggest using `screen` while querying | | ![dom-badge][] ![angular-badge][] ![react-badge][] ![vue-badge][] |
214214
| [`testing-library/prefer-user-event`](./docs/rules/prefer-user-event.md) | Suggest using `userEvent` over `fireEvent` for simulating user interactions | | |
215215
| [`testing-library/prefer-wait-for`](./docs/rules/prefer-wait-for.md) | Use `waitFor` instead of deprecated wait methods | 🔧 | |

lib/configs/angular.ts

+1
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ export = {
2020
'testing-library/no-wait-for-snapshot': 'error',
2121
'testing-library/prefer-find-by': 'error',
2222
'testing-library/prefer-presence-queries': 'error',
23+
'testing-library/prefer-query-by-disappearance': 'error',
2324
'testing-library/prefer-screen-queries': 'error',
2425
'testing-library/render-result-naming-convention': 'error',
2526
},

lib/configs/dom.ts

+1
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ export = {
1515
'testing-library/no-wait-for-snapshot': 'error',
1616
'testing-library/prefer-find-by': 'error',
1717
'testing-library/prefer-presence-queries': 'error',
18+
'testing-library/prefer-query-by-disappearance': 'error',
1819
'testing-library/prefer-screen-queries': 'error',
1920
},
2021
};

lib/configs/react.ts

+1
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ export = {
2121
'testing-library/no-wait-for-snapshot': 'error',
2222
'testing-library/prefer-find-by': 'error',
2323
'testing-library/prefer-presence-queries': 'error',
24+
'testing-library/prefer-query-by-disappearance': 'error',
2425
'testing-library/prefer-screen-queries': 'error',
2526
'testing-library/render-result-naming-convention': 'error',
2627
},

lib/configs/vue.ts

+1
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ export = {
2121
'testing-library/no-wait-for-snapshot': 'error',
2222
'testing-library/prefer-find-by': 'error',
2323
'testing-library/prefer-presence-queries': 'error',
24+
'testing-library/prefer-query-by-disappearance': 'error',
2425
'testing-library/prefer-screen-queries': 'error',
2526
'testing-library/render-result-naming-convention': 'error',
2627
},

lib/rules/prefer-query-by-disappearance.ts

+4-4
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,10 @@ export default createTestingLibraryRule<Options, MessageIds>({
2525
'Suggest using `queryBy*` queries when waiting for disappearance',
2626
category: 'Possible Errors',
2727
recommendedConfig: {
28-
dom: false,
29-
angular: false,
30-
react: false,
31-
vue: false,
28+
dom: 'error',
29+
angular: 'error',
30+
react: 'error',
31+
vue: 'error',
3232
},
3333
},
3434
messages: {

tests/__snapshots__/index.test.ts.snap

+4
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ Object {
2525
"testing-library/no-wait-for-snapshot": "error",
2626
"testing-library/prefer-find-by": "error",
2727
"testing-library/prefer-presence-queries": "error",
28+
"testing-library/prefer-query-by-disappearance": "error",
2829
"testing-library/prefer-screen-queries": "error",
2930
"testing-library/render-result-naming-convention": "error",
3031
},
@@ -44,6 +45,7 @@ Object {
4445
"testing-library/no-wait-for-snapshot": "error",
4546
"testing-library/prefer-find-by": "error",
4647
"testing-library/prefer-presence-queries": "error",
48+
"testing-library/prefer-query-by-disappearance": "error",
4749
"testing-library/prefer-screen-queries": "error",
4850
},
4951
},
@@ -76,6 +78,7 @@ Object {
7678
"testing-library/no-wait-for-snapshot": "error",
7779
"testing-library/prefer-find-by": "error",
7880
"testing-library/prefer-presence-queries": "error",
81+
"testing-library/prefer-query-by-disappearance": "error",
7982
"testing-library/prefer-screen-queries": "error",
8083
"testing-library/render-result-naming-convention": "error",
8184
},
@@ -104,6 +107,7 @@ Object {
104107
"testing-library/no-wait-for-snapshot": "error",
105108
"testing-library/prefer-find-by": "error",
106109
"testing-library/prefer-presence-queries": "error",
110+
"testing-library/prefer-query-by-disappearance": "error",
107111
"testing-library/prefer-screen-queries": "error",
108112
"testing-library/render-result-naming-convention": "error",
109113
},

0 commit comments

Comments
 (0)