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

Document that temp dir tweaks should be applied when running Pytest in parallel with Tox #3187

Closed
faph opened this issue Jan 17, 2024 · 5 comments · Fixed by #3192
Closed
Assignees

Comments

@faph
Copy link
Contributor

faph commented Jan 17, 2024

What's the problem this feature will solve?

Parallel runs of Pytest are supported only when the base temporary directory is unique for each concurrent run. For Tox 3, this is documented by way of an example here: https://tox.wiki/en/3.27.0/example/pytest.html#extended-example-change-dir-before-test-and-use-per-virtualenv-tempdir

Describe the solution you'd like

I appreciate that this really is a feature of Pytest and not Tox. However, since Tox provides a unique temporary directory per virtualenv out of the box, it may be helpful to document how this can be used to run Pytest with Tox's --parallel option.

I don't know where within the Tox 4 documentation pages this should be included though...

Alternative Solutions

Do nothing: keep the documentation as is.

Additional context

None.

@faph
Copy link
Contributor Author

faph commented Jan 17, 2024

See also: pytest-dev/pytest#11790

@gaborbernat
Copy link
Member

PR welcome 👍

@faph
Copy link
Contributor Author

faph commented Jan 17, 2024

@gaborbernat Thanks. Any suggestion where about we could document this? Any existing sections/pages?

@gaborbernat
Copy link
Member

Should go inside https://tox.wiki/en/4.12.1/faq.html

@faph
Copy link
Contributor Author

faph commented Jan 17, 2024

Perfect. Let me get something drafted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants