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

pytest-xdist 3.40 and higher not honoring --exitfirst #1034

Open
burnettk opened this issue Mar 4, 2024 · 1 comment
Open

pytest-xdist 3.40 and higher not honoring --exitfirst #1034

burnettk opened this issue Mar 4, 2024 · 1 comment

Comments

@burnettk
Copy link

burnettk commented Mar 4, 2024

https://stackoverflow.com/q/77667559/6090676

i am not the filer of the stackoverflow question (but i also observed the issue -- works fine on version 3.3.1), and it appears to be the only place this issue is documented, and i was thinking this would be a more appropriate place.

thanks for the awesome project! it made our test suite like 5x faster or something.

@charles-cooper
Copy link

chiming in here as i noticed the same issue. if it helps, here it my pip freeze | grep pytest:

pytest==8.1.1
pytest-cov==4.1.0
pytest-forked==1.6.0
pytest-instafail==0.5.0
pytest-split==0.8.2
pytest-xdist==3.5.0

charles-cooper added a commit to charles-cooper/vyper that referenced this issue Mar 23, 2024
charles-cooper added a commit to vyperlang/vyper that referenced this issue Mar 26, 2024
this commit adds `xfail_strict` to pytest options, and cleans up some
code relating to test and lint setup:
- move pytest coverage-enabling options to CI *only*
- suppress annoying mypy notes
- add `--dist worksteal` to setup.cfg
- pin pytest-xdist due to a regression (pytest-dev/pytest-xdist#1034)
electriclilies pushed a commit to electriclilies/vyper that referenced this issue Apr 27, 2024
this commit adds `xfail_strict` to pytest options, and cleans up some
code relating to test and lint setup:
- move pytest coverage-enabling options to CI *only*
- suppress annoying mypy notes
- add `--dist worksteal` to setup.cfg
- pin pytest-xdist due to a regression (pytest-dev/pytest-xdist#1034)
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

No branches or pull requests

2 participants