fix wait-for

Author: eps1lon

src/wait-for.js

@@ -9,6 +9,7 @@ import {
   // an environment that's faked the timers out.
   checkContainerType,
 } from './dtlHelpers'
+import act, {asyncAct} from './act-compat'
 
 // Not supported for external libraries. Only supported internally in @testing-library/dom
 function runWithExpensiveErrorDiagnosticsDisabled(callback) {
@@ -77,7 +78,9 @@ function waitFor(
         // third party code that's setting up recursive timers so rapidly that
         // the user's timer's don't get a chance to resolve. So we'll advance
         // by an interval instead. (We have a test for this case).
-        jest.advanceTimersByTime(interval)
+        act(() => {
+          jest.advanceTimersByTime(interval)
+        })
 
         // It's really important that checkCallback is run *before* we flush
         // in-flight promises. To be honest, I'm not sure why, and I can't quite
@@ -90,9 +93,11 @@ function waitFor(
         // of parallelization so we're fine.
         // https://stackoverflow.com/a/59243586/971592
         // eslint-disable-next-line no-await-in-loop
-        await new Promise(r => {
-          setTimeout(r, 0)
-          jest.advanceTimersByTime(0)
+        await asyncAct(async () => {
+          await new Promise(r => {
+            setTimeout(r, 0)
+            jest.advanceTimersByTime(0)
+          })
         })
       }
     } else {