fix(coverage): isolate: false with v8 provider

Author: AriPerkkio

packages/browser/src/client/tester/tester.ts

@@ -122,7 +122,7 @@ async function executeTests(method: 'run' | 'collect', files: string[]) {
   try {
     await Promise.all([
       setupCommonEnv(config),
-      startCoverageInsideWorker(config.coverage, executor),
+      startCoverageInsideWorker(config.coverage, executor, { isolate: config.browser.isolate }),
       (async () => {
         const VitestIndex = await import('vitest')
         Object.defineProperty(window, '__vitest_index__', {
@@ -160,7 +160,7 @@ async function executeTests(method: 'run' | 'collect', files: string[]) {
       }, 'Cleanup Error')
     }
     state.environmentTeardownRun = true
-    await stopCoverageInsideWorker(config.coverage, executor).catch((error) => {
+    await stopCoverageInsideWorker(config.coverage, executor, { isolate: config.browser.isolate }).catch((error) => {
       client.rpc.onUnhandledError({
         name: error.name,
         message: error.message,

packages/coverage-v8/src/index.ts

@@ -1,12 +1,20 @@
+import type { CoverageProviderModule } from 'vitest/node'
 import type { V8CoverageProvider } from './provider'
 import inspector, { type Profiler } from 'node:inspector'
 import { provider } from 'std-env'
 import { loadProvider } from './load-provider'
 
 const session = new inspector.Session()
+let enabled = false
+
+export default <CoverageProviderModule>{
+  startCoverage({ isolate }) {
+    if (isolate === false && enabled) {
+      return
+    }
+
+    enabled = true
 
-export default {
-  startCoverage(): void {
     session.connect()
     session.post('Profiler.enable')
     session.post('Profiler.startPreciseCoverage', {
@@ -34,7 +42,11 @@ export default {
     })
   },
 
-  stopCoverage(): void {
+  stopCoverage({ isolate }) {
+    if (isolate === false) {
+      return
+    }
+
     session.post('Profiler.stopPreciseCoverage')
     session.post('Profiler.disable')
     session.disconnect()

packages/vitest/src/integrations/coverage.ts

@@ -79,11 +79,12 @@ export async function getCoverageProvider(
 export async function startCoverageInsideWorker(
   options: SerializedCoverageConfig | undefined,
   loader: Loader,
+  runtimeOptions: { isolate: boolean },
 ) {
   const coverageModule = await resolveCoverageProviderModule(options, loader)
 
   if (coverageModule) {
-    return coverageModule.startCoverage?.()
+    return coverageModule.startCoverage?.(runtimeOptions)
   }
 
   return null
@@ -105,11 +106,12 @@ export async function takeCoverageInsideWorker(
 export async function stopCoverageInsideWorker(
   options: SerializedCoverageConfig | undefined,
   loader: Loader,
+  runtimeOptions: { isolate: boolean },
 ) {
   const coverageModule = await resolveCoverageProviderModule(options, loader)
 
   if (coverageModule) {
-    return coverageModule.stopCoverage?.()
+    return coverageModule.stopCoverage?.(runtimeOptions)
   }
 
   return null

packages/vitest/src/node/types/coverage.ts

@@ -66,7 +66,7 @@ export interface CoverageProviderModule {
   /**
    * Executed before tests are run in the worker thread.
    */
-  startCoverage?: () => unknown | Promise<unknown>
+  startCoverage?: (runtimeOptions: { isolate: boolean }) => unknown | Promise<unknown>
 
   /**
    * Executed on after each run in the worker thread. Possible to return a payload passed to the provider
@@ -76,7 +76,7 @@ export interface CoverageProviderModule {
   /**
    * Executed after all tests have been run in the worker thread.
    */
-  stopCoverage?: () => unknown | Promise<unknown>
+  stopCoverage?: (runtimeOptions: { isolate: boolean }) => unknown | Promise<unknown>
 }
 
 export type CoverageReporter = keyof ReportOptions | (string & {})

packages/vitest/src/runtime/runBaseTests.ts

@@ -25,8 +25,12 @@ export async function run(
 ): Promise<void> {
   const workerState = getWorkerState()
 
+  const isIsolatedThreads = config.pool === 'threads' && (config.poolOptions?.threads?.isolate ?? true)
+  const isIsolatedForks = config.pool === 'forks' && (config.poolOptions?.forks?.isolate ?? true)
+  const isolate = isIsolatedThreads || isIsolatedForks
+
   await setupGlobalEnv(config, environment, executor)
-  await startCoverageInsideWorker(config.coverage, executor)
+  await startCoverageInsideWorker(config.coverage, executor, { isolate })
 
   if (config.chaiConfig) {
     setupChaiConfig(config.chaiConfig)
@@ -50,14 +54,7 @@ export async function run(
         = performance.now() - workerState.durations.environment
 
       for (const file of files) {
-        const isIsolatedThreads
-          = config.pool === 'threads'
-          && (config.poolOptions?.threads?.isolate ?? true)
-        const isIsolatedForks
-          = config.pool === 'forks'
-          && (config.poolOptions?.forks?.isolate ?? true)
-
-        if (isIsolatedThreads || isIsolatedForks) {
+        if (isolate) {
           executor.mocker.reset()
           resetModules(workerState.moduleCache, true)
         }
@@ -77,7 +74,7 @@ export async function run(
         vi.restoreAllMocks()
       }
 
-      await stopCoverageInsideWorker(config.coverage, executor)
+      await stopCoverageInsideWorker(config.coverage, executor, { isolate })
     },
   )
 

packages/vitest/src/runtime/runVmTests.ts

@@ -62,7 +62,7 @@ export async function run(
     getSourceMap: source => workerState.moduleCache.getSourceMap(source),
   })
 
-  await startCoverageInsideWorker(config.coverage, executor)
+  await startCoverageInsideWorker(config.coverage, executor, { isolate: false })
 
   if (config.chaiConfig) {
     setupChaiConfig(config.chaiConfig)
@@ -101,7 +101,7 @@ export async function run(
     vi.restoreAllMocks()
   }
 
-  await stopCoverageInsideWorker(config.coverage, executor)
+  await stopCoverageInsideWorker(config.coverage, executor, { isolate: false })
 }
 
 function resolveCss(mod: NodeJS.Module) {

test/coverage-test/test/isolation.test.ts

@@ -1,12 +1,9 @@
 import type { WorkspaceSpec } from 'vitest/node'
 import { expect, test } from 'vitest'
-import { isV8Provider, readCoverageMap, runVitest } from '../utils'
+import { readCoverageMap, runVitest } from '../utils'
 
 for (const isolate of [true, false]) {
-  // TODO: Requires #6736
-  const fails = isV8Provider() && isolate === false
-
-  test(`{ isolate: ${isolate} }`, { fails }, async () => {
+  test(`{ isolate: ${isolate} }`, async () => {
     await runVitest({
       include: ['fixtures/test/isolation-*'],
       setupFiles: ['fixtures/setup.isolation.ts'],