* test(server): reduce slow unit test waits
Co-authored-by: multica-agent <[email protected]>
* test(server): cut slow Go unit test time further
Follow-up to the first pass on MUL-7271. Test-only changes plus a few
package-private seams whose production values are unchanged.
- Re-executed race-built test binaries no longer pay the race runtime's 1s
atexit sleep (GORACE=atexit_sleep_ms=0 for child processes only).
- pkg/agent: event-driven or shorter test-only bounds for handshake, drain
and idle graces; redundant timeout variants merged.
- daemon/execenv: git fixtures copied from a template repo built once; tests
that re-ran the same worktree lifecycle folded into their supersets;
ForkLock-guarded fake-CLI writes so parallel tests cannot hit ETXTBSY.
- handler/service: cap and paging tests shrink their caps instead of seeding
thousands of rows; lock waits probe pg_locks instead of sleeping; one
shared pgxpool per service test binary; cheaper cron and RSA fixtures.
- wecom: the phishing-link sweep reuses one parser and derives CRLF from LF
with a sampled cross-check; fake sockets acknowledge instead of waiting out
the ack timeout.
- migrate: the migration corpus is parsed once; isolated DB tests run in
parallel.
Co-authored-by: multica-agent <[email protected]>
* test(server): remove transport timeout unit tests
Co-authored-by: multica-agent <[email protected]>
* test(server): restore two non-timeout tests and make a bound load-bearing
Follow-up to the review of #8318. All test-only; no production code changes.
- daemon: restore the three read-liveness cases that the transport-timeout
sweep took with it. They assert that a ping, a pong and an application
frame each EXTEND the read deadline, so a healthy wakeup socket survives
without traffic of our own — the deadline never fires on the happy path,
which is why they are not transport-timeout tests. Verified they fail
(all three) when the extension calls are removed from wakeup.go, and pass
in 0.57s together. The pure "times out without peer traffic" case stays
deleted.
- pkg/agent: restore the parent-DEADLINE half of the opaque-env redaction
pair. codex.go treats a parent deadline and a parent cancellation as one
ending; only the Canceled arm was left covered. The fake now writes the
secret to stderr before reading the handshake, so the check no longer
races our own initialize write, and the budget is 2s against a measured
430-465ms launch. Verified it fails when the !contextEnded guard is
removed.
- pkg/agent: TestRunCollectQuietDoesNotWaitWhenTheAnswerIsIn had its bound
widened to a literal 5s, which the shrunk 750ms drain grace can never
reach — the test could no longer fail. It now raises the drain grace to
10s for its own duration and asserts under 3s: healthy runs take
0.19-0.44s, and a broken short-circuit takes 10.19s and fails.
- wecom: gofmt the import block in relay_settle_budget_test.go.
Co-authored-by: multica-agent <[email protected]>
---------
Co-authored-by: Sol-Boy <[email protected]>
Co-authored-by: multica-agent <[email protected]>
Co-authored-by: J <[email protected]>
