stacklok / toolhive / 18972917503

51%

Build Type

push

github

Committed by web-flow

Commit Message

Fix flaky vMCP health endpoint tests (#2415)

The health endpoint tests were failing intermittently in CI due to port
conflicts when running parallel tests. All test instances tried to bind
to the default port 4483, causing "address already in use" errors.

Changes:
- Add Ready() channel to vMCP server to signal when listener is created
- Use networking.FindAvailable() to assign unique random ports to tests
- Improve test synchronization by waiting on Ready() channel
- Add error channel to catch and report startup failures

The tests now run reliably with 20 parallel iterations.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude <noreply@anthropic.com>

Run Details

7 of 7 new or added lines in 1 file covered. (100.0%)

11 existing lines in 3 files now uncovered.

20795 of 43817 relevant lines covered (47.46%)

22.18 hits per line