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

sdist is missing tests #287

Closed
mtelka opened this issue Nov 9, 2023 · 7 comments
Closed

sdist is missing tests #287

mtelka opened this issue Nov 9, 2023 · 7 comments

Comments

@mtelka
Copy link

mtelka commented Nov 9, 2023

The sdist package at PyPI is missing tests. Please add tests to sdist to make downstream testing easier. Thank you.

@dbatten5
Copy link
Owner

dbatten5 commented Nov 9, 2023

Hi @mtelka thanks for raising an issue. I'm hesitant to add tests to the package on PyPI, from reading around a bit it doesn't seem like standard practice. The tests are freely available on this repository though so you're more than welcome to download them from here.

@mtelka
Copy link
Author

mtelka commented Nov 9, 2023

Yes, there are two sets of PyPI projects. Projects in the first set adds tests to sdist and their developers fix all bugs related to tests (for example wcwidth or virtualenv, just to name few). The other sets of projects refuses to add tests to sdists.

Since I'm packaging many Python projects for OpenIndiana I do have pretty good overview of both sets of projects. The first one is far bigger: more than 80% of Python projects we packaged includes tests in sdist.

Yes, I can use github tarball to get tests, but there are two issues with such approach:

  • It needs a bit more manual work for us.
  • Github tarballs could change in future and this affects build reproducibility (again, more manual work is needed).

Thank you.

@dbatten5
Copy link
Owner

dbatten5 commented Nov 9, 2023

Ok will try and get the tests added to sdist

@dbatten5
Copy link
Owner

dbatten5 commented Nov 9, 2023

Ok I've published a test version with (hopefully) the tests included here https://test.pypi.org/project/maison/
Could you give that a try @mtelka ?

@mtelka
Copy link
Author

mtelka commented Nov 9, 2023

I tested the 1.4.2.dev1699547871 sdist and I'm getting the same test results as with https://github.com/dbatten5/maison/archive/refs/tags/v1.4.1.tar.gz. IOW, all tests are properly added to sdist.

Thank you!

@mtelka mtelka closed this as completed Nov 9, 2023
@dbatten5
Copy link
Owner

Hi @mtelka, i've just published a production version (1.4.2) with the above changes. Let me know if anything doesn't look right

@mtelka
Copy link
Author

mtelka commented Nov 10, 2023

Everything works well. I was able to drop two manual configuration tweaks in our packaging: OpenIndiana/oi-userland#14775

Thanks.

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

No branches or pull requests

2 participants