getsentry / sentry-python

The official Python SDK for Sentry.io
https://sentry.io/for/python/
MIT License
1.93k stars 509 forks source link

Fix test_dsc and twp baggage handling #3789

Closed sl0thentr0py closed 4 days ago

codecov[bot] commented 6 days ago

:x: 1198 Tests Failed:

Tests completed Failed Passed Skipped
19893 1198 18695 4413
View the top 1 failed tests by shortest run time > > ```python > tests.integrations.opentelemetry.test_span_processor test_on_start_transaction > ``` > >
Stack Traces | 0.001s run time > > > > > ```python > > .../integrations/opentelemetry/test_span_processor.py:289: in test_on_start_transaction > > fake_start_transaction, > > .../hostedtoolcache/Python/3.7.17....../x64/lib/python3.7/unittest/mock.py:1307: in __enter__ > > original, local = self.get_original() > > .../hostedtoolcache/Python/3.7.17....../x64/lib/python3.7/unittest/mock.py:1281: in get_original > > "%s does not have the attribute %r" % (target, name) > > E AttributeError: does not have the attribute 'start_transaction' > > ``` > >

View the full list of 2 :snowflake: flaky tests > > ```python > tests.test_scope test_with_isolation_scope_data > ``` > > **Flake rate in main:** 99.17% (Passed 2 times, Failed 239 times) >
Stack Traces | 0.001s run time > > > > > ```python > > tests/test_scope.py:387: in test_with_isolation_scope_data > > assert scope._tags == {"before_isolation_scope": 1} > > E AssertionError: assert {} == {'before_isolation_scope': 1} > > E > > E Right contains 1 more item: > > E {'before_isolation_scope': 1} > > E > > E Full diff: > > E + {} > > E - { > > E - 'before_isolation_scope': 1, > > E - } > > ``` > >
tests.test_scope test_with_new_scope

Flake rate in main: 97.03% (Passed 8 times, Failed 261 times)

Stack Traces | 0.001s run time > > ```python > tests/test_scope.py:587: in test_with_new_scope > assert scope is in_with_current_scope > E assert is > ```

To view more test analytics, go to the Test Analytics Dashboard Got feedback? Let us know on Github