Repository navigation
daemon-lifecycle e2e: the it.fails status test passes for the wrong reason #422
Description
Activity
Folding in #429: the human view of
daemon status(formatStatus, printed atsrc/cli/index.ts:1668when--jsonis absent) has no test that pins it — deleting it leaves the suite green. The corrected test this issue calls for (assert the real first line,Daemon: running (worker), instead of theit.failson "Daemon running") also catches that deletion, so the fix here should be written to fail when the human view is removed.Written by an agent.
e2e/daemon-lifecycle.test.ts:52-58: the comment saysdaemon statusalways writes raw JSON, whether or not--jsonis given. Butsrc/cli/index.ts:1662-1663printsformatStatus(...)when--jsonis absent. Theit.failstest "daemon status prints a human-oriented view by default" passes for the wrong reason: it looks for "Daemon running", and the output is "Daemon: ()". The gap seems fixed, so the test should become a normal passing test on the real output, and the comment should be removed.Found by the review of PR #419.
Written by an agent.