From 2cb91211d7b05c5ad69846fbff5da7be5825d998 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ari=20Perkki=C3=B6?= Date: Tue, 28 Mar 2023 14:41:57 +0300 Subject: [PATCH] fix: report coverage even when no tests found (#3091) --- packages/vitest/src/node/config.ts | 2 +- packages/vitest/src/node/core.ts | 1 + test/coverage-test/testing.mjs | 4 +++- 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/packages/vitest/src/node/config.ts b/packages/vitest/src/node/config.ts index 0e37c092b635..c9b6d7cfd7ab 100644 --- 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 index 607ee65d050f..474c1c2b1d60 100644 --- 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 index 9fb28beed6cb..7b924d0c4a4f 100644 --- 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() + } } } }