From 88848f04ac4b2423a3b735208434fc3543775972 Mon Sep 17 00:00:00 2001 From: Michal Date: Wed, 14 Oct 2026 19:10:48 +0100 Subject: [PATCH] smoke test: surface stderr when the bastion banner never appears The CI failure said only "expected '' to contain ...": the process died before printing anything to stdout and the test swallowed stderr, which is the half that says why. Co-Authored-By: Claude Fable 5 Claude-Session: https://claude.ai/code/session_01V6HNx6cyr4J4CiMGCy1wkF --- bastion/src/cli/tests/smoke-bastion.test.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/bastion/src/cli/tests/smoke-bastion.test.ts b/bastion/src/cli/tests/smoke-bastion.test.ts index dd50230..2030e50 100644 --- a/bastion/src/cli/tests/smoke-bastion.test.ts +++ b/bastion/src/cli/tests/smoke-bastion.test.ts @@ -143,7 +143,10 @@ describe("bastion smoke tests", () => { await sleep(200); } - expect(stdout).toContain("Waiting for PXE boot requests"); + // On failure, show what the process actually said -- an empty stdout with + // a hidden stderr is undiagnosable from a CI log. + expect(stdout, `bastion never printed its banner; stderr was:\n${stderr}`) + .toContain("Waiting for PXE boot requests"); expect(stdout).toContain("HTTP server listening"); // Verify the process is still alive after startup