Skip to content

test(node): Unflake captureMessage with_level test - #25358

Merged
mydea merged 1 commit into
developfrom
test/unflake-capture-message-with-level
Oct 14, 2026
Merged

mydea merged 1 commit into
developfrom
test/unflake-capture-message-with-level

Conversation

@mydea

@mydea mydea commented Oct 14, 2026

Copy link
Copy Markdown
Member

The with_level scenario sends six captureMessage calls back to back, and the test expected their events to arrive in call order. Events are prepared asynchronously before being sent, so the SDK doesn't guarantee that order, and the test occasionally failed with info_message arriving before debug_message. The test now uses .unordered(), so it still asserts all six messages with their levels but no longer depends on arrival order.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mydea
mydea marked this pull request as ready for review October 14, 2026 11:17
@mydea
mydea requested a review from a team as a code owner October 14, 2026 11:17
@mydea
mydea requested review from JPeer264 and isaacs and removed request for a team October 14, 2026 11:17
@mydea
mydea merged commit b18abae into develop Oct 14, 2026
108 of 110 checks passed
@mydea
mydea deleted the test/unflake-capture-message-with-level branch October 14, 2026 13:02
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.

2 participants