Skip to content

Commit 2578709

Browse files
committed
fix: the Cloud live run stops collecting the Service Management suites
`vitest.config.live.ts` globs everything under tests/live and excluded tests/live/server for exactly this reason — those need a container brought up first, under their own config and their own global setup. tests/live/jsm needed the same line and did not get it, so the Cloud run collected thirty-four tests whose fixtures this run's global setup never provides and which failed on `undefined`. The comment now names the rule rather than the one directory, so the next self-hosted surface is not a third instance of this.
1 parent 53ff06b commit 2578709

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

vitest.config.live.ts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,13 @@ const repoRoot = import.meta.dirname;
66

77
export default defineConfig(({ mode }) => ({
88
test: {
9-
// Not `tests/live/server` — those run against a container that has to be brought up first, under their own
10-
// config and their own global setup. Left in, every one of them fails with a connection refused.
9+
// Every suite that runs against a container of its own is excluded here, because each has its own config and its
10+
// own global setup and the container has to be brought up first: `tests/live/server` needs `pnpm jira-dc:up`,
11+
// `tests/live/jsm` needs `pnpm jsm-dc:up`. Left in, they do not merely fail — they fail with `undefined` where
12+
// their fixtures should be, because this run's global setup provides different ones. A new self-hosted surface
13+
// belongs on this list the day its directory appears.
1114
include: ['tests/live/**/*.test.ts'],
12-
exclude: ['tests/live/server/**'],
15+
exclude: ['tests/live/server/**', 'tests/live/jsm/**'],
1316
environment: 'node',
1417
reporters: ['verbose'],
1518
env: loadEnv(mode, repoRoot, ''),

0 commit comments

Comments
 (0)