search

pytest-dev / apipkg

MIT License
55 stars 16 forks source link

test_get_distribution_version is broken with pytest 7.2 and tox 4 #43

Closed frenzymadness closed 1 year ago

frenzymadness commented 1 year ago

This assert

https://github.com/pytest-dev/apipkg/blob/70a9a036bdefad85ffa628315c4e134199eb5f00/test_apipkg.py#L835

counts that py will always be installed but pytest 7.2 and tox 4 no longer depend on py so it might not be available which makes this test to fail.

mgorny commented 1 year ago

Making assertion on setuptools also doesn't sound future-proof for a PEP517 world where build requirements are only installed in a temporary venv.