fix: remove coverage file path query string

vitest-dev · Oct 28, 2022 · 6dea179 · 6dea179
1 parent 6f7fcab
commit 6dea179
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/packages/coverage-c8/src/provider.ts b/packages/coverage-c8/src/provider.ts
@@ -58,7 +58,7 @@ export class C8CoverageProvider implements CoverageProvider {
         if (!map)
           return
 
-        const url = _url.pathToFileURL(file).href.replace(/\?(.*)/, '')
+        const url = _url.pathToFileURL(file.replace(/\?(.*)/, '')).href
 
         let code: string | undefined
         try {
@@ -86,7 +86,7 @@ export class C8CoverageProvider implements CoverageProvider {
     const offset = 224
 
     report._getSourceMap = (coverage: Profiler.ScriptCoverage) => {
-      const path = _url.pathToFileURL(coverage.url).href.replace(/\?(.*)/, '')
+      const path = _url.pathToFileURL(coverage.url.replace(/\?(.*)/, '')).href
       const data = sourceMapMeta[path]
 
       if (!data)

diff --git a/test/coverage-test/coverage-test/coverage.c8.test.ts b/test/coverage-test/coverage-test/coverage.c8.test.ts
@@ -39,4 +39,4 @@ test('Should show coverage', async () => {
   // For ts and js files
   expect(stdout).contain('math.ts')
   expect(stdout).contain('utils.js')
-})
+}, 10000)
diff --git a/test/coverage-test/vitest.config-c8-coverage.ts b/test/coverage-test/vitest.config-c8-coverage.ts
@@ -9,6 +9,7 @@ export default defineConfig({
       './coverage-test/c8/**/*test.ts',
     ],
     coverage: {
+      reporter: ['html', 'text', 'lcov'],
       include: ['src/**'],
       extension: ['.ts', '.vue', '.js'],
     },