mock setTimeout and update tests to make sure use fake timers is imported as early as possible

Author: billyvg

packages/replay-internal/test/integration/coreHandlers/handleAfterSendEvent.test.ts

@@ -1,5 +1,8 @@
 import { vi } from 'vitest';
 import type { MockInstance } from 'vitest';
+import { useFakeTimers } from '../../utils/use-fake-timers';
+
+useFakeTimers();
 
 import { getClient } from '@sentry/core';
 import type { ErrorEvent, Event } from '@sentry/types';
@@ -10,9 +13,7 @@ import type { ReplayContainer } from '../../../src/replay';
 import { Error } from '../../fixtures/error';
 import { Transaction } from '../../fixtures/transaction';
 import { resetSdkMock } from '../../mocks/resetSdkMock';
-import { useFakeTimers } from '../../utils/use-fake-timers';
 
-useFakeTimers();
 let replay: ReplayContainer;
 
 describe('Integration | coreHandlers | handleAfterSendEvent', () => {

packages/replay-internal/test/integration/flush.test.ts

@@ -1,6 +1,10 @@
 import { vi } from 'vitest';
 import type { MockedFunction } from 'vitest';
 
+import { useFakeTimers } from '../utils/use-fake-timers';
+
+useFakeTimers();
+
 import * as SentryBrowserUtils from '@sentry-internal/browser-utils';
 import * as SentryUtils from '@sentry/utils';
 
@@ -14,9 +18,6 @@ import * as SendReplay from '../../src/util/sendReplay';
 import { BASE_TIMESTAMP, mockRrweb, mockSdk } from '../index';
 import type { DomHandler } from '../types';
 import { getTestEventCheckout } from '../utils/getTestEvent';
-import { useFakeTimers } from '../utils/use-fake-timers';
-
-useFakeTimers();
 
 type MockSendReplay = MockedFunction<any>;
 type MockAddPerformanceEntries = MockedFunction<ReplayContainer['_addPerformanceEntries']>;

packages/replay-internal/test/unit/coreHandlers/handleClick.test.ts

@@ -1,12 +1,12 @@
+import { useFakeTimers } from '../../utils/use-fake-timers';
+
+useFakeTimers();
+
 import type { Breadcrumb } from '@sentry/types';
 
 import { BASE_TIMESTAMP } from '../..';
 import { ClickDetector, ignoreElement } from '../../../src/coreHandlers/handleClick';
 import type { ReplayContainer } from '../../../src/types';
-import { useFakeTimers } from '../../utils/use-fake-timers';
-
-useFakeTimers();
-
 describe('Unit | coreHandlers | handleClick', () => {
   describe('ClickDetector', () => {
     beforeEach(() => {

packages/replay-internal/test/unit/coreHandlers/util/fetchUtils.test.ts

@@ -1,3 +1,9 @@
+import { useFakeTimers } from '../../../utils/use-fake-timers';
+
+useFakeTimers();
+
+import { vi } from 'vitest';
+
 import { _getResponseInfo } from '../../../../src/coreHandlers/util/fetchUtils';
 
 describe('Unit | coreHandlers | util | fetchUtils', () => {
@@ -112,15 +118,18 @@ describe('Unit | coreHandlers | util | fetchUtils', () => {
         text: () => new Promise(resolve => setTimeout(() => resolve('text body'), 1000)),
       } as unknown as Response;
 
-      const res = await _getResponseInfo(
-        true,
-        {
-          networkCaptureBodies: true,
-          networkResponseHeaders: [],
-        },
-        response,
-        undefined,
-      );
+      const [res] = await Promise.all([
+        _getResponseInfo(
+          true,
+          {
+            networkCaptureBodies: true,
+            networkResponseHeaders: [],
+          },
+          response,
+          undefined,
+        ),
+        vi.runAllTimersAsync(),
+      ]);
 
       expect(res).toEqual({
         _meta: { warnings: ['BODY_PARSE_ERROR'] },

packages/replay-internal/test/unit/coreHandlers/util/networkUtils.test.ts

@@ -1,5 +1,3 @@
-import { vi } from 'vitest';
-
 import { NETWORK_BODY_MAX_SIZE } from '../../../../src/constants';
 import {
   buildNetworkRequestOrResponse,

packages/replay-internal/test/unit/util/debounce.test.ts

@@ -1,8 +1,12 @@
-import { debounce } from '../../../src/util/debounce';
 import { useFakeTimers } from '../../utils/use-fake-timers';
 
+useFakeTimers();
+
+import { vi } from 'vitest';
+
+import { debounce } from '../../../src/util/debounce';
+
 describe('Unit | util | debounce', () => {
-  useFakeTimers();
   it('delay the execution of the passed callback function by the passed minDelay', () => {
     const callback = vi.fn();
     const debouncedCallback = debounce(callback, 100);

packages/replay-internal/test/utils/use-fake-timers.ts

@@ -1,5 +1,12 @@
 import { vi } from 'vitest';
 
+vi.mock('@sentry-internal/browser-utils', async () => ({
+  ...(await vi.importActual('@sentry-internal/browser-utils')),
+  setTimeout: (...args) => {
+    return setTimeout.call(global, ...args);
+  },
+}));
+
 export function useFakeTimers(): void {
   vi.useFakeTimers();
 }