-
Notifications
You must be signed in to change notification settings - Fork 1
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
Test full chain in multiple chunks and clean up input tests #184
Conversation
Pull Request Test Coverage Report for Build 8750603288Details
💛 - Coveralls |
I will add some more tests to increase the coverage. |
Hi @mhliu0001, what is the status of this PR? |
Hi @HenningSE , I think this can be merged now if you want. My original plan was to add some tests with empty instructions, but for the moment this is not the priority. |
Thanks for the PR @mhliu0001 . I re-enabled the single chunk full chain tests (more tests won't hurt us) and will merge this PR when the checks are successful. We can add more tests like you suggested in future PRs. |
Previously, the tests only simulated one chunk. This PR adds tests for multiple chunks.
Also, this PR rewrites
test_input.py
to capture exceptions more elegantly.