Skip to content

Commit a2e9daf

Browse files
authoredNov 21, 2022
fix(coverage): env-replacer to add filenames into sourcemaps (#2338)
1 parent 2499828 commit a2e9daf

File tree

4 files changed

+17
-2
lines changed

4 files changed

+17
-2
lines changed
 

‎packages/vitest/src/node/plugins/envReplacer.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ export const EnvReplacerPlugin = (): Plugin => {
88
return {
99
name: 'vitest:env-replacer',
1010
enforce: 'pre',
11-
transform(code) {
11+
transform(code, id) {
1212
if (!/\bimport\.meta\.env\b/g.test(code))
1313
return null
1414

@@ -27,7 +27,7 @@ export const EnvReplacerPlugin = (): Plugin => {
2727
if (s) {
2828
return {
2929
code: s.toString(),
30-
map: s.generateMap({ hires: true }),
30+
map: s.generateMap({ hires: true, source: id }),
3131
}
3232
}
3333
},

‎test/coverage-test/coverage-test/coverage.istanbul.test.ts

+7
Original file line numberDiff line numberDiff line change
@@ -50,3 +50,10 @@ test('implicit else is included in branch count', async () => {
5050
expect(fileCoverage.b).toHaveProperty('0')
5151
expect(fileCoverage.b['0']).toHaveLength(2)
5252
})
53+
54+
test('file using import.meta.env is included in report', async () => {
55+
const coveragePath = resolve('./coverage/src')
56+
const files = fs.readdirSync(coveragePath)
57+
58+
expect(files).toContain('importEnv.ts.html')
59+
})

‎test/coverage-test/src/importEnv.ts

+3
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
export function useImportEnv() {
2+
return import.meta.env.SOME_VARIABLE == null
3+
}

‎test/coverage-test/test/coverage.test.ts

+5
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
import { expect, test } from 'vitest'
22
import { pythagoras } from '../src'
33
import { implicitElse } from '../src/implicitElse'
4+
import { useImportEnv } from '../src/importEnv'
45

56
test('Math.sqrt()', async () => {
67
expect(pythagoras(3, 4)).toBe(5)
@@ -9,3 +10,7 @@ test('Math.sqrt()', async () => {
910
test('implicit else', () => {
1011
expect(implicitElse(true)).toBe(2)
1112
})
13+
14+
test('import meta env', () => {
15+
expect(useImportEnv()).toBe(true)
16+
})

0 commit comments

Comments
 (0)
Please sign in to comment.