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.

Sign up for GitHub

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

Jump to bottom

test(tox): Unpin pytest for celery tests #3701

Merged
merged 1 commit into from
Oct 28, 2024
Merged

test(tox): Unpin pytest for celery tests #3701

merged 1 commit into from
Oct 28, 2024

Conversation

szokeasaurusrex
Copy link
Member

@szokeasaurusrex szokeasaurusrex commented Oct 28, 2024 •
edited
Loading

Unpin pytest for Celery tests. This requires adding a placeholder test to workaround a bug with pytest-forked.

ref #3035

@codecov Codecov
Copy link

codecov bot commented Oct 28, 2024 •
edited
Loading

❌ 19 Tests Failed:

Tests completed Failed Passed Skipped
14556 19 14537 4123
View the top 3 failed tests by shortest run time 
tests.integrations.celery.test_celery_beat_crons test_get_headers
Stack Traces | 0.001s run time 
No failure message available

tests.integrations.celery.test_celery_beat_crons test_get_headers
Stack Traces | 0.001s run time 
No failure message available

tests.integrations.celery.test_celery_beat_crons test_get_headers
Stack Traces | 0.001s run time 
No failure message available

To view individual test run time comparison to the main branch, go to the Test Analytics Dashboard

@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/unpin-pytest-celery branch from 2eafc8b to 3be78aa Compare October 28, 2024 13:47
@szokeasaurusrex szokeasaurusrex marked this pull request as ready for review October 28, 2024 14:02
@szokeasaurusrex szokeasaurusrex merged commit b6482f0 into master Oct 28, 2024
136 checks passed
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/unpin-pytest-celery branch October 28, 2024 14:03
This was referenced Oct 29, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sl0thentr0py sl0thentr0py sl0thentr0py approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@szokeasaurusrex @sl0thentr0py