chore: cleanup

packages/coverage-v8/src/provider.ts

@@ -334,7 +334,10 @@ export class V8CoverageProvider extends BaseCoverageProvider<ResolvedCoverageOpt
     code: string
     map?: Vite.Rollup.SourceMap
   }> {
-    const filepath = url.replace(/file:\/\/\/?/, '') // Windows will have file:///C:, unix will have file://
+    // TODO: need to standardize file urls before this call somehow, this is messy
+    const filepath = url.match(/^file:\/\/\/\w:\//)
+      ? url.slice(8)
+      : removeStartsWith(url, FILE_PROTOCOL)
     // TODO: do we still need to "catch" here? why would it fail?
     const transformResult = await onTransform(filepath).catch(() => null)
 

test/cli/fixtures/bail-race/a.test.ts

@@ -1,6 +1,4 @@
-import { setTimeout } from 'node:timers/promises'
-import {afterAll, expect, test } from 'vitest'
-
+import { expect, test } from 'vitest'
 
 test('adds two numbers', () => {
   expect(2 + 3).toBe(5)

test/cli/fixtures/bail-race/b.test.ts

@@ -1,7 +1,5 @@
-import { expect, test } from 'vitest'
+import { test } from 'vitest'
 
 test('does not run', () => {
   throw new Error("Should never run")
 })
-
-

test/cli/test/bail-race.test.ts

@@ -22,7 +22,7 @@ test('cancels previous run before starting new one', async () => {
     }],
   })
 
-  for (let i = 0; i <= 10; i++) {
+  for (let i = 0; i <= 4; i++) {
     await vitest.start()
   }