feat: support ESLint 8.x

BREAKING CHANGE: Requires ESLint@^8.0.0
testing-library · Oct 13, 2021 · c8d3a53 · c8d3a53
1 parent cf78530
commit c8d3a53
Show file tree

Hide file tree

Showing 29 changed files with 7 additions and 34 deletions.
diff --git a/.github/workflows/pipeline.yml b/.github/workflows/pipeline.yml
@@ -52,7 +52,7 @@ jobs:
     strategy:
       matrix:
         node: [12.22.0, 12, 14.17.0, 14, '16.0', 16]
-        eslint: [7.5, 7]
+        eslint: [8]
 
     steps:
       - name: Cancel Previous Runs

diff --git a/lib/rules/await-async-query.ts b/lib/rules/await-async-query.ts
@@ -20,7 +20,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: 'Enforce promises from async queries to be handled',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: 'error',
         angular: 'error',

diff --git a/lib/rules/await-async-utils.ts b/lib/rules/await-async-utils.ts
@@ -19,7 +19,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: 'Enforce promises from async utils to be awaited properly',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: 'error',
         angular: 'error',

diff --git a/lib/rules/await-fire-event.ts b/lib/rules/await-fire-event.ts
@@ -19,7 +19,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: 'Enforce promises from `fireEvent` methods to be handled',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/consistent-data-testid.ts b/lib/rules/consistent-data-testid.ts
@@ -18,7 +18,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'suggestion',
     docs: {
       description: 'Ensures consistent usage of `data-testid`',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/no-await-sync-events.ts b/lib/rules/no-await-sync-events.ts
@@ -21,7 +21,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: 'Disallow unnecessary `await` for sync events',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/no-await-sync-query.ts b/lib/rules/no-await-sync-query.ts
@@ -13,7 +13,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: 'Disallow unnecessary `await` for sync queries',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: 'error',
         angular: 'error',

diff --git a/lib/rules/no-container.ts b/lib/rules/no-container.ts
@@ -20,7 +20,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: 'Disallow the use of `container` methods',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: 'error',

diff --git a/lib/rules/no-debugging-utils.ts b/lib/rules/no-debugging-utils.ts
@@ -31,7 +31,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: 'Disallow the use of debugging utilities like `debug`',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: 'error',

diff --git a/lib/rules/no-dom-import.ts b/lib/rules/no-dom-import.ts
@@ -18,7 +18,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: 'Disallow importing from DOM Testing Library',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: ['error', 'angular'],

diff --git a/lib/rules/no-manual-cleanup.ts b/lib/rules/no-manual-cleanup.ts
@@ -29,7 +29,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: 'Disallow the use of `cleanup`',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/no-node-access.ts b/lib/rules/no-node-access.ts
@@ -13,7 +13,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: 'Disallow direct Node access',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: 'error',

diff --git a/lib/rules/no-promise-in-fire-event.ts b/lib/rules/no-promise-in-fire-event.ts
@@ -20,7 +20,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     docs: {
       description:
         'Disallow the use of promises passed to a `fireEvent` method',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: 'error',
         angular: 'error',

diff --git a/lib/rules/no-render-in-setup.ts b/lib/rules/no-render-in-setup.ts
@@ -50,7 +50,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     docs: {
       description:
         'Disallow the use of `render` in testing frameworks setup functions',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/no-unnecessary-act.ts b/lib/rules/no-unnecessary-act.ts
@@ -23,7 +23,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     docs: {
       description:
         'Disallow wrapping Testing Library utils or empty callbacks in `act`',
-      category: 'Possible Errors',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/no-wait-for-empty-callback.ts b/lib/rules/no-wait-for-empty-callback.ts
@@ -18,7 +18,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     docs: {
       description:
         'Disallow empty callbacks for `waitFor` and `waitForElementToBeRemoved`',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: 'error',
         angular: 'error',

diff --git a/lib/rules/no-wait-for-multiple-assertions.ts b/lib/rules/no-wait-for-multiple-assertions.ts
@@ -17,7 +17,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     docs: {
       description:
         'Disallow the use of multiple `expect` calls inside `waitFor`',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/no-wait-for-side-effects.ts b/lib/rules/no-wait-for-side-effects.ts
@@ -20,7 +20,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'suggestion',
     docs: {
       description: 'Disallow the use of side effects in `waitFor`',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/no-wait-for-snapshot.ts b/lib/rules/no-wait-for-snapshot.ts
@@ -19,7 +19,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     docs: {
       description:
         'Ensures no snapshot is generated inside of a `waitFor` call',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/prefer-explicit-assert.ts b/lib/rules/prefer-explicit-assert.ts
@@ -69,7 +69,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     docs: {
       description:
         'Suggest using explicit assertions rather than standalone queries',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/prefer-find-by.ts b/lib/rules/prefer-find-by.ts
@@ -56,7 +56,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     docs: {
       description:
         'Suggest using `find(All)By*` query instead of `waitFor` + `get(All)By*` to wait for elements',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: 'error',
         angular: 'error',

diff --git a/lib/rules/prefer-presence-queries.ts b/lib/rules/prefer-presence-queries.ts
@@ -11,7 +11,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
   name: RULE_NAME,
   meta: {
     docs: {
-      category: 'Best Practices',
       description:
         'Ensure appropriate `get*`/`query*` queries are used with their respective matchers',
       recommendedConfig: {

diff --git a/lib/rules/prefer-query-by-disappearance.ts b/lib/rules/prefer-query-by-disappearance.ts
@@ -23,7 +23,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     docs: {
       description:
         'Suggest using `queryBy*` queries when waiting for disappearance',
-      category: 'Possible Errors',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/prefer-screen-queries.ts b/lib/rules/prefer-screen-queries.ts
@@ -40,7 +40,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'suggestion',
     docs: {
       description: 'Suggest using `screen` while querying',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: 'error',
         angular: 'error',

diff --git a/lib/rules/prefer-user-event.ts b/lib/rules/prefer-user-event.ts
@@ -71,7 +71,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     docs: {
       description:
         'Suggest using `userEvent` over `fireEvent` for simulating user interactions',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/prefer-wait-for.ts b/lib/rules/prefer-wait-for.ts
@@ -26,7 +26,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'suggestion',
     docs: {
       description: 'Use `waitFor` instead of deprecated wait methods',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: false,

diff --git a/lib/rules/render-result-naming-convention.ts b/lib/rules/render-result-naming-convention.ts
@@ -24,7 +24,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'suggestion',
     docs: {
       description: 'Enforce a valid naming for return value from `render`',
-      category: 'Best Practices',
       recommendedConfig: {
         dom: false,
         angular: 'error',

diff --git a/package.json b/package.json
@@ -42,20 +42,20 @@
     "prepare": "is-ci || husky install"
   },
   "dependencies": {
-    "@typescript-eslint/experimental-utils": "^4.30.0"
+    "@typescript-eslint/experimental-utils": "^5.0.0"
   },
   "devDependencies": {
     "@commitlint/cli": "^13.1.0",
     "@commitlint/config-conventional": "^13.1.0",
     "@types/jest": "^27.0.1",
     "@types/node": "^12.20.23",
-    "@typescript-eslint/eslint-plugin": "^4.30.0",
-    "@typescript-eslint/parser": "^4.30.0",
+    "@typescript-eslint/eslint-plugin": "^5.0.0",
+    "@typescript-eslint/parser": "^5.0.0",
     "cpy-cli": "^3.1.1",
-    "eslint": "^7.32.0",
+    "eslint": "^8.0.0",
     "eslint-config-kentcdodds": "^19.1.1",
     "eslint-config-prettier": "^8.3.0",
-    "eslint-plugin-import": "^2.24.2",
+    "eslint-plugin-import": "^2.25.2",
     "eslint-plugin-jest": "^24.4.0",
     "eslint-plugin-jest-formatting": "^3.0.0",
     "eslint-plugin-node": "^11.1.0",
@@ -72,7 +72,7 @@
     "typescript": "^4.4.2"
   },
   "peerDependencies": {
-    "eslint": "^7.5.0"
+    "eslint": "^8.0.0"
   },
   "engines": {
     "node": "^12.22.0 || ^14.17.0 || >=16.0.0",

diff --git a/tests/fake-rule.ts b/tests/fake-rule.ts
@@ -26,7 +26,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     type: 'problem',
     docs: {
       description: 'Fake rule to test rule maker and detection helpers',
-      category: 'Possible Errors',
       recommendedConfig: {
         dom: false,
         angular: false,