Skip to content

test(django): Add tests for data_collection gating of request body - #7213

Open
ericapisani wants to merge 1 commit into
py-2419-request-body-flaskfrom
py-2419-request-body-django
Open

test(django): Add tests for data_collection gating of request body#7213
ericapisani wants to merge 1 commit into
py-2419-request-body-flaskfrom
py-2419-request-body-django

Conversation

@ericapisani

@ericapisani ericapisani commented Aug 14, 2026

Copy link
Copy Markdown
Member

Django uses same underlying request extractor as Flask, so we just need to add test coverage to ensure the request bodies are gated correctly

Refs PY-2419
Refs #6283

@ericapisani
ericapisani requested a review from a team as a code owner August 14, 2026 20:57
@linear-code

linear-code Bot commented Aug 14, 2026

Copy link
Copy Markdown

PY-2419

@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

113689 passed | ⏭️ 6804 skipped | Total: 120493 | Pass Rate: 94.35% | Execution Time: 418m 21s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +570
Passed Tests 📈 +532
Failed Tests 📉 -1
Skipped Tests 📈 +39

➖ Removed Tests (1)

View removed tests
  • test_span_streaming_quart_auth_user_id[42-True]
    • File: tests.integrations.quart.test_quart

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    90.21%    90.21%        —%
==========================================
  Files          193       193         —
  Lines        25423     25426        +3
  Branches      9336      9338        +2
==========================================
+ Hits         22933     22936        +3
- Misses        2490      2490         —
- Partials      1436      1437        +1

Generated by Codecov Action

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