michal/lab
1
0
Fork 0
Code Issues Pull Requests 3 Actions Packages Projects Releases Wiki Activity

smoke test: surface stderr when the bastion banner never appears
Some checks failed
CI/CD / lint (pull_request) Successful in 48s
Details
CI/CD / typecheck (pull_request) Successful in 47s
Details
CI/CD / test (pull_request) Failing after 1m26s
Details
CI/CD / build (pull_request) Has been skipped
Details
CI/CD / publish-rpm (pull_request) Has been skipped
Details
CI/CD / publish-deb (pull_request) Has been skipped
Details

Browse Source 
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 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01V6HNx6cyr4J4CiMGCy1wkF
This commit is contained in:
Michal
2026-10-14 19:10:48 +01:00
parent 9bbaf2a385
commit 88848f04ac
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
bastion/src/cli/tests/smoke-bastion.test.ts
View File

@@ -143,7 +143,10 @@ describe("bastion smoke tests", () => {
await sleep(200); 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"); expect(stdout).toContain("HTTP server listening");
// Verify the process is still alive after startup // Verify the process is still alive after startup