Skip to content

tests: Move streaming decorator tests#6695

Merged
sentrivana merged 1 commit into
masterfrom
ivana/move-streaming-deco-tests
Jun 30, 2026
Merged

tests: Move streaming decorator tests#6695
sentrivana merged 1 commit into
masterfrom
ivana/move-streaming-deco-tests

Conversation

@sentrivana

@sentrivana sentrivana commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Description

Moving streaming @trace decorator tests from test_span_streaming to test_decorator and adapting existing tests to also test the streamed variant if applicable.

Issues

Reminders

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

github-actions Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

89922 passed | ⏭️ 6288 skipped | Total: 96210 | Pass Rate: 93.46% | Execution Time: 311m 43s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +71
Passed Tests 📈 +80
Failed Tests
Skipped Tests 📉 -9

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.74%    89.77%    +0.03%
==========================================
  Files          192       192         —
  Lines        23809     23809         —
  Branches      8218      8218         —
==========================================
+ Hits         21367     21374        +7
- Misses        2442      2435        -7
- Partials      1347      1347         —

Generated by Codecov Action

@sentrivana sentrivana merged commit 4abb01f into master Jun 30, 2026
272 of 274 checks passed
@sentrivana sentrivana deleted the ivana/move-streaming-deco-tests branch June 30, 2026 11:38
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