fix(browser): resolve coverage.reporter from string values (#5920)

AriPerkkio · web-flow · commit f33dabbef2f3 · 2024-06-19T11:53:04.000+03:00
diff --git a/packages/browser/src/node/index.ts b/packages/browser/src/node/index.ts
@@ -358,7 +358,7 @@ export default (project: WorkspaceProject, base = '/'): Plugin[] => {
 function resolveCoverageFolder(project: WorkspaceProject) {
   const options = project.ctx.config
   const htmlReporter = options.coverage?.enabled
-    ? options.coverage.reporter.find((reporter) => {
+    ? toArray(options.coverage.reporter).find((reporter) => {
       if (typeof reporter === 'string') {
         return reporter === 'html'
       }
diff --git a/packages/ui/node/index.ts b/packages/ui/node/index.ts
@@ -4,6 +4,7 @@ import sirv from 'sirv'
 import type { Plugin } from 'vite'
 import { coverageConfigDefaults } from 'vitest/config'
 import type { Vitest } from 'vitest'
+import { toArray } from '@vitest/utils'
 
 export default (ctx: Vitest): Plugin => {
   return <Plugin>{
@@ -50,7 +51,7 @@ function resolveCoverageFolder(ctx: Vitest) {
   const options = ctx.config
   const htmlReporter
     = options.api?.port && options.coverage?.enabled
-      ? options.coverage.reporter.find((reporter) => {
+      ? toArray(options.coverage.reporter).find((reporter) => {
         if (typeof reporter === 'string') {
           return reporter === 'html'
         }

Original file line number	Diff line number	Diff line change
`@@ -358,7 +358,7 @@ export default (project: WorkspaceProject, base = '/'): Plugin[] => {`
`358`	`358`	`function resolveCoverageFolder(project: WorkspaceProject) {`
`359`	`359`	`const options = project.ctx.config`
`360`	`360`	`const htmlReporter = options.coverage?.enabled`
`361`		`- ? options.coverage.reporter.find((reporter) => {`
	`361`	`+ ? toArray(options.coverage.reporter).find((reporter) => {`
`362`	`362`	`if (typeof reporter === 'string') {`
`363`	`363`	`return reporter === 'html'`
`364`	`364`	`}`