replay test changes

Author: mydea

packages/replay-internal/test.setup.ts

@@ -1,5 +1,6 @@
-import type { ReplayRecordingData, Transport } from '@sentry/core';
 /* eslint-disable @typescript-eslint/no-unsafe-member-access */
+
+import type { ReplayRecordingData, Transport } from '@sentry/core';
 import { getClient } from '@sentry/core';
 import * as SentryUtils from '@sentry/core';
 import { printDiffOrStringify } from 'jest-matcher-utils';

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

@@ -2,15 +2,13 @@
  * @vitest-environment jsdom
  */
 
+import '../utils/use-fake-timers';
 import { EventType } from '@sentry-internal/rrweb';
-import { afterEach, describe, expect, test, vi } from 'vitest';
+import { afterEach, beforeAll, describe, expect, test, vi } from 'vitest';
 import { saveSession } from '../../src/session/saveSession';
 import type { RecordingEvent } from '../../src/types';
 import { addEvent } from '../../src/util/addEvent';
 import { resetSdkMock } from '../mocks/resetSdkMock';
-import { useFakeTimers } from '../utils/use-fake-timers';
-
-useFakeTimers();
 
 vi.mock('../../src/session/saveSession', () => {
   return {
@@ -19,6 +17,9 @@ vi.mock('../../src/session/saveSession', () => {
 });
 
 describe('Integration | autoSaveSession', () => {
+  beforeAll(() => {
+    vi.useFakeTimers();
+  });
   afterEach(() => {
     vi.clearAllMocks();
   });

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

@@ -2,6 +2,7 @@
  * @vitest-environment jsdom
  */
 
+import '../utils/use-fake-timers';
 import type { Transport } from '@sentry/core';
 import * as SentryCore from '@sentry/core';
 import * as SentryBrowserUtils from '@sentry-internal/browser-utils';
@@ -15,9 +16,6 @@ import { createPerformanceSpans } from '../../src/util/createPerformanceSpans';
 import * as SendReplayRequest from '../../src/util/sendReplayRequest';
 import { BASE_TIMESTAMP, mockRrweb, mockSdk } from '../index';
 import type { DomHandler } from '../types';
-import { useFakeTimers } from '../utils/use-fake-timers';
-
-useFakeTimers();
 
 type MockTransportSend = MockedFunction<Transport['send']>;
 
@@ -30,6 +28,7 @@ describe('Integration | beforeAddRecordingEvent', () => {
   const { record: mockRecord } = mockRrweb();
 
   beforeAll(async () => {
+    vi.useFakeTimers();
     vi.setSystemTime(new Date(BASE_TIMESTAMP));
     vi.spyOn(SentryBrowserUtils, 'addClickKeypressInstrumentationHandler').mockImplementation(handler => {
       domHandler = handler;

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

@@ -2,23 +2,25 @@
  * @vitest-environment jsdom
  */
 
+import '../../utils/use-fake-timers';
 import type { ErrorEvent, Event } from '@sentry/core';
 import { getClient } from '@sentry/core';
 import type { MockInstance } from 'vitest';
-import { afterEach, describe, expect, it, vi } from 'vitest';
+import { afterEach, beforeAll, describe, expect, it, vi } from 'vitest';
 import { UNABLE_TO_SEND_REPLAY } from '../../../src/constants';
 import { handleAfterSendEvent } from '../../../src/coreHandlers/handleAfterSendEvent';
 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', () => {
+  beforeAll(() => {
+    vi.useFakeTimers();
+  });
+
   afterEach(() => {
     replay.stop();
   });

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

@@ -2,17 +2,19 @@
  * @vitest-environment jsdom
  */
 
-import { afterEach, describe, expect, it, vi } from 'vitest';
+import { afterEach, beforeAll, describe, expect, it, vi } from 'vitest';
 import { handleBeforeSendEvent } from '../../../src/coreHandlers/handleBeforeSendEvent';
 import type { ReplayContainer } from '../../../src/replay';
 import { Error } from '../../fixtures/error';
 import { resetSdkMock } from '../../mocks/resetSdkMock';
-import { useFakeTimers } from '../../utils/use-fake-timers';
 
-useFakeTimers();
 let replay: ReplayContainer;
 
 describe('Integration | coreHandlers | handleBeforeSendEvent', () => {
+  beforeAll(() => {
+    vi.useFakeTimers();
+  });
+
   afterEach(() => {
     replay.stop();
   });

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

@@ -2,9 +2,10 @@
  * @vitest-environment jsdom
  */
 
+import '../../utils/use-fake-timers';
 import type { Event } from '@sentry/core';
 import { getClient } from '@sentry/core';
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from 'vitest';
 import { REPLAY_EVENT_NAME, SESSION_IDLE_EXPIRE_DURATION } from '../../../src/constants';
 import { handleGlobalEventListener } from '../../../src/coreHandlers/handleGlobalEvent';
 import type { ReplayContainer } from '../../../src/replay';
@@ -13,12 +14,14 @@ import * as resetReplayIdOnDynamicSamplingContextModule from '../../../src/util/
 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 | handleGlobalEvent', () => {
+  beforeAll(() => {
+    vi.useFakeTimers();
+  });
+
   beforeEach(async () => {
     ({ replay } = await resetSdkMock({
       replayOptions: {

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

@@ -1,15 +1,16 @@
 /**
  * @vitest-environment jsdom
  */
-
-import { beforeEach, describe, expect, it, vi } from 'vitest';
+import '../utils/use-fake-timers';
+import { beforeAll, beforeEach, describe, expect, it, vi } from 'vitest';
 import { BASE_TIMESTAMP } from '..';
 import { resetSdkMock } from '../mocks/resetSdkMock';
-import { useFakeTimers } from '../utils/use-fake-timers';
-
-useFakeTimers();
 
 describe('Integration | early events', () => {
+  beforeAll(() => {
+    vi.useFakeTimers();
+  });
+
   beforeEach(() => {
     vi.clearAllMocks();
   });

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

@@ -2,8 +2,9 @@
  * @vitest-environment jsdom
  */
 
+import '../utils/use-fake-timers';
 import { captureException, getClient } from '@sentry/core';
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from 'vitest';
 import {
   BUFFER_CHECKOUT_TIME,
   DEFAULT_FLUSH_MIN_DELAY,
@@ -22,9 +23,6 @@ import { BASE_TIMESTAMP } from '../index';
 import { resetSdkMock } from '../mocks/resetSdkMock';
 import type { DomHandler } from '../types';
 import { getTestEventCheckout, getTestEventIncremental } from '../utils/getTestEvent';
-import { useFakeTimers } from '../utils/use-fake-timers';
-
-useFakeTimers();
 
 async function advanceTimers(time: number) {
   vi.advanceTimersByTime(time);
@@ -41,6 +39,10 @@ describe('Integration | errorSampleRate', () => {
     let mockRecord: RecordMock;
     let domHandler: DomHandler;
 
+    beforeAll(() => {
+      vi.useFakeTimers();
+    });
+
     beforeEach(async () => {
       ({ mockRecord, domHandler, replay } = await resetSdkMock({
         replayOptions: {

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

@@ -2,23 +2,24 @@
  * @vitest-environment jsdom
  */
 
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import '../utils/use-fake-timers';
+import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from 'vitest';
 import { WINDOW } from '../../src/constants';
 import type { Replay } from '../../src/integration';
 import type { ReplayContainer } from '../../src/replay';
 import { addEvent } from '../../src/util/addEvent';
-// mock functions need to be imported first
 import { BASE_TIMESTAMP, mockSdk } from '../index';
 import { getTestEventCheckout, getTestEventIncremental } from '../utils/getTestEvent';
-import { useFakeTimers } from '../utils/use-fake-timers';
-
-useFakeTimers();
 
 describe('Integration | eventBuffer | Event Buffer Max Size', () => {
   let replay: ReplayContainer;
   let integration: Replay;
   const prevLocation = WINDOW.location;
 
+  beforeAll(() => {
+    vi.useFakeTimers();
+  });
+
   beforeEach(async () => {
     vi.setSystemTime(new Date(BASE_TIMESTAMP));
 

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

@@ -2,17 +2,19 @@
  * @vitest-environment jsdom
  */
 
+import '../utils/use-fake-timers';
 import type { Event } from '@sentry/core';
 import { getClient, getCurrentScope } from '@sentry/core';
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
+import { afterEach, beforeAll, beforeEach, describe, expect, it, vi } from 'vitest';
 import { BASE_TIMESTAMP } from '..';
 import { resetSdkMock } from '../mocks/resetSdkMock';
 import { getTestEventIncremental } from '../utils/getTestEvent';
-import { useFakeTimers } from '../utils/use-fake-timers';
-
-useFakeTimers();
 
 describe('Integration | eventProcessors', () => {
+  beforeAll(() => {
+    vi.useFakeTimers();
+  });
+
   beforeEach(() => {
     getCurrentScope().clear();
     getCurrentScope().setClient(undefined);