Skip to content

test: make client-stream time-budget emission tests deterministic - #392

Merged
SunSi12138 merged 2 commits into
devfrom
fix/387-client-stream-time-budget-race
Aug 29, 2026
Merged

test: make client-stream time-budget emission tests deterministic#392
SunSi12138 merged 2 commits into
devfrom
fix/387-client-stream-time-budget-race

Conversation

@SunSi12138

Copy link
Copy Markdown
Owner

Fixes the client-stream time-budget synchronization flakes tracked in #387.

  • drain ConnectAsync send work before arming the one-shot output hook
  • advance ManualTimeProvider at the target Request output-buffer acquisition
  • remove the request-registration vs clock-advance race
  • no retries or timeout expansion

Observed while validating #386.

@SunSi12138
SunSi12138 merged commit 3821a59 into dev Aug 29, 2026
25 checks passed
@SunSi12138
SunSi12138 deleted the fix/387-client-stream-time-budget-race branch August 29, 2026 08:35
SunSi12138 added a commit that referenced this pull request Aug 29, 2026
Sync #386 with current dev, including #392 time-budget test stabilization.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant