You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Follow-up from PR #136 review. Tracks test coverage gaps not blocking merge but worth filling before Phase 7 burn-in.
Gaps
Phase 3 cross-enclave-peer fanout — TestWSPutStoresLocallyAndAcks covers the local-store side of a relayed PUT but not the criterion "Transient PUT shows up in all enclave-peer stores within gossip latency." Needs a 2-substrate test fixture.
Phase 5 receive path multi-node variants — "two-substrate, transient on each" path; "transient A writes, transient B sees it" path. Both stop short of being tested end-to-end.
--mcp multi-seed fallback — cmd/repram/main.go:281-310 loops through peers and gives up after all fail. No automated test; would need an observability hook (returned error / metric / log capture).
Heartbeat at production cadence (30s) — unit tests use 30ms; the 5-min idle acceptance is implicit. Phase 7 burn-in covers this.
Follow-up from PR #136 review. Tracks test coverage gaps not blocking merge but worth filling before Phase 7 burn-in.
Gaps
Phase 3 cross-enclave-peer fanout —
TestWSPutStoresLocallyAndAckscovers the local-store side of a relayed PUT but not the criterion "Transient PUT shows up in all enclave-peer stores within gossip latency." Needs a 2-substrate test fixture.Phase 5 receive path multi-node variants — "two-substrate, transient on each" path; "transient A writes, transient B sees it" path. Both stop short of being tested end-to-end.
TCP-RST acceptance criterion — issue mcp: port WS transport + tree manager to Go (Option A — restore NAT-traversal / transient participation) #135 calls out "Forced TCP RST surfaces a clean disconnect within 30s." Currently only graceful
Close()is tested.--mcpmulti-seed fallback —cmd/repram/main.go:281-310loops through peers and gives up after all fail. No automated test; would need an observability hook (returned error / metric / log capture).Heartbeat at production cadence (30s) — unit tests use 30ms; the 5-min idle acceptance is implicit. Phase 7 burn-in covers this.
Related: #131 (Phase 4 multi-node quorum_status="pending" test).
Acceptance
// ticktockbent