fix(coverage): remove coverage/.tmp files after run (#5008)

vitest-dev · Jan 19, 2024 · d53b858 · d53b858
1 parent 538975e
commit d53b858
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/packages/coverage-v8/src/provider.ts b/packages/coverage-v8/src/provider.ts
@@ -235,10 +235,10 @@ export class V8CoverageProvider extends BaseCoverageProvider implements Coverage
           },
         })
       }
-
-      this.coverageFiles = new Map()
-      await fs.rm(this.coverageFilesDirectory, { recursive: true })
     }
+
+    this.coverageFiles = new Map()
+    await fs.rm(this.coverageFilesDirectory, { recursive: true })
   }
 
   private async getUntestedFiles(testedFiles: string[]): Promise<RawCoverage> {

diff --git a/test/coverage-test/coverage-report-tests/generic.report.test.ts b/test/coverage-test/coverage-report-tests/generic.report.test.ts
@@ -184,3 +184,10 @@ test('multi environment coverage is merged correctly', async () => {
   // Condition covered by both tests
   expect(lineCoverage[30]).toBe(2)
 })
+
+test('temporary files are removed after test', async () => {
+  const coveragePath = resolve('./coverage')
+  const files = fs.readdirSync(coveragePath)
+
+  expect(files).not.toContain('.tmp')
+})