feat(waitFor): improve error message for non-function callbacks

Author: Belco90

src/__tests__/wait-for.js

@@ -32,3 +32,9 @@ test('uses generic error if there was no last error', async () => {
   ).catch(e => e)
   expect(result).toMatchInlineSnapshot(`[Error: Timed out in waitFor.]`)
 })
+
+test('throws nice error if provided callback is not a function', () => {
+  expect(() => waitFor('not a function')).toThrow(
+    'Received `callback` arg must be a function',
+  )
+})

src/wait-for.js

@@ -22,6 +22,10 @@ function waitFor(
     },
   } = {},
 ) {
+  if (typeof callback !== 'function') {
+    throw new Error('Received `callback` arg must be a function')
+  }
+
   // created here so we get a nice stacktrace
   const timedOutError = new Error('Timed out in waitFor.')
   if (interval < 1) interval = 1