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

add integration tests for the CLI #46

Open
keewis opened this issue Sep 28, 2020 · 1 comment
Open

add integration tests for the CLI #46

keewis opened this issue Sep 28, 2020 · 1 comment

Comments

@keewis
Copy link
Owner

keewis commented Sep 28, 2020

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.

@keewis
Copy link
Owner Author

keewis commented Feb 13, 2022

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 hypothesis for this, which might help catch a lot of bugs.

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

1 participant