fix: dont incorrectly mark run failed if filename pattern excludes previously failed tests

AriPerkkio · sheremet-va · commit 61cf9a7a9809 · 2023-01-27T12:32:53.000+01:00
diff --git a/packages/vitest/src/node/core.ts b/packages/vitest/src/node/core.ts
@@ -374,7 +374,7 @@ export class Vitest {
     await this.reportCoverage(!trigger)
 
     if (!this.config.browser)
-      await this.report('onWatcherStart')
+      await this.report('onWatcherStart', this.state.getFiles(files))
   }
 
   async changeNamePattern(pattern: string, files: string[] = this.state.getFilepaths(), trigger?: string) {
@@ -467,7 +467,7 @@ export class Vitest {
       await this.reportCoverage(false)
 
       if (!this.config.browser)
-        await this.report('onWatcherStart')
+        await this.report('onWatcherStart', this.state.getFiles(files))
     }, WATCHER_DEBOUNCE)
   }
 

Original file line number	Diff line number	Diff line change
`@@ -374,7 +374,7 @@ export class Vitest {`
`374`	`374`	`await this.reportCoverage(!trigger)`
`375`	`375`
`376`	`376`	`if (!this.config.browser)`
`377`		`- await this.report('onWatcherStart')`
	`377`	`+ await this.report('onWatcherStart', this.state.getFiles(files))`
`378`	`378`	`}`
`379`	`379`
`380`	`380`	`async changeNamePattern(pattern: string, files: string[] = this.state.getFilepaths(), trigger?: string) {`
`@@ -467,7 +467,7 @@ export class Vitest {`
`467`	`467`	`await this.reportCoverage(false)`
`468`	`468`
`469`	`469`	`if (!this.config.browser)`
`470`		`- await this.report('onWatcherStart')`
	`470`	`+ await this.report('onWatcherStart', this.state.getFiles(files))`
`471`	`471`	`}, WATCHER_DEBOUNCE)`
`472`	`472`	`}`
`473`	`473`