1
69%
main: 69%

Ran 24 Jul 2026 08:58AM UTC

Files 823

Run time 36s

Badge

Embed ▾

Committed 24 Jul 2026 08:53AM UTC coverage: 69.208% (-0.08%) from 69.286%

Job # 30080549571.1

Build Type

push

github

Committed by

web-flow

Commit Message

Deflake real-backend forwarding tests (#5963)

TestForwarding_*_RealBackend intermittently timed out on CI waiting for a
forwarded, server-initiated notification. waitNotification used a hardcoded
time.After deadline (5s originally, 15s after #5941) that sat inside the
per-test context: a timer shorter than the context flaked under the
full-suite parallel -race load (the async backend -> vMCP -> downstream
relay can take many seconds), while a longer one would mask a genuine hang.

Collapse the two deadlines into one: waitNotification now selects on the
caller's ctx.Done() instead of its own timer, and every real-backend
forwarding test shares a single, generous forwardingRealBackendTimeout
(60s) — well under the 10m go-test global timeout, so a real hang still
fails fast with a clear ctx error. notifCh stays buffered at 8; each test
emits and reads a single notification, so drop-on-full is not a factor.

Verified with `go test -race -count=5 -run TestForwarding_ ./pkg/vmcp/server/`.

Closes #5962.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

77840 of 112473 relevant lines covered (69.21%)

79.72 hits per line

stacklok / toolhive / 30080549571 / 1
69%
main: 69%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 30080549571.1

stacklok / toolhive / 30080549571 / 1 69% main: 69%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 30080549571.1

stacklok / toolhive / 30080549571 / 1
69%
main: 69%

README BADGES
x