Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix common tests in Potel #3864

Merged
merged 1 commit into from
Dec 6, 2024
Merged

Conversation

antonpirker
Copy link
Member

No description provided.

Copy link

codecov bot commented Dec 6, 2024

❌ 277 Tests Failed:

Tests completed Failed Passed Skipped
12694 277 12417 3318
View the top 3 failed tests by shortest run time
tests.integrations.celery.test_celery test_baggage_propagation
Stack Traces | 0.064s run time
.../integrations/celery/test_celery.py:519: in test_baggage_propagation
    assert sorted(result["baggage"].split(",")) == sorted(
E   AssertionError: assert ['custom=value', 'sentry-environment=production', 'sentry-release=abcdef', 'sentry-sample_rate=1.0', 'sentry-sampled=true', 'sentry-trace_id=e935a436eea4b7911779b76e6fe826c8', 'sentry-transaction=%3Cunlabeled%20span%3E'] == ['custom=value', 'sentry-environment=production', 'sentry-release=abcdef', 'sentry-sample_rate=1.0', 'sentry-sampled=true', 'sentry-trace_id=e935a436eea4b7911779b76e6fe826c8']
E     
E     Left contains one more item: 'sentry-transaction=%3Cunlabeled%20span%3E'
E     
E     Full diff:
E       [
E           'custom=value',
E           'sentry-environment=production',
E           'sentry-release=abcdef',
E           'sentry-sample_rate=1.0',
E           'sentry-sampled=true',
E           'sentry-trace_id=e935a436eea4b7911779b76e6fe826c8',
E     +     'sentry-transaction=%3Cunlabeled%20span%3E',
E       ]
tests.integrations.celery.test_celery test_baggage_propagation
Stack Traces | 0.065s run time
.../integrations/celery/test_celery.py:519: in test_baggage_propagation
    assert sorted(result["baggage"].split(",")) == sorted(
E   AssertionError: assert ['custom=value', 'sentry-environment=production', 'sentry-release=abcdef', 'sentry-sample_rate=1.0', 'sentry-sampled=true', 'sentry-trace_id=24b0e83041172c9a7e48fe2a227e16fa', 'sentry-transaction=%3Cunlabeled%20span%3E'] == ['custom=value', 'sentry-environment=production', 'sentry-release=abcdef', 'sentry-sample_rate=1.0', 'sentry-sampled=true', 'sentry-trace_id=24b0e83041172c9a7e48fe2a227e16fa']
E     
E     Left contains one more item: 'sentry-transaction=%3Cunlabeled%20span%3E'
E     
E     Full diff:
E       [
E           'custom=value',
E           'sentry-environment=production',
E           'sentry-release=abcdef',
E           'sentry-sample_rate=1.0',
E           'sentry-sampled=true',
E           'sentry-trace_id=24b0e83041172c9a7e48fe2a227e16fa',
E     +     'sentry-transaction=%3Cunlabeled%20span%3E',
E       ]
tests.integrations.celery.test_celery test_sentry_propagate_traces_override
Stack Traces | 0.067s run time
.../integrations/celery/test_celery.py:559: in test_sentry_propagate_traces_override
    assert transaction_trace_id != task_transaction_id
E   AssertionError: assert 'f745179b14b02e872fa3b7fa09253493' != 'f745179b14b02e872fa3b7fa09253493'

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

@antonpirker antonpirker marked this pull request as ready for review December 6, 2024 14:52
@antonpirker antonpirker merged commit a545ec0 into potel-base Dec 6, 2024
77 of 125 checks passed
@antonpirker antonpirker deleted the antonpirker/potel/common branch December 6, 2024 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants