test: run each test with fake timers

Amp-Thread-ID: https://ampcode.com/threads/T-019feeff-305b-73a5-bb82-3ae3a74e816d

Author: ampagent

apps/desktop-ui/vitest.config.mts

@@ -18,6 +18,7 @@ export default defineConfig({
     restoreMocks: true,
     unstubEnvs: true,
     unstubGlobals: true,
+    fakeTimers: { shouldAdvanceTime: true },
     globals: true,
     environment: 'happy-dom',
     include: ['src/**/*.{test,spec}.{ts,tsx}'],

apps/website/vitest.config.ts

@@ -14,6 +14,7 @@ export default defineConfig({
     restoreMocks: true,
     unstubEnvs: true,
     unstubGlobals: true,
+    fakeTimers: { shouldAdvanceTime: true },
     environment: 'node',
     include: ['src/**/*.{test,spec}.ts'],
     globals: false,

eslint.config.ts

@@ -103,7 +103,8 @@ export default defineConfig([
           allowDefaultProject: [
             'packages/object-info-parser/vitest.config.ts',
             'vite.electron.config.mts',
-            'vite.types.config.mts'
+            'vite.types.config.mts',
+            'vitest.timer.setup.ts'
           ]
         }
       }

packages/object-info-parser/vitest.config.ts

@@ -6,6 +6,7 @@ export default defineConfig({
     restoreMocks: true,
     unstubEnvs: true,
     unstubGlobals: true,
+    fakeTimers: { shouldAdvanceTime: true },
     environment: 'node',
     include: ['src/__tests__/**/*.test.ts'],
     globals: false,

src/components/queue/job/JobAssetsList.test.ts

@@ -381,7 +381,7 @@ describe('JobAssetsList', () => {
   })
 
   it('shows and hides the job details popover with hover delays', async () => {
-    vi.useFakeTimers()
+    vi.useFakeTimers({ shouldAdvanceTime: false })
     const job = buildJob()
     const { container } = renderJobAssetsList({ jobs: [job] })
 
@@ -411,7 +411,7 @@ describe('JobAssetsList', () => {
   })
 
   it('keeps the job details popover open while hovering the popover', async () => {
-    vi.useFakeTimers()
+    vi.useFakeTimers({ shouldAdvanceTime: false })
     const job = buildJob()
     const { container } = renderJobAssetsList({ jobs: [job] })
 

src/platform/telemetry/providers/cloud/DatadogRumTelemetryProvider.test.ts

@@ -152,15 +152,15 @@ describe('DatadogRumTelemetryProvider', () => {
         ...workflowExecutionIntent
       },
       duration: 20,
-      context: {
+      context: () => ({
         success: false,
         failure_reason: 'submission_rejected',
         terminal_stage: 'submission',
         workflow_started_at_unix_ms: performance.timeOrigin + 42,
         workflow_ended_at_unix_ms: performance.timeOrigin + 62,
         submission_duration_ms: 20,
         trigger_source: 'keybinding'
-      }
+      })
     },
     {
       name: 'queue wait',
@@ -173,7 +173,7 @@ describe('DatadogRumTelemetryProvider', () => {
         ...workflowExecutionIntent
       },
       duration: 40,
-      context: {
+      context: () => ({
         success: false,
         failure_reason: 'execution_failed',
         terminal_stage: 'queue_wait',
@@ -183,7 +183,7 @@ describe('DatadogRumTelemetryProvider', () => {
         submission_duration_ms: 20,
         queue_wait_duration_ms: 20,
         trigger_source: 'keybinding'
-      }
+      })
     },
     {
       name: 'execution',
@@ -197,7 +197,7 @@ describe('DatadogRumTelemetryProvider', () => {
         ...workflowExecutionIntent
       },
       duration: 100,
-      context: {
+      context: () => ({
         success: false,
         failure_reason: 'execution_failed',
         terminal_stage: 'execution',
@@ -209,7 +209,7 @@ describe('DatadogRumTelemetryProvider', () => {
         queue_wait_duration_ms: 30,
         execution_duration_ms: 50,
         trigger_source: 'keybinding'
-      }
+      })
     }
   ] as const)(
     'records a failed workflow vital ending during $name',
@@ -221,7 +221,7 @@ describe('DatadogRumTelemetryProvider', () => {
       expect(addDurationVital).toHaveBeenCalledWith('workflow_execution', {
         startTime: performance.timeOrigin + 42,
         duration,
-        context
+        context: context()
       })
     }
   )

src/platform/workspace/stores/useWorkspaceAuth.test.ts

@@ -107,7 +107,7 @@ describe('useWorkspaceAuthStore', () => {
       origin: 'http://localhost'
     })
     setActivePinia(createPinia())
-    vi.useFakeTimers()
+    vi.useFakeTimers({ shouldAdvanceTime: false })
     sessionStorage.clear()
     mockUnifiedCloudAuthEnabled.value = false
     mockCurrentUser.value = { uid: 'user-a' }

vite.config.mts

@@ -745,6 +745,7 @@ export default defineConfig({
     restoreMocks: true,
     unstubEnvs: true,
     unstubGlobals: true,
+    fakeTimers: { shouldAdvanceTime: true },
     globals: true,
     environment: 'happy-dom',
     // Pin the timezone so date-formatting assertions are deterministic

vitest.timer.setup.ts

@@ -1,4 +1,8 @@
-import { afterEach, vi } from 'vitest'
+import { afterEach, beforeEach, vi } from 'vitest'
+
+beforeEach(() => {
+  vi.useFakeTimers()
+})
 
 afterEach(() => {
   vi.useRealTimers()