spawn/packages
A b9b0b40856
test: remove duplicate isValidManifest block from commands-swap-resolve (#1977)
The manifest validation (isValidManifest) describe block in
commands-swap-resolve.test.ts used an always-pass anti-pattern:
try { await loadManifest() } catch {} followed by console.error.some()
assertions. This pattern silently passes even when the expected rejection
path is not triggered.

The same coverage (missing agents/clouds/matrix fields, null data, HTTP
errors, valid manifests) is already provided by manifest-cache-lifecycle.test.ts
with proper expect().rejects.toThrow() assertions.

Remove the duplicate 145-line block. No regression: pass/fail counts unchanged.

-- qa/dedup-scanner

Co-authored-by: spawn-qa-bot <qa@openrouter.ai>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-02-27 00:04:41 -05:00
..
cli test: remove duplicate isValidManifest block from commands-swap-resolve (#1977) 2026-02-27 00:04:41 -05:00
shared refactor: Remove dead code and consolidate duplicate parseJson helpers (#1963) 2026-02-26 16:17:47 -05:00