Skip to content

test(asgi): Remove transaction tests - #7316

Merged
sentrivana merged 1 commit into
major/3.0from
ivana/major/remove-transactions-asgi
Aug 31, 2026
Merged

test(asgi): Remove transaction tests#7316
sentrivana merged 1 commit into
major/3.0from
ivana/major/remove-transactions-asgi

Conversation

@sentrivana

@sentrivana sentrivana commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Looks like the asgi integration itself doesn't have any span streaming branching, only the tests do.

Closes https://linear.app/getsentry/issue/PY-2679/remove-transaction-based-tracing-from-asgi

@linear-code

linear-code Bot commented Aug 31, 2026

Copy link
Copy Markdown

PY-2679

@sentrivana
sentrivana marked this pull request as ready for review August 31, 2026 12:09
@sentrivana
sentrivana requested a review from a team as a code owner August 31, 2026 12:09
@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

107780 passed | ⏭️ 5338 skipped | Total: 113118 | Pass Rate: 95.28% | Execution Time: 342m 25s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -12704
Passed Tests 📉 -11280
Failed Tests
Skipped Tests 📉 -1424

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    90.30%    90.55%    +0.25%
==========================================
  Files          193       185        -8
  Lines        25601     23705     -1896
  Branches      9436      8734      -702
==========================================
+ Hits         23116     21464     -1652
- Misses        2485      2241      -244
- Partials      1435      1365       -70

Generated by Codecov Action

@sentrivana
sentrivana merged commit 140d789 into major/3.0 Aug 31, 2026
129 checks passed
@sentrivana
sentrivana deleted the ivana/major/remove-transactions-asgi branch August 31, 2026 12:30
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