Skip to content

test: Add span origin span streaming tests#6697

Merged
sentrivana merged 1 commit into
masterfrom
ivana/add-span-origin-streaming-tests
Jun 30, 2026
Merged

test: Add span origin span streaming tests#6697
sentrivana merged 1 commit into
masterfrom
ivana/add-span-origin-streaming-tests

Conversation

@sentrivana

Copy link
Copy Markdown
Contributor

Description

Span origin is just a regular attribute in span streaming, so it might not actually need separate tests. However, the case where we check that it's set to manual by default makes sense to cover, so added streaming counterparts to the two existing origin tests.

Issues

Part of #5395

Reminders

@sentrivana sentrivana marked this pull request as ready for review June 30, 2026 12:53
@sentrivana sentrivana requested a review from a team as a code owner June 30, 2026 12:53
@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

89952 passed | ⏭️ 6288 skipped | Total: 96240 | Pass Rate: 93.47% | Execution Time: 312m 44s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +30
Passed Tests 📈 +30
Failed Tests
Skipped Tests

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2435 uncovered lines.
✅ Project coverage is 89.77%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.77%    89.77%        —%
==========================================
  Files          192       192         —
  Lines        23809     23809         —
  Branches      8218      8218         —
==========================================
+ Hits         21374     21374         —
- Misses        2435      2435         —
- Partials      1347      1347         —

Generated by Codecov Action

@sentrivana sentrivana merged commit 9dc9857 into master Jun 30, 2026
146 checks passed
@sentrivana sentrivana deleted the ivana/add-span-origin-streaming-tests branch June 30, 2026 12:58
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