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 test/test_fixtures #272

Open
mtelka opened this issue Jul 19, 2023 · 4 comments
Open

sdist is missing test/test_fixtures #272

mtelka opened this issue Jul 19, 2023 · 4 comments

Comments

@mtelka
Copy link

mtelka commented Jul 19, 2023

The sdist package at PyPI is missing test/test_fixtures. Without test/test_fixtures testing fails. Please add the missing directory to sdist. Thank you.

@Kriechi
Copy link
Member

Kriechi commented Jul 20, 2023

We actually explicitly removed them some time ago, including the integration tests: 09dcf5a

Which tests do you see failing?

@mtelka
Copy link
Author

mtelka commented Jul 20, 2023

Testing is unable to start at all when started via tox:

py39: commands[0]> pytest --cov-report=xml --cov-report=term --cov=hpack
ImportError while loading conftest '$(BUILD_DIR)/test/conftest.py'.
test/conftest.py:19: in <module>
    for d in os.listdir('test/test_fixtures')
E   FileNotFoundError: [Errno 2] No such file or directory: 'test/test_fixtures'

@Kriechi
Copy link
Member

Kriechi commented Jul 20, 2023

Thanks - I think this makes sense.
We removed the test file itself (https://github.com/python-hyper/hpack/blob/master/MANIFEST.in#L9), but not the conftest which configures the fixtures and tests.

Please delete the conftest.py and try again. All other tests should now run cleanly.
If this works, we can add this exclusion to the manifest as well.

@mtelka
Copy link
Author

mtelka commented Jul 20, 2023

With the conftest.py file removed all regular tests pass, but coverage fails:

Name                                                            Stmts   Miss Branch BrPart  Cover   Missing
-----------------------------------------------------------------------------------------------------------
/usr/lib/python3.9/vendor-packages/hpack/__init__.py                6      0      0      0   100%
/usr/lib/python3.9/vendor-packages/hpack/exceptions.py             11      0      0      0   100%
/usr/lib/python3.9/vendor-packages/hpack/hpack.py                 246      0     88      0   100%
/usr/lib/python3.9/vendor-packages/hpack/huffman.py                27      3      6      2    85%   27, 63-64
/usr/lib/python3.9/vendor-packages/hpack/huffman_constants.py       3      0      0      0   100%
/usr/lib/python3.9/vendor-packages/hpack/huffman_table.py          29      0     14      0   100%
/usr/lib/python3.9/vendor-packages/hpack/struct.py                  9      0      0      0   100%
/usr/lib/python3.9/vendor-packages/hpack/table.py                  80      0     28      1    99%   188->184
-----------------------------------------------------------------------------------------------------------
TOTAL                                                             411      3    136      3    99%
Coverage XML written to file coverage.xml

FAIL Required test coverage of 100.0% not reached. Total coverage: 98.90%

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