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

Add default testpaths to pytest's config #2082

Closed
vytas7 opened this issue Jun 27, 2022 · 0 comments · Fixed by #2083
Closed

Add default testpaths to pytest's config #2082

vytas7 opened this issue Jun 27, 2022 · 0 comments · Fixed by #2083

Comments

@vytas7
Copy link
Member

vytas7 commented Jun 27, 2022

Some of our users/packagers have expressed a wish to be able to run pytest without any parameters. Otherwise pytest seems to pick all tests, including tutorials and what not.

Add testpaths = tests to our configs.

While at it, maybe we should consolidate these configs inside pyproject.toml, as opposed to setup.cfg?
The pytest docs state that

Usage of setup.cfg is not recommended unless for very simple use cases. .cfg files use a different parser than pytest.ini and tox.ini which might cause hard to track down problems. When possible, it is recommended to use the latter files, or pyproject.toml, to hold your pytest configuration.

@vytas7 vytas7 added this to the Triaged (Non-Breaking Changes) milestone Jun 27, 2022
@vytas7 vytas7 added good first issue Comment on this issue if you'd like to volunteer to work on this. Thanks! needs contributor Comment on this issue if you'd like to volunteer to work on this. Thanks! labels Jun 27, 2022
@CaselIT CaselIT self-assigned this Jun 27, 2022
@vytas7 vytas7 removed good first issue Comment on this issue if you'd like to volunteer to work on this. Thanks! needs contributor Comment on this issue if you'd like to volunteer to work on this. Thanks! labels Jun 27, 2022
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