Skip to content

test(client): add coverage for SSE edge cases and retry#17

Merged
lostmygithubaccount merged 1 commit intomainfrom
cody/test-sse-edge-cases
Apr 6, 2026
Merged

test(client): add coverage for SSE edge cases and retry#17
lostmygithubaccount merged 1 commit intomainfrom
cody/test-sse-edge-cases

Conversation

@lostmygithubaccount
Copy link
Copy Markdown
Member

Summary

  • Add tests for empty SSE data fields (data: \n\n) — verifies silent skip
  • Add tests for malformed JSON in stream chunks — verifies no crash, no event emitted, and subsequent valid events still parse
  • Add tests for retry_delay — verifies strictly increasing delays and exact jitter formula values

7 new unit tests, all passing.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@lostmygithubaccount lostmygithubaccount enabled auto-merge (squash) April 6, 2026 00:25
@lostmygithubaccount lostmygithubaccount merged commit b0ce068 into main Apr 6, 2026
1 check passed
@lostmygithubaccount lostmygithubaccount deleted the cody/test-sse-edge-cases branch April 6, 2026 00:26
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