fair-research / native-login

Provides Native App login and token storage for multiple providers
Apache License 2.0
3 stars 2 forks source link

Wheel distribution contains extraneous files #56

Closed NickolausDS closed 3 years ago

NickolausDS commented 3 years ago

The wheel distribution contains all unit tests contained in the repo when running the following:

python setup.py bdist_wheel

Tests are NOT contained in the source distribution. The wheel seems to ignore the prune line in the MANIFEST.in file when packaging things. Removing the __init__.py under tests/ fixed this (but caused pytest to fail), and additionally replacing find_packages() in setup.py with fair_research_login also fixed this (but failed to load critical subdir modules such as token_storage/). Some permutation of settings will likely fix both, perhaps by adding an explicit line for test modules to setup.cfg for pytest while removing the __init__.py in tests.

Related to #46