Skip to content

test(clickhouse-driver, cohere, falcon, fastapi, fastmcp): Remove redundant empty data_collection from sentry_init - #7907

Open
ericapisani wants to merge 2 commits into
py-2936-asgi-asyncio-awslambdafrom
py-2936-clickhouse-driver-cohere-falcon
Open

ericapisani wants to merge 2 commits into
py-2936-asgi-asyncio-awslambdafrom
py-2936-clickhouse-driver-cohere-falcon

Conversation

@ericapisani

@ericapisani ericapisani commented Oct 6, 2026 •

Copy link
Copy Markdown
Member

Refs PY-2936

…undant empty `data_collection` from sentry_init

Also covers google_genai, graphene, httpx2, huggingface_hub and langchain.
An empty `data_collection={}` is equivalent to the default, so the
argument is dropped from `sentry_init` in these tests.

Refs PY-2936
@ericapisani
ericapisani added this pull request to stack #7897 October 6, 2026 19:29
@linear-code

linear-code Bot commented Oct 6, 2026

Copy link
Copy Markdown

PY-2936

@ericapisani
ericapisani marked this pull request as ready for review October 6, 2026 19:30
@ericapisani
ericapisani requested a review from a team as a code owner October 6, 2026 19:30
@github-actions

github-actions Bot commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

✅ 55349 passed | ⏭️ 2727 skipped | Total: 58076 | Pass Rate: 95.3% | Execution Time: 153m 49s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -82309
Passed Tests 📉 -77813
Failed Tests 📉 -1
Skipped Tests 📉 -4495

➖ Removed Tests (1)

View removed tests
  • test_cache_spans_decorator[True]
    • File: tests.integrations.django.test_cache_module

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2061 uncovered lines.
❌ Project coverage is 90.03%. Comparing base (0a88b1d) to head (5ef4564).

Coverage diff
@@            Coverage Diff             @@
##        master       #PR       +/-##
==========================================
- Coverage    90.38%    90.03%    -0.35%
==========================================
  Files          202       185       -17
  Lines        26552     20672     -5880
  Branches      9858      7116     -2742
==========================================
+ Hits         23997     18611     -5386
- Misses        2555      2061      -494
- Partials      1487      1216      -271

Generated by Codecov Action

This branch has not been deployed

No deployments
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