✅ Migrate 257 spec files from Jasmine to Vitest API

Author: mormubis

AGENTS.md

@@ -22,10 +22,10 @@ yarn build:apps
 yarn test:unit
 
 # Run specific test file
-yarn test:unit --spec packages/core/src/path/to/feature.spec.ts
+yarn vitest run packages/core/src/path/to/feature.spec.ts
 
 # Run tests on a specific seed
-yarn test:unit --seed 123
+yarn vitest run --sequence.seed 123
 
 # setup E2E tests (installs Playwright and builds test apps)
 yarn test:e2e:init
@@ -66,7 +66,7 @@ test/
 ├── apps/            # Test apps for E2E and performance testing
 ├── e2e/             # Playwright E2E test scenarios
 ├── performance/     # Performance benchmarking tests
-└── unit/            # Karma/Jasmine unit test configuration
+└── unit/            # Vitest unit test configuration
 
 scripts/             # Build, deploy, release automation
 ```
@@ -86,9 +86,9 @@ For deeper context, see:
 
 ### Unit Tests
 
-- Test framework: Jasmine + Karma. Spec files co-located with implementation: `feature.ts` → `feature.spec.ts`
-- Focus tests with `fit()` / `fdescribe()`, skip with `xit()` / `xdescribe()`
+- Spec files co-located with implementation: `feature.ts` → `feature.spec.ts`
 - Use `registerCleanupTask()` for cleanup, NOT `afterEach()`
+- Test framework: Vitest (browser mode with Playwright)
 - Mock values/functions: wrap with `mockable()` in source, use `replaceMockable()` or `replaceMockableWithSpy()` in tests (auto-cleanup)
 
 ### Naming Conventions

developer-extension/src/panel/components/tabs/eventsTab/computeFacetState.spec.ts

@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest'
 import type { RumActionEvent, RumResourceEvent } from '@datadog/browser-rum'
 import { FacetRegistry } from '../../../hooks/useEvents'
 import type { FacetValuesFilter } from '../../../hooks/useEvents'

developer-extension/src/panel/components/tabs/eventsTab/copyEvent.spec.ts

@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest'
 import type { TelemetryEvent } from '../../../../../../packages/core/src/domain/telemetry'
 import type { LogsEvent } from '../../../../../../packages/logs/src/logsEvent.types'
 import type { RumEvent } from '../../../../../../packages/rum-core/src/rumEvent.types'

developer-extension/src/panel/flushEvents.spec.ts

@@ -1,15 +1,16 @@
+import { vi, beforeEach, describe, expect, it, type Mock } from 'vitest'
 import type { Configuration } from '@datadog/browser-core'
 import { registerCleanupTask } from '../../../packages/core/test'
 import type { PageMayExitEvent } from '../../../packages/core/src/browser/pageMayExitObservable'
 import { createPageMayExitObservable } from '../../../packages/core/src/browser/pageMayExitObservable'
 import { flushScript } from './flushEvents'
 
 describe('flushEvents', () => {
-  let onExitSpy: jasmine.Spy<(event: PageMayExitEvent) => void>
+  let onExitSpy: Mock<(event: PageMayExitEvent) => void>
   let configuration: Configuration
 
   beforeEach(() => {
-    onExitSpy = jasmine.createSpy()
+    onExitSpy = vi.fn()
     configuration = {} as Configuration
     registerCleanupTask(createPageMayExitObservable(configuration).subscribe(onExitSpy).unsubscribe)
   })

developer-extension/src/panel/hooks/useEvents/eventFilters.spec.ts

@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest'
 import type { RumEvent } from '../../../../../packages/rum-core/src/rumEvent.types'
 import type { LogsEvent } from '../../../../../packages/logs/src/logsEvent.types'
 import { isSafari } from '../../../../../packages/core/src/tools/utils/browserDetection'

developer-extension/src/panel/hooks/useEvents/facetRegistry.spec.ts

@@ -1,10 +1,12 @@
+import { beforeEach, describe, expect, it } from 'vitest'
 import { isChromium } from '../../../../../packages/core/src/tools/utils/browserDetection'
 import { getAllFields } from './facetRegistry'
 
 describe('getAllFields', () => {
-  beforeEach(() => {
+  beforeEach((ctx) => {
     if (!isChromium()) {
-      pending('Extension only supported in chromium')
+      ctx.skip()
+      return
     }
   })
 

packages/core/src/boot/displayAlreadyInitializedError.spec.ts

@@ -1,17 +1,18 @@
+import { vi, describe, expect, it } from 'vitest'
 import type { InitConfiguration } from '../domain/configuration'
 import { display } from '../tools/display'
 import { displayAlreadyInitializedError } from './displayAlreadyInitializedError'
 
 describe('displayAlreadyInitializedError', () => {
   it('should display an error', () => {
-    const displayErrorSpy = spyOn(display, 'error')
+    const displayErrorSpy = vi.spyOn(display, 'error')
     displayAlreadyInitializedError('DD_RUM', {} as InitConfiguration)
     expect(displayErrorSpy).toHaveBeenCalledTimes(1)
     expect(displayErrorSpy).toHaveBeenCalledWith('DD_RUM is already initialized.')
   })
 
   it('should not display an error if the "silentMultipleInit" option is used', () => {
-    const displayErrorSpy = spyOn(display, 'error')
+    const displayErrorSpy = vi.spyOn(display, 'error')
     displayAlreadyInitializedError('DD_RUM', { silentMultipleInit: true } as InitConfiguration)
     expect(displayErrorSpy).not.toHaveBeenCalled()
   })

packages/core/src/boot/init.spec.ts

@@ -1,3 +1,4 @@
+import { vi, describe, expect, it } from 'vitest'
 import { display } from '../tools/display'
 import { defineGlobal } from './init'
 
@@ -17,8 +18,8 @@ describe('defineGlobal', () => {
   })
 
   it('run the queued callbacks on the old value', () => {
-    const fn1 = jasmine.createSpy()
-    const fn2 = jasmine.createSpy()
+    const fn1 = vi.fn()
+    const fn2 = vi.fn()
     const myGlobal: any = {
       foo: {
         q: [fn1, fn2],
@@ -42,7 +43,7 @@ describe('defineGlobal', () => {
         q: [onReady],
       },
     }
-    const displaySpy = spyOn(display, 'error')
+    const displaySpy = vi.spyOn(display, 'error')
 
     defineGlobal(myGlobal, 'foo', {})
     expect(displaySpy).toHaveBeenCalledWith('onReady callback threw an error:', myError)

packages/core/src/browser/addEventListener.spec.ts

@@ -1,3 +1,4 @@
+import { vi, beforeEach, describe, expect, it } from 'vitest'
 import type { Configuration } from '../domain/configuration'
 import { createNewEvent, mockZoneJs, registerCleanupTask } from '../../test'
 import type { MockZoneJs } from '../../test'
@@ -16,7 +17,7 @@ describe('addEventListener', () => {
     })
 
     it('uses the original addEventListener method instead of the method patched by Zone.js', () => {
-      const zoneJsPatchedAddEventListener = jasmine.createSpy()
+      const zoneJsPatchedAddEventListener = vi.fn()
       const eventTarget = document.createElement('div')
       zoneJs.replaceProperty(eventTarget, 'addEventListener', zoneJsPatchedAddEventListener)
 
@@ -25,7 +26,7 @@ describe('addEventListener', () => {
     })
 
     it('uses the original removeEventListener method instead of the method patched by Zone.js', () => {
-      const zoneJsPatchedRemoveEventListener = jasmine.createSpy()
+      const zoneJsPatchedRemoveEventListener = vi.fn()
       const eventTarget = document.createElement('div')
       zoneJs.replaceProperty(eventTarget, 'removeEventListener', zoneJsPatchedRemoveEventListener)
 
@@ -41,17 +42,17 @@ describe('addEventListener', () => {
     // eslint-disable-next-line @typescript-eslint/unbound-method
     const originalRemoveEventListener = EventTarget.prototype.removeEventListener
 
-    EventTarget.prototype.addEventListener = jasmine.createSpy()
-    EventTarget.prototype.removeEventListener = jasmine.createSpy()
+    EventTarget.prototype.addEventListener = vi.fn()
+    EventTarget.prototype.removeEventListener = vi.fn()
 
     registerCleanupTask(() => {
       EventTarget.prototype.addEventListener = originalAddEventListener
       EventTarget.prototype.removeEventListener = originalRemoveEventListener
     })
 
     const htmlDivElement = document.createElement('div')
-    htmlDivElement.addEventListener = jasmine.createSpy()
-    htmlDivElement.removeEventListener = jasmine.createSpy()
+    htmlDivElement.addEventListener = vi.fn()
+    htmlDivElement.removeEventListener = vi.fn()
 
     const { stop } = addEventListener({ allowUntrustedEvents: false }, htmlDivElement, DOM_EVENT.CLICK, noop)
 
@@ -71,11 +72,11 @@ describe('addEventListener', () => {
   })
 
   it('Use the addEventListener method when the eventTarget is not an instance of EventTarget', () => {
-    const listener = jasmine.createSpy()
+    const listener = vi.fn()
 
     const customEventTarget = {
-      addEventListener: jasmine.createSpy(),
-      removeEventListener: jasmine.createSpy(),
+      addEventListener: vi.fn(),
+      removeEventListener: vi.fn(),
     } as unknown as HTMLElement
 
     const { stop } = addEventListener({ allowUntrustedEvents: false }, customEventTarget, 'change', listener)
@@ -93,7 +94,7 @@ describe('addEventListener', () => {
     })
 
     it('should be ignored if __ddIsTrusted is absent', () => {
-      const listener = jasmine.createSpy()
+      const listener = vi.fn()
       const eventTarget = document.createElement('div')
       addEventListener(configuration, eventTarget, DOM_EVENT.CLICK, listener)
 
@@ -103,7 +104,7 @@ describe('addEventListener', () => {
     })
 
     it('should be ignored if __ddIsTrusted is false', () => {
-      const listener = jasmine.createSpy()
+      const listener = vi.fn()
       const eventTarget = document.createElement('div')
       addEventListener(configuration, eventTarget, DOM_EVENT.CLICK, listener)
 
@@ -113,7 +114,7 @@ describe('addEventListener', () => {
     })
 
     it('should not be ignored if __ddIsTrusted is true', () => {
-      const listener = jasmine.createSpy()
+      const listener = vi.fn()
       const eventTarget = document.createElement('div')
       addEventListener(configuration, eventTarget, DOM_EVENT.CLICK, listener)
 
@@ -124,7 +125,7 @@ describe('addEventListener', () => {
     })
 
     it('should not be ignored if allowUntrustedEvents is true', () => {
-      const listener = jasmine.createSpy()
+      const listener = vi.fn()
       const eventTarget = document.createElement('div')
       configuration = { allowUntrustedEvents: true } as Configuration
 

packages/core/src/browser/cookie.spec.ts

@@ -1,3 +1,4 @@
+import { describe, expect, it } from 'vitest'
 import { mockCookies } from '../../test'
 import { getCurrentSite } from './cookie'