fix: remove invalid heroledger 404 smoke tests #20

Merged

mik-tf merged 1 commit from development_fix_smoke_tests into development

2026-02-27 21:20:29 +00:00

mik-tf commented

2026-02-27 21:18:08 +00:00

Member

Summary

Remove 2 invalid heroledger 404 smoke tests from smoke-test.sh
These endpoints hit the SPA catch-all (returns 200 with index.html), not actual API routes
Fixes deploy-staging CI failure (run #51)

Closes #19

Test

Verified locally: ./scripts/smoke-test.sh https://staging.hub.projectmycelium.com → 30/30 pass

## Summary - Remove 2 invalid heroledger 404 smoke tests from `smoke-test.sh` - These endpoints hit the SPA catch-all (returns 200 with index.html), not actual API routes - Fixes deploy-staging CI failure (run #51) Closes #19 ## Test Verified locally: `./scripts/smoke-test.sh https://staging.hub.projectmycelium.com` → 30/30 pass

mik-tf added 1 commit

2026-02-27 21:18:08 +00:00

fix: remove invalid heroledger 404 smoke tests

Test / check (pull_request) Successful in 2m9s

Details

c10588ff54

These endpoints were never backend API routes — the SPA catch-all
serves index.html for any unknown path, so they always return 200.
Removes the false failures that block deploy-staging CI.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>