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

-n auto --inline-snashot=disable should not raise an Error #90

Closed
15r10nk opened this issue May 26, 2024 · 0 comments · Fixed by #93
Closed

-n auto --inline-snashot=disable should not raise an Error #90

15r10nk opened this issue May 26, 2024 · 0 comments · Fixed by #93

Comments

@15r10nk
Copy link
Owner

15r10nk commented May 26, 2024

$ pytest tests --color yes -c config/pytest.ini -n auto --inline-snapshot=disable
ERROR: inline-snapshot can not be combined with xdist

Since inline-snapshot is disabled, maybe it shouldn't error out because xdist is enabled?

Also at the end of the run it says:

INFO: inline-snapshot was disabled because you used xdist

But that's not really true. I disabled it explicitly, and also didn't enable xdist explicitly because inline-snapshot wouldn't let me 😉

Originally posted by @pawamoy in #89 (reply in thread)

@15r10nk 15r10nk changed the title -n auto --inline-snashot=disable should raise no Error -n auto --inline-snashot=disable should not raise an Error May 26, 2024
15r10nk added a commit that referenced this issue May 28, 2024
fix: changed how --inline-snapshot=disable works with xdist (#90)
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 a pull request may close this issue.

1 participant