Skip to content

Conversation

@szokeasaurusrex
Copy link
Member

Add a test which demonstrates how unfinished spans are handled. We will use this to compare behavior with potel-base


Thank you for contributing to sentry-python! Please add tests to validate your changes, and lint your code using tox -e linters.

Running the test suite on your PR might require maintainer approval.

Add a test which demonstrates how unfinished spans are handled. We will use this to compare behavior with `potel-base`
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/test-unfinished-spans branch from d31749c to 3f40de3 Compare July 3, 2025 12:25
@codecov
Copy link

codecov bot commented Jul 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.63%. Comparing base (9e1cedd) to head (3f40de3).
⚠️ Report is 130 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4546      +/-   ##
==========================================
- Coverage   80.63%   80.63%   -0.01%     
==========================================
  Files         156      156              
  Lines       16482    16482              
  Branches     2802     2802              
==========================================
- Hits        13291    13290       -1     
+ Misses       2308     2307       -1     
- Partials      883      885       +2     

see 5 files with indirect coverage changes

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