repro probable bug?

Author: silesky

packages/core/src/analytics/__tests__/dispatch.test.ts

@@ -99,30 +99,28 @@ describe('Dispatch', () => {
       expect.anything()
     )
   })
-  // TODO: Weird and inconsistent behavior
-  it('should have bizarre and inconsistent timeout behavior where the delay is different based on whether timeout is explicitly set to 1000 or not', async () => {
+  // TODO: Inconsistent behavior? This seems like a bug.
+  it('should have inconsistent timeout behavior where the delay is different based on whether timeout is explicitly set to 1000 or not', async () => {
     {
       const TIMEOUT = 1000
       await dispatch({ type: 'screen' }, queue, emitter, {
         callback: jest.fn(),
         timeout: TIMEOUT,
       })
       const [, , delay] = invokeCallback.mock.calls[0]
-      // timeout = 1000, delay equals 998
-      expect(delay).toBeGreaterThan(990)
-      expect(delay).toBeLessThanOrEqual(1000)
+      expect(delay).toBeGreaterThan(990) // ???
+      expect(delay).toBeLessThanOrEqual(1000) // ???
     }
     {
-      // wtf
       invokeCallback.mockReset()
       const TIMEOUT = undefined // this defaults to 1000 in the invokeCallback function
       await dispatch({ type: 'screen' }, queue, emitter, {
         callback: jest.fn(),
         timeout: TIMEOUT,
       })
       const [, , delay] = invokeCallback.mock.calls[0]
-      expect(delay).toBeGreaterThan(290)
-      expect(delay).toBeLessThanOrEqual(300)
+      expect(delay).toBeGreaterThan(290) // ???
+      expect(delay).toBeLessThanOrEqual(300) // ???
     }
   })
 })