Skip to content

test: Add streaming API tests - #6883

Merged
sentrivana merged 1 commit into
masterfrom
ivana/add-streaming-api-tests
Jul 27, 2026
Merged

sentrivana merged 1 commit into
masterfrom
ivana/add-streaming-api-tests

Conversation

@sentrivana

Copy link
Copy Markdown
Collaborator

Description

Add streaming versions of tests in test_api.py

Issues

Part of #5395

Reminders

@sentrivana
sentrivana marked this pull request as ready for review July 24, 2026 12:15
@sentrivana
sentrivana requested a review from a team as a code owner July 24, 2026 12:15

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 58a01bc. Configure here.

Comment thread tests/test_api.py
@github-actions

github-actions Bot commented Jul 24, 2026 •

Copy link
Copy Markdown
Contributor

Codecov Results 📊

✅ 96635 passed | ⏭️ 6340 skipped | Total: 102975 | Pass Rate: 93.84% | Execution Time: 348m 7s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +200
Passed Tests 📈 +200
Failed Tests 📉 -1
Skipped Tests 📈 +1

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.69%    89.70%    +0.01%
==========================================
  Files          193       193         —
  Lines        24330     24330         —
  Branches      8578      8578         —
==========================================
+ Hits         21820     21823        +3
- Misses        2510      2507        -3
- Partials      1397      1396        -1

Generated by Codecov Action

@sentrivana
sentrivana merged commit ab1f308 into master Jul 27, 2026
271 of 273 checks passed
@sentrivana
sentrivana deleted the ivana/add-streaming-api-tests branch July 27, 2026 10:54
mgaligniana pushed a commit to mgaligniana/sentry-python that referenced this pull request Aug 9, 2026
### Description
Add streaming versions of tests in `test_api.py`

#### Issues
Part of getsentry#5395

#### Reminders
- Please add tests to validate your changes, and lint your code using
`uv run ruff`.
- Add GH Issue ID _&_ Linear ID (if applicable)
- PR title should use [conventional
commit](https://develop.sentry.dev/engineering-practices/commit-messages/#type)
style (`feat:`, `fix:`, `ref:`, `meta:`)
- For external contributors:
[CONTRIBUTING.md](https://github.com/getsentry/sentry-python/blob/master/CONTRIBUTING.md),
[Sentry SDK development docs](https://develop.sentry.dev/sdk/), [Discord
community](https://discord.gg/Ww9hbqr)
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