fix(coverage): env-replacer to remove query params from sourcemaps

- Fixes bugs where vue components lost coverage when import.meta.env is used
vitest-dev · Dec 31, 2022 · 7a972b8 · 7a972b8
1 parent 548bd49
commit 7a972b8
Show file tree

Hide file tree

Showing 2 changed files with 427 additions and 6 deletions.
diff --git a/packages/vitest/src/node/plugins/envReplacer.ts b/packages/vitest/src/node/plugins/envReplacer.ts
@@ -1,6 +1,7 @@
 import MagicString from 'magic-string'
 import type { Plugin } from 'vite'
 import { stripLiteral } from 'strip-literal'
+import { cleanUrl } from 'vite-node/utils'
 
 // so people can reassign envs at runtime
 // import.meta.env.VITE_NAME = 'app' -> process.env.VITE_NAME = 'app'
@@ -27,7 +28,12 @@ export const EnvReplacerPlugin = (): Plugin => {
       if (s) {
         return {
           code: s.toString(),
-          map: s.generateMap({ hires: true, source: id }),
+          map: s.generateMap({
+            hires: true,
+
+            // Remove possible query parameters, e.g. vue's "?vue&type=script&src=true&lang.ts"
+            source: cleanUrl(id),
+          }),
         }
       }
     },