fix: reportCoverage context can be optional

vitest-dev · Jan 19, 2023 · 4cd1e5d · 4cd1e5d
1 parent 488e4b9
commit 4cd1e5d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/packages/coverage-c8/src/provider.ts b/packages/coverage-c8/src/provider.ts
@@ -44,7 +44,7 @@ export class C8CoverageProvider implements CoverageProvider {
     takeCoverage()
   }
 
-  async reportCoverage({ allTestsRun }: ReportContext) {
+  async reportCoverage({ allTestsRun }: ReportContext = {}) {
     takeCoverage()
 
     const options = {

diff --git a/packages/coverage-istanbul/src/provider.ts b/packages/coverage-istanbul/src/provider.ts
@@ -98,7 +98,7 @@ export class IstanbulCoverageProvider implements CoverageProvider {
     this.coverages = []
   }
 
-  async reportCoverage({ allTestsRun }: ReportContext) {
+  async reportCoverage({ allTestsRun }: ReportContext = {}) {
     const mergedCoverage: CoverageMap = this.coverages.reduce((coverage, previousCoverageMap) => {
       const map = libCoverage.createCoverageMap(coverage)
       map.merge(previousCoverageMap)

diff --git a/packages/vitest/src/types/coverage.ts b/packages/vitest/src/types/coverage.ts
@@ -13,7 +13,7 @@ export interface CoverageProvider {
   onBeforeFilesRun?(): void | Promise<void>
   onAfterSuiteRun(meta: AfterSuiteRunMeta): void | Promise<void>
 
-  reportCoverage(reportContext: ReportContext): void | Promise<void>
+  reportCoverage(reportContext?: ReportContext): void | Promise<void>
 
   onFileTransform?(
     sourceCode: string,