Fixes fake timers jest issue after getsentry/sentry-javascript#16890

Author: antonis

packages/core/test/profiling/integration.test.ts

@@ -42,7 +42,10 @@ describe('profiling integration', () => {
         type: 'sourcemap',
       },
     ]);
-    jest.useFakeTimers();
+    jest.useFakeTimers({
+      advanceTimers: true,
+      doNotFake: ['performance'], // Keep real performance API
+    });
   });
 
   afterEach(async () => {

packages/core/test/tracing/gesturetracing.test.ts

@@ -54,7 +54,10 @@ describe('GestureTracing', () => {
 
     beforeEach(() => {
       jest.clearAllMocks();
-      jest.useFakeTimers();
+      jest.useFakeTimers({
+        advanceTimers: true,
+        doNotFake: ['performance'], // Keep real performance API
+      });
       client = setupTestClient({
         enableUserInteractionTracing: true,
       });

packages/core/test/tracing/integrations/nativeframes.test.ts

@@ -16,7 +16,10 @@ jest.mock('../../../src/js/wrapper', () => {
   };
 });
 
-jest.useFakeTimers({ advanceTimers: true });
+jest.useFakeTimers({
+  advanceTimers: true,
+  doNotFake: ['performance'], // Keep real performance API
+});
 
 const mockDate = new Date(2024, 7, 14); // Set your desired mock date here
 const originalDateNow = Date.now; // Store the original Date.now function

packages/core/test/tracing/integrations/stallTracking/stalltracking.background.test.ts

@@ -20,7 +20,10 @@ describe('BackgroundEventListener', () => {
     stallTracking['_internalState']['lastIntervalMs'] = currentTime;
     stallTracking['_internalState']['timeout'] = setTimeout(() => {}, LOOP_TIMEOUT_INTERVAL_MS); // Create a fake timeout to simulate a running interval
     stallTracking['_internalState']['isBackground'] = true;
-    jest.useFakeTimers(); // Enable fake timers to control timeouts
+    jest.useFakeTimers({
+      advanceTimers: true,
+      doNotFake: ['performance'], // Keep real performance API
+    });
     stallTracking['_internalState']['backgroundEventListener']('active' as AppStateStatus);
     // Check if _isBackground is set to false and _lastIntervalMs is updated correctly
     expect(stallTracking['_internalState']['isBackground']).toBe(false);
@@ -39,7 +42,10 @@ describe('BackgroundEventListener', () => {
     stallTracking['_internalState']['timeout'] = null;
     // Mock _iteration
     stallTracking['_internalState']['iteration'] = jest.fn();
-    jest.useFakeTimers(); // Enable fake timers to control timeouts
+    jest.useFakeTimers({
+      advanceTimers: true,
+      doNotFake: ['performance'], // Keep real performance API
+    });
     stallTracking['_internalState']['backgroundEventListener']('active' as AppStateStatus);
 
     expect(stallTracking['_internalState']['iteration']).not.toBeCalled();
@@ -49,7 +55,10 @@ describe('BackgroundEventListener', () => {
     stallTracking['_internalState']['timeout'] = setTimeout(() => {}, 500);
     // Mock _iteration
     stallTracking['_internalState']['iteration'] = jest.fn(); // Create a fake timeout to simulate a running interval
-    jest.useFakeTimers(); // Enable fake timers to control timeouts
+    jest.useFakeTimers({
+      advanceTimers: true,
+      doNotFake: ['performance'], // Keep real performance API
+    });
     stallTracking['_internalState']['backgroundEventListener']('active' as AppStateStatus);
     expect(stallTracking['_internalState']['iteration']).toBeCalled();
   });

packages/core/test/tracing/integrations/stallTracking/stalltracking.iteration.test.ts

@@ -19,7 +19,10 @@ describe('Iteration', () => {
     stallTracking['isTracking'] = false;
     stallTracking['_internalState']['isBackground'] = false;
     stallTracking['_internalState']['lastIntervalMs'] = Date.now() - 1000; // Force a timeout
-    jest.useFakeTimers();
+    jest.useFakeTimers({
+      advanceTimers: true,
+      doNotFake: ['performance'], // Keep real performance API
+    });
     // Invokes the private _interaction function.
     stallTracking['_internalState']['iteration']();
     expect(stallTracking['_internalState']['timeout']).toBeNull();
@@ -29,7 +32,10 @@ describe('Iteration', () => {
     stallTracking['isTracking'] = true;
     stallTracking['_internalState']['isBackground'] = true;
     stallTracking['_internalState']['lastIntervalMs'] = Date.now() - 1000; // Force a timeout
-    jest.useFakeTimers();
+    jest.useFakeTimers({
+      advanceTimers: true,
+      doNotFake: ['performance'], // Keep real performance API
+    });
     // Invokes the private _interaction function.
     stallTracking['_internalState']['iteration']();
     expect(stallTracking['_internalState']['timeout']).toBeNull();
@@ -38,7 +44,10 @@ describe('Iteration', () => {
     const stallTracking = stallTrackingIntegration() as StallTrackingWithTestProperties;
     stallTracking['isTracking'] = true;
     stallTracking['_internalState']['isBackground'] = false;
-    jest.useFakeTimers();
+    jest.useFakeTimers({
+      advanceTimers: true,
+      doNotFake: ['performance'], // Keep real performance API
+    });
     stallTracking['_internalState']['lastIntervalMs'] = Date.now(); // Force a timeout
     // Invokes the private _interaction function.
     stallTracking['_internalState']['iteration']();
@@ -50,7 +59,10 @@ describe('Iteration', () => {
     const _minimumStallThreshold = 100;
     // Call _iteration with totalTimeTaken >= LOOP_TIMEOUT_INTERVAL_MS + _minimumStallThreshold
     const totalTimeTaken = LOOP_TIMEOUT_INTERVAL_MS + _minimumStallThreshold;
-    jest.useFakeTimers();
+    jest.useFakeTimers({
+      advanceTimers: true,
+      doNotFake: ['performance'], // Keep real performance API
+    });
     stallTracking['_internalState']['lastIntervalMs'] = Date.now() - totalTimeTaken;
     stallTracking['_internalState']['statsByTransaction'] = new Map();
     stallTracking['_internalState']['iteration']();

packages/core/test/tracing/integrations/stallTracking/stalltracking.test.ts

@@ -13,7 +13,10 @@ import { stallTrackingIntegration } from '../../../../src/js/tracing/integration
 import { getDefaultTestClientOptions, TestClient } from '../../../mocks/client';
 import { expectNonZeroStallMeasurements, expectStallMeasurements } from './stalltrackingutils';
 
-jest.useFakeTimers({ advanceTimers: true });
+jest.useFakeTimers({
+  advanceTimers: true,
+  doNotFake: ['Date', 'performance'] // Keep real Date/performance APIs
+});
 
 const expensiveOperation = () => {
   const expensiveObject: { value: string[] } = {

packages/core/test/tracing/integrations/userInteraction.test.ts

@@ -62,7 +62,10 @@ describe('User Interaction Tracing', () => {
   let mockedUserInteractionId: { elementId: string | undefined; op: string };
 
   beforeEach(() => {
-    jest.useFakeTimers();
+    jest.useFakeTimers({
+      advanceTimers: true,
+      doNotFake: ['performance'], // Keep real performance API
+    });
     NATIVE.enableNative = true;
     mockedAppState.isAvailable = true;
     mockedAppState.currentState = 'active';

packages/core/test/tracing/reactnativenavigation.test.ts

@@ -43,7 +43,10 @@ interface MockEventsRegistry extends EventsRegistry {
   onBottomTabPressed(event: BottomTabPressedEvent): void;
 }
 
-jest.useFakeTimers({ advanceTimers: true });
+jest.useFakeTimers({
+  advanceTimers: true,
+  doNotFake: ['performance'], // Keep real performance API
+});
 
 describe('React Native Navigation Instrumentation', () => {
   let mockEventsRegistry: MockEventsRegistry;

packages/core/test/tracing/reactnavigation.stalltracking.test.ts

@@ -12,7 +12,10 @@ import { getDefaultTestClientOptions, TestClient } from '../mocks/client';
 import { expectStallMeasurements } from './integrations/stallTracking/stalltrackingutils';
 import { createMockNavigationAndAttachTo } from './reactnavigationutils';
 
-jest.useFakeTimers({ advanceTimers: 1 });
+jest.useFakeTimers({
+  advanceTimers: true,
+  doNotFake: ['performance'], // Keep real performance API
+});
 
 describe('StallTracking with ReactNavigation', () => {
   let client: TestClient;

packages/core/test/tracing/reactnavigation.test.ts

@@ -43,7 +43,10 @@ jest.mock('../../src/js/wrapper.ts', () => jest.requireActual('../mockWrapper.ts
 jest.mock('./../../src/js/integrations/debugsymbolicatorutils', () => ({
   getDevServer: jest.fn(),
 }));
-jest.useFakeTimers({ advanceTimers: true });
+jest.useFakeTimers({
+  advanceTimers: true,
+  doNotFake: ['performance'], // Keep real performance API
+});
 
 class MockNavigationContainer {
   currentRoute: NavigationRoute | undefined = dummyRoute;