Disable interval checking by default

Author: mpeyper

docs/api-reference.md

@@ -181,7 +181,8 @@ in the callback to perform assertion or to test values.
 #### `interval`
 
 The amount of time in milliseconds (ms) to wait between checks of the callback if no renders occur.
-By default, an interval of 50ms is used.
+Interval checking is disabled if `interval` is not provided in the options or provided as a `falsy`
+value. By default, it is disabled.
 
 #### `timeout`
 
@@ -210,7 +211,8 @@ comparison.
 #### `interval`
 
 The amount of time in milliseconds (ms) to wait between checks of the callback if no renders occur.
-By default, an interval of 50ms is used.
+Interval checking is disabled if `interval` is not provided in the options or provided as a `falsy`
+value. By default, it is disabled.
 
 #### `timeout`
 

src/asyncUtils.js

@@ -38,7 +38,7 @@ function asyncUtils(addResolver) {
     await nextUpdatePromise
   }
 
-  const waitFor = async (callback, { interval = 50, timeout, suppressErrors = true } = {}) => {
+  const waitFor = async (callback, { interval, timeout, suppressErrors = true } = {}) => {
     const checkResult = () => {
       try {
         const callbackResult = callback()
@@ -55,7 +55,12 @@ function asyncUtils(addResolver) {
       while (true) {
         const startTime = Date.now()
         try {
-          await Promise.race([waitForNextUpdate({ timeout }), resolveAfter(interval)])
+          const nextCheck = interval
+            ? Promise.race([waitForNextUpdate({ timeout }), resolveAfter(interval)])
+            : waitForNextUpdate({ timeout })
+
+          await nextCheck
+
           if (checkResult()) {
             return
           }

test/asyncHook.test.js

@@ -280,28 +280,6 @@ describe('async hook tests', () => {
     expect(complete).toBe(true)
   })
 
-  test('should wait for arbitrary expectation to pass (deprecated)', async () => {
-    const { wait } = renderHook(() => null)
-
-    let actual = 0
-    let expected = 1
-
-    setTimeout(() => {
-      actual = expected
-    }, 200)
-
-    let complete = false
-    await wait(
-      () => {
-        expect(actual).toBe(expected)
-        complete = true
-      },
-      { interval: 100 }
-    )
-
-    expect(complete).toBe(true)
-  })
-
   test('should not hang if expectation is already passing (deprecated)', async () => {
     const { result, wait } = renderHook(() => useSequence('first', 'second'))