fix: report coverage even when no tests found (#3091)

vitest-dev · Mar 28, 2023 · 2cb9121 · 2cb9121
1 parent 0f44d2c
commit 2cb9121
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/packages/vitest/src/node/config.ts b/packages/vitest/src/node/config.ts
@@ -268,7 +268,7 @@ export function resolveConfig(
 }
 
 export function isBrowserEnabled(config: ResolvedConfig) {
-  if (config.browser.enabled)
+  if (config.browser?.enabled)
     return true
 
   return config.poolMatchGlobs?.length && config.poolMatchGlobs.some(([, pool]) => pool === 'browser')

diff --git a/packages/vitest/src/node/core.ts b/packages/vitest/src/node/core.ts
@@ -266,6 +266,7 @@ export class Vitest {
     if (!files.length) {
       const exitCode = this.config.passWithNoTests ? 0 : 1
 
+      await this.reportCoverage(true)
       this.logger.printNoTestFound(filters)
 
       process.exit(exitCode)

diff --git a/test/coverage-test/testing.mjs b/test/coverage-test/testing.mjs
@@ -44,8 +44,10 @@ for (const threads of [{ threads: true }, { threads: false }, { singleThread: tr
         isolate,
       })
 
-      if (process.exitCode)
+      if (process.exitCode) {
+        console.error(`process.exitCode was set to ${process.exitCode}, exiting.`)
         exit()
+      }
     }
   }
 }