Skip to content

test: Add streaming logs tests#6880

Open
sentrivana wants to merge 1 commit into
masterfrom
ivana/add-streaming-log-tests
Open

test: Add streaming logs tests#6880
sentrivana wants to merge 1 commit into
masterfrom
ivana/add-streaming-log-tests

Conversation

@sentrivana

Copy link
Copy Markdown
Contributor

Description

Adding streaming versions of relevant logs tests.

Issues

Part of #5395

Reminders

@sentrivana
sentrivana marked this pull request as ready for review July 24, 2026 11:44
@sentrivana
sentrivana requested a review from a team as a code owner July 24, 2026 11:44
@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

96526 passed | ⏭️ 6344 skipped | Total: 102870 | Pass Rate: 93.83% | Execution Time: 344m 17s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +95
Passed Tests 📈 +91
Failed Tests 📉 -1
Skipped Tests 📈 +5

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.68%    89.68%        —%
==========================================
  Files          193       193         —
  Lines        24330     24330         —
  Branches      8578      8578         —
==========================================
+ Hits         21820     21820         —
- Misses        2510      2510         —
- Partials      1397      1397         —

Generated by Codecov Action

Comment thread tests/test_logs.py
sentry_init(enable_logs=True, traces_sample_rate=1.0, trace_lifecycle="stream")
items = capture_items("log")

with sentry_sdk.traces.start_span(name="test-transaction"):

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this maybe be called test-segment?

Suggested change
with sentry_sdk.traces.start_span(name="test-transaction"):
with sentry_sdk.traces.start_span(name="test-segment"):

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