-
Notifications
You must be signed in to change notification settings - Fork 4
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
add integration tests for the CLI #46
Comments
the code should definitely be split up into separate modules such that testing becomes much easier: while it is possible to test CLI, this is really tedious. If, instead, we had a function that does the conversion, one that compiles the report, and ones that parse the CLI args, the result might be much cleaner. We could also use |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now, the test suite contains only unit tests, leaving the code in
__main__.py
untested. That code should be moved to a different module and covered by integration tests.The text was updated successfully, but these errors were encountered: