fix: don't call onCoverage hook if there is no coverage to report

vitest-dev · Jan 14, 2025 · 33c8544 · 33c8544
1 parent 077f230
commit 33c8544
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/docs/advanced/api/reporters.md b/docs/advanced/api/reporters.md
@@ -192,6 +192,8 @@ import type { CoverageMap } from 'istanbul-lib-coverage'
 declare function onCoverage(coverage: CoverageMap): Awaitable<void>
 ```
 
+If Vitest didn't perform any coverage, this hook is not called.
+
 ## onTestModuleQueued
 
 ```ts

diff --git a/packages/vitest/src/node/test-run.ts b/packages/vitest/src/node/test-run.ts
@@ -73,7 +73,9 @@ export class TestRun {
       ])
     }
     finally {
-      await this.vitest.report('onCoverage', coverage)
+      if (coverage) {
+        await this.vitest.report('onCoverage', coverage)
+      }
     }
   }
-Original file line number
+Diff line change
@@ Expand Up / @@ -192,6 +192,8 @@ import type { CoverageMap } from 'istanbul-lib-coverage' @@
     declare function onCoverage(coverage: CoverageMap): Awaitable<void>
     ```
+    If Vitest didn't perform any coverage, this hook is not called.
     ## onTestModuleQueued
     ```ts
@@ Expand Down @@