Skip to content

test(e2e): Disable span streaming in e2e test applications#22590

Merged
Lms24 merged 2 commits into
developfrom
test/opt-out-span-streaming-e2e
Jul 24, 2026
Merged

test(e2e): Disable span streaming in e2e test applications#22590
Lms24 merged 2 commits into
developfrom
test/opt-out-span-streaming-e2e

Conversation

@Lms24

@Lms24 Lms24 commented Jul 24, 2026

Copy link
Copy Markdown
Member

Prework for enabling span streaming by default without breaking existing E2E tests.

These test applications explicitly stay on the static trace lifecycle for now. This is the E2E slice of #22577.

Refs #22344

Keep transaction-asserting E2E applications on the static trace lifecycle
while span streaming is enabled by default.

Refs #22344
Co-Authored-By: Cursor <cursoragent@cursor.com>

Co-authored-by: Cursor <cursoragent@cursor.com>
@Lms24
Lms24 force-pushed the test/opt-out-span-streaming-e2e branch from 5c1feff to 23e427a Compare July 24, 2026 11:50

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 23e427a. Configure here.

@Lms24 Lms24 changed the title test(e2e): Disable span streaming in test applications test(e2e): Disable span streaming in e2e test applications Jul 24, 2026
Keep transaction-based E2E suites stable when the default trace lifecycle
changes by explicitly opting indirect SDK initialization paths into static mode.

Co-Authored-By: Cursor <cursoragent@cursor.com>
@Lms24
Lms24 marked this pull request as ready for review July 24, 2026 14:44
@Lms24 Lms24 self-assigned this Jul 24, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this test changed because the new option init option increments the lineno

@Lms24
Lms24 requested review from a team, logaretm, msonnb, nicohrubec and stephanie-anderson and removed request for a team and stephanie-anderson July 24, 2026 14:48
@Lms24
Lms24 merged commit db75f02 into develop Jul 24, 2026
260 checks passed
@Lms24
Lms24 deleted the test/opt-out-span-streaming-e2e branch July 24, 2026 14:55
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