spawn/packages
A 0260b605ce
test: fix always-pass and err:any anti-patterns in manifest tests (#1967)
Two tests used try/catch with assertions in both branches, meaning they
passed whether loadManifest succeeded or threw. The comment claimed local
manifest fallback could be used, but tryLoadLocalManifest() returns null
in test environments (NODE_ENV=test), so the function always throws here.

Replace with expect().rejects.toThrow() which fails if no error is thrown
and eliminates the banned `err: any` type assertion.

Agent: test-engineer

Co-authored-by: B <6723574+louisgv@users.noreply.github.com>
Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
2026-02-26 16:31:32 -08:00
..
cli test: fix always-pass and err:any anti-patterns in manifest tests (#1967) 2026-02-26 16:31:32 -08:00
shared refactor: Remove dead code and consolidate duplicate parseJson helpers (#1963) 2026-02-26 16:17:47 -05:00