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

Support external tox.pytest usage via "test" extra #3422

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

mbra
Copy link
Contributor

@mbra mbra commented Oct 22, 2024

In order to allow plugin writers to easily pull in all dependencies needed to use the tox.pytest module, add a new test extra with a subset of the test dependency group. Though the pytest package should be present in the users' environment anyway, we still add it for consistency and to produce a conflict just in case the version has been constrained to an older version by the project.

Fixes #3415

  • ran the linter to address style issues (tox -e fix)
  • wrote descriptive pull request text
  • ensured there are test(s) validating the fix
  • added news fragment in docs/changelog folder
  • updated/extended the documentation

Not sure how to test this without splitting off the plugin tests into an extra test-suite, which seems like quite the invasive change.
Also did not find a good place to update this in the docs without starting a whole new section on testing plugins with pytest.

In order to allow plugin writers to easily pull in all dependencies
needed to use the `tox.pytest` module, add a new `test` extra with a
subset of the test dependency group. Though the pytest package should
be present in the users' environment anyway, we still add it for
consistency and to produce a conflict just in case the version has
been constrained to an older version by the project.
@gaborbernat gaborbernat merged commit f0799ac into tox-dev:main Oct 22, 2024
28 checks passed
CyclingNinja added a commit to CyclingNinja/package-template that referenced this pull request Oct 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Please reinstate "testing" extra
2 participants