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

Use tox for testing #177

Open
BenjaminRodenberg opened this issue Aug 20, 2024 · 2 comments
Open

Use tox for testing #177

BenjaminRodenberg opened this issue Aug 20, 2024 · 2 comments

Comments

@BenjaminRodenberg
Copy link
Member

We currently use python3 setup.py test which is already deprecated for quite some time. I, therefore, get the following warning when running tests via python3 setup.py test:

WARNING: Testing via this command is deprecated and will be removed in a future version. Users looking for a generic test entry point independent of test runner are encouraged to use tox.
@BenjaminRodenberg
Copy link
Member Author

I will probably start working on this issue with a student visiting our chair end of March

@BenjaminRodenberg
Copy link
Member Author

Note that for newer versions of setuptools the command python3 setup.py test actually is not supported anymore. After installing setuptools==72 via pip3 install setuptools==72 I get the following:

fenics-adapter$ python3 setup.py test
...
error: invalid command 'test'

Downgrading to setuptools==71.1.0 via pip3 install setuptools==71.1.0 fixes the issue and python3 setup.py test works as expected.

This clearly shows that we should move from python3 setup.py test to something more modern.

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

No branches or pull requests

1 participant