Repository navigation
Commit 6eb5c4f
authored
✨ feat(packaging): declare tox.pytest deps via a testing extra (#3940)
The `tox.pytest` module exposes a pytest plugin for testing tox itself
and tox plugins, but its runtime imports of `pytest`, `pytest-mock` and
`devpi-process` were never declared in the package metadata. 🐛 Plugin
authors who wired `pytest_plugins = "tox.pytest"` into their
`conftest.py` had to discover and pin those transitive dependencies by
hand, and would otherwise hit an `ImportError` on `devpi_process` before
any test could run, as reported in #3938.
The fix exposes them through a new `testing` optional dependency, so
consumers can pull the full fixture surface in via `tox[testing]` while
the base install stays free of test-only packages. ✨ The version floors
mirror those already in the `test` dependency group used by tox itself,
keeping a single source of truth for the supported lower bounds.
Plugin-author docs in `docs/plugin/howto.rst` gain a short `Testing
plugins` section pointing at the new extra and showing the
`pytest_plugins` hookup, so the workflow is discoverable from the plugin
guide.
Closes #3938.1 parent 1ad47dd commit 6eb5c4f
7 files changed
Lines changed: 47 additions & 0 deletions
File tree
- docs
- changelog
- plugin
- tests/session/cmd
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
153 | 153 | | |
154 | 154 | | |
155 | 155 | | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
200 | 200 | | |
201 | 201 | | |
202 | 202 | | |
| 203 | + | |
| 204 | + | |
| 205 | + | |
| 206 | + | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
| 210 | + | |
| 211 | + | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
203 | 224 | | |
204 | 225 | | |
205 | 226 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
65 | 65 | | |
66 | 66 | | |
67 | 67 | | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
68 | 73 | | |
69 | 74 | | |
70 | 75 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
535 | 535 | | |
536 | 536 | | |
537 | 537 | | |
| 538 | + | |
538 | 539 | | |
539 | 540 | | |
540 | 541 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
70 | 70 | | |
71 | 71 | | |
72 | 72 | | |
| 73 | + | |
73 | 74 | | |
74 | 75 | | |
75 | 76 | | |
| |||
0 commit comments