From 1934bf224f9d45f3fc91cb722e31d3885aa9c7a0 Mon Sep 17 00:00:00 2001
From: Sebastian Silbermann <silbermann.sebastian@gmail.com>
Date: Wed, 15 Feb 2023 17:52:48 +0100
Subject: [PATCH] Bump kcd-scripts to 13.0.0 (#1170)

* Bump kcd-scripts to 13.0.0

* Resolve lint issues
---
 package.json                | 8 ++++++--
 src/__tests__/cleanup.js    | 3 ++-
 src/__tests__/debug.js      | 3 +--
 src/__tests__/new-act.js    | 8 ++++----
 src/__tests__/renderHook.js | 2 +-
 tests/setup-env.js          | 3 +++
 tests/toWarnDev.js          | 2 +-
 types/test.tsx              | 1 -
 8 files changed, 18 insertions(+), 12 deletions(-)

diff --git a/package.json b/package.json
index 9ee97f1d..b475390a 100644
--- a/package.json
+++ b/package.json
@@ -53,8 +53,8 @@
     "@testing-library/jest-dom": "^5.11.6",
     "chalk": "^4.1.2",
     "dotenv-cli": "^4.0.0",
-    "jest-diff": "^27.5.1",
-    "kcd-scripts": "^11.1.0",
+    "jest-diff": "^29.4.1",
+    "kcd-scripts": "^13.0.0",
     "npm-run-all": "^4.1.5",
     "react": "^18.0.0",
     "react-dom": "^18.0.0",
@@ -67,6 +67,9 @@
   },
   "eslintConfig": {
     "extends": "./node_modules/kcd-scripts/eslint.js",
+    "parserOptions": {
+      "ecmaVersion": 2022
+    },
     "globals": {
       "globalThis": "readonly"
     },
@@ -76,6 +79,7 @@
       "import/no-unassigned-import": "off",
       "import/named": "off",
       "testing-library/no-container": "off",
+      "testing-library/no-debugging-utils": "off",
       "testing-library/no-dom-import": "off",
       "testing-library/no-unnecessary-act": "off",
       "testing-library/prefer-user-event": "off"
diff --git a/src/__tests__/cleanup.js b/src/__tests__/cleanup.js
index 4517c098..9f17c722 100644
--- a/src/__tests__/cleanup.js
+++ b/src/__tests__/cleanup.js
@@ -64,7 +64,7 @@ describe('fake timers and missing act warnings', () => {
         let cancelled = false
         Promise.resolve().then(() => {
           microTaskSpy()
-          // eslint-disable-next-line jest/no-if -- false positive
+          // eslint-disable-next-line jest/no-if, jest/no-conditional-in-test -- false positive
           if (!cancelled) {
             setDeferredCounter(counter)
           }
@@ -96,6 +96,7 @@ describe('fake timers and missing act warnings', () => {
         let cancelled = false
         setTimeout(() => {
           deferredStateUpdateSpy()
+          // eslint-disable-next-line jest/no-conditional-in-test -- false-positive
           if (!cancelled) {
             setDeferredCounter(counter)
           }
diff --git a/src/__tests__/debug.js b/src/__tests__/debug.js
index f3aad595..c6a1d1fe 100644
--- a/src/__tests__/debug.js
+++ b/src/__tests__/debug.js
@@ -42,7 +42,7 @@ test('allows same arguments as prettyDOM', () => {
   debug(container, 6, {highlight: false})
   expect(console.log).toHaveBeenCalledTimes(1)
   expect(console.log.mock.calls[0]).toMatchInlineSnapshot(`
-    Array [
+    [
       <div>
     ...,
     ]
@@ -52,5 +52,4 @@ test('allows same arguments as prettyDOM', () => {
 /*
 eslint
   no-console: "off",
-  testing-library/no-debug: "off",
 */
diff --git a/src/__tests__/new-act.js b/src/__tests__/new-act.js
index 4909d4a6..0412a8a3 100644
--- a/src/__tests__/new-act.js
+++ b/src/__tests__/new-act.js
@@ -47,8 +47,8 @@ test('async act recovers from errors', async () => {
   }
   expect(console.error).toHaveBeenCalledTimes(1)
   expect(console.error.mock.calls).toMatchInlineSnapshot(`
-    Array [
-      Array [
+    [
+      [
         call console.error,
       ],
     ]
@@ -65,8 +65,8 @@ test('async act recovers from sync errors', async () => {
   }
   expect(console.error).toHaveBeenCalledTimes(1)
   expect(console.error.mock.calls).toMatchInlineSnapshot(`
-    Array [
-      Array [
+    [
+      [
         call console.error,
       ],
     ]
diff --git a/src/__tests__/renderHook.js b/src/__tests__/renderHook.js
index f6b7a343..11b7009a 100644
--- a/src/__tests__/renderHook.js
+++ b/src/__tests__/renderHook.js
@@ -21,7 +21,7 @@ test('allows rerendering', () => {
       const [left, setLeft] = React.useState('left')
       const [right, setRight] = React.useState('right')
 
-      // eslint-disable-next-line jest/no-if
+      // eslint-disable-next-line jest/no-if, jest/no-conditional-in-test -- false-positive
       switch (branch) {
         case 'left':
           return [left, setLeft]
diff --git a/tests/setup-env.js b/tests/setup-env.js
index a4ddfa17..c9b976f5 100644
--- a/tests/setup-env.js
+++ b/tests/setup-env.js
@@ -1,2 +1,5 @@
 import '@testing-library/jest-dom/extend-expect'
 import './failOnUnexpectedConsoleCalls'
+import {TextEncoder} from 'util'
+
+global.TextEncoder = TextEncoder
diff --git a/tests/toWarnDev.js b/tests/toWarnDev.js
index ac5f1b19..ca58346f 100644
--- a/tests/toWarnDev.js
+++ b/tests/toWarnDev.js
@@ -24,7 +24,7 @@ SOFTWARE.
  */
 /* eslint-disable no-unsafe-finally */
 /* eslint-disable no-negated-condition */
-/* eslint-disable @babel/no-invalid-this */
+/* eslint-disable no-invalid-this */
 /* eslint-disable prefer-template */
 /* eslint-disable func-names */
 /* eslint-disable complexity */
diff --git a/types/test.tsx b/types/test.tsx
index 17ba7012..c33f07b6 100644
--- a/types/test.tsx
+++ b/types/test.tsx
@@ -188,7 +188,6 @@ export function testRenderHookProps() {
 eslint
   testing-library/prefer-explicit-assert: "off",
   testing-library/no-wait-for-empty-callback: "off",
-  testing-library/no-debug: "off",
   testing-library/prefer-screen-queries: "off"
 */