Skip to content

test: Run tracing tests in stream mode - #7345

Open
sentrivana wants to merge 7 commits into
major/3.0from
ivana/major/adapt-missed-tests
Open

test: Run tracing tests in stream mode#7345
sentrivana wants to merge 7 commits into
major/3.0from
ivana/major/adapt-missed-tests

Conversation

@sentrivana

@sentrivana sentrivana commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Adapt tests of integrations we've already fully ported to span streaming so that they actually use span streaming everytime they define a traces_sampler or traces_sample_rate. Otherwise, they're implicitly using static tracing, and they'll break the moment we remove it.

Note: there are at least two tests in test_threading.py that also need this change but I'll submit them separately because those will need additional changes.

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

117009 passed | ⏭️ 5544 skipped | Total: 122553 | Pass Rate: 95.48% | Execution Time: 396m 35s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -14986
Passed Tests 📉 -13420
Failed Tests
Skipped Tests 📉 -1566

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    90.25%    90.45%     +0.2%
==========================================
  Files          193       185        -8
  Lines        25655     23612     -2043
  Branches      9454      8664      -790
==========================================
+ Hits         23153     21356     -1797
- Misses        2502      2256      -246
- Partials      1435      1373       -62

Generated by Codecov Action

@sentrivana
sentrivana marked this pull request as ready for review September 3, 2026 07:37
@sentrivana
sentrivana requested a review from a team as a code owner September 3, 2026 07:37
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