fix: reportCoverage context can be optional

antfu · antfu · commit 4cd1e5d7fb58 · 2023-01-19T11:30:28.000+01:00
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,

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ export class C8CoverageProvider implements CoverageProvider {`
`44`	`44`	`takeCoverage()`
`45`	`45`	`}`
`46`	`46`
`47`		`- async reportCoverage({ allTestsRun }: ReportContext) {`
	`47`	`+ async reportCoverage({ allTestsRun }: ReportContext = {}) {`
`48`	`48`	`takeCoverage()`
`49`	`49`
`50`	`50`	`const options = {`
Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,7 @@ export class IstanbulCoverageProvider implements CoverageProvider {`
`98`	`98`	`this.coverages = []`
`99`	`99`	`}`
`100`	`100`
`101`		`- async reportCoverage({ allTestsRun }: ReportContext) {`
	`101`	`+ async reportCoverage({ allTestsRun }: ReportContext = {}) {`
`102`	`102`	`const mergedCoverage: CoverageMap = this.coverages.reduce((coverage, previousCoverageMap) => {`
`103`	`103`	`const map = libCoverage.createCoverageMap(coverage)`
`104`	`104`	`map.merge(previousCoverageMap)`