Skip to content

Conversation

@MackinnonBuck
Copy link
Collaborator

Various fixes for Client_CanResumePostResponseStream_AfterDisconnection:

  1. Make the test wait to receive the first byte from the response stream before allowing the test server to send notifications
    • This prevents the case where the server sends a notification before StreamableHttpHandler takes over the response stream, causing the message to be dropped.
  2. Update StreamableHttpHandler to wait to flush the response stream until after it has a chance to write the priming event
    • This just makes testing easier because we know the server is ready to send notifications after the client receives the first byte.
  3. Add timeouts to each awaited test TCS so that tests fail with a stack trace rather than crash

@MackinnonBuck MackinnonBuck merged commit 86bc96d into main Jan 27, 2026
15 of 17 checks passed
@MackinnonBuck MackinnonBuck deleted the mbuck/fix-flaky-tests branch January 27, 2026 19:41
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.

3 participants