-
Notifications
You must be signed in to change notification settings - Fork 21
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
Refactor tests to use pytest & remove warnings #48
Conversation
eleftherioszisis
commented
Mar 4, 2022
•
edited
edited
- Replace nose with pytest
- Fix warnings
- Deprecated python3.6
- Add all supported pythons
a1b864e
to
66b4a5f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, thanks!
Just a remark: what about using some fixtures for input data? It would be cleaner because currently if a test update an input data it will be also changed for the following tests.
That would be too much work. I just wanted to bring it up to shape and remove the annoying warnings, not to rewrite all the tests :P |
I understand 😄 |