Skip to content

Commit d53b858

Browse files
authoredJan 19, 2024
fix(coverage): remove coverage/.tmp files after run (#5008)
1 parent 538975e commit d53b858

File tree

2 files changed

+10
-3
lines changed

2 files changed

+10
-3
lines changed
 

‎packages/coverage-v8/src/provider.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -235,10 +235,10 @@ export class V8CoverageProvider extends BaseCoverageProvider implements Coverage
235235
},
236236
})
237237
}
238-
239-
this.coverageFiles = new Map()
240-
await fs.rm(this.coverageFilesDirectory, { recursive: true })
241238
}
239+
240+
this.coverageFiles = new Map()
241+
await fs.rm(this.coverageFilesDirectory, { recursive: true })
242242
}
243243

244244
private async getUntestedFiles(testedFiles: string[]): Promise<RawCoverage> {

‎test/coverage-test/coverage-report-tests/generic.report.test.ts

+7
Original file line numberDiff line numberDiff line change
@@ -184,3 +184,10 @@ test('multi environment coverage is merged correctly', async () => {
184184
// Condition covered by both tests
185185
expect(lineCoverage[30]).toBe(2)
186186
})
187+
188+
test('temporary files are removed after test', async () => {
189+
const coveragePath = resolve('./coverage')
190+
const files = fs.readdirSync(coveragePath)
191+
192+
expect(files).not.toContain('.tmp')
193+
})

0 commit comments

Comments
 (0)
Please sign in to comment.