chore(tests): eliminate rtl patch (#2878)

* chore(tests): eliminate rtl patch

* fix workflow files

* consistent workflow files

* remove unused Promise.resolve

* rerender in tests

.github/workflows/test-multiple-builds.yml

@@ -26,7 +26,6 @@ jobs:
       - name: Use React 17 for production test
         if: ${{ matrix.env == 'production' }}
         run: |
-          pnpm json -I -f package.json -e "this.pnpm.patchedDependencies={};"
           pnpm add -D [email protected] [email protected] @testing-library/[email protected]
       - name: Patch for DEV-ONLY
         if: ${{ matrix.env == 'development' }}

.github/workflows/test-multiple-versions.yml

@@ -46,7 +46,6 @@ jobs:
       - name: Install legacy testing-library
         if: ${{ startsWith(matrix.react, '16.') || startsWith(matrix.react, '17.') }}
         run: |
-          pnpm json -I -f package.json -e "this.pnpm.patchedDependencies={};"
           pnpm add -D @testing-library/[email protected]
       - name: Patch for React 16
         if: ${{ startsWith(matrix.react, '16.') }}

.github/workflows/test-old-typescript.yml

@@ -90,7 +90,6 @@ jobs:
       - name: Patch testing setup for older TS
         if: ${{ matrix.typescript == '4.0.5' || startsWith(matrix.typescript, '3.') }}
         run: |
-          pnpm json -I -f package.json -e "this.pnpm.patchedDependencies={};"
           pnpm add -D @testing-library/[email protected] @testing-library/[email protected] @types/[email protected]
           rm node_modules/vitest/dist/*.d.ts
           echo "declare module 'vitest'" >> ./src/types.d.ts

package.json

@@ -158,8 +158,8 @@
     "jsdom": "^25.0.1",
     "json": "^11.0.0",
     "prettier": "^3.4.2",
-    "react": "19.0.0",
-    "react-dom": "19.0.0",
+    "react": "^19.0.0",
+    "react-dom": "^19.0.0",
     "rollup": "^4.28.1",
     "rollup-plugin-banner2": "^1.3.1",
     "rollup-plugin-esbuild": "^6.1.1",
@@ -182,10 +182,5 @@
     "react": {
       "optional": true
     }
-  },
-  "pnpm": {
-    "patchedDependencies": {
-      "@testing-library/[email protected]": "patches/@[email protected]"
-    }
   }
 }

tests/react/abortable.test.tsx

@@ -38,14 +38,17 @@ describe('abortable atom test', () => {
       )
     }
 
-    render(
-      <StrictMode>
-        <Suspense fallback="loading">
-          <Component />
-          <Controls />
-        </Suspense>
-      </StrictMode>,
-    )
+    // eslint-disable-next-line testing-library/no-unnecessary-act
+    await act(async () => {
+      render(
+        <StrictMode>
+          <Suspense fallback="loading">
+            <Component />
+            <Controls />
+          </Suspense>
+        </StrictMode>,
+      )
+    })
 
     await screen.findByText('loading')
 
@@ -95,14 +98,17 @@ describe('abortable atom test', () => {
       )
     }
 
-    render(
-      <StrictMode>
-        <Suspense fallback="loading">
-          <Component />
-          <Controls />
-        </Suspense>
-      </StrictMode>,
-    )
+    // eslint-disable-next-line testing-library/no-unnecessary-act
+    await act(async () => {
+      render(
+        <StrictMode>
+          <Suspense fallback="loading">
+            <Component />
+            <Controls />
+          </Suspense>
+        </StrictMode>,
+      )
+    })
 
     await screen.findByText('loading')
     resolve.splice(0).forEach((fn) => fn())
@@ -154,13 +160,16 @@ describe('abortable atom test', () => {
       )
     }
 
-    render(
-      <StrictMode>
-        <Suspense fallback="loading">
-          <Parent />
-        </Suspense>
-      </StrictMode>,
-    )
+    // eslint-disable-next-line testing-library/no-unnecessary-act
+    await act(async () => {
+      render(
+        <StrictMode>
+          <Suspense fallback="loading">
+            <Parent />
+          </Suspense>
+        </StrictMode>,
+      )
+    })
 
     await screen.findByText('loading')
 
@@ -203,14 +212,17 @@ describe('abortable atom test', () => {
       )
     }
 
-    render(
-      <StrictMode>
-        <Suspense fallback="loading">
-          <Component />
-          <Controls />
-        </Suspense>
-      </StrictMode>,
-    )
+    // eslint-disable-next-line testing-library/no-unnecessary-act
+    await act(async () => {
+      render(
+        <StrictMode>
+          <Suspense fallback="loading">
+            <Component />
+            <Controls />
+          </Suspense>
+        </StrictMode>,
+      )
+    })
 
     await screen.findByText('loading')