-
Notifications
You must be signed in to change notification settings - Fork 23
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
Testing + code coverage #98
Conversation
…used to throw exceptions.
Codecov Report
@@ Coverage Diff @@
## dev #98 +/- ##
======================================
Coverage ? 64.46%
======================================
Files ? 9
Lines ? 515
Branches ? 0
======================================
Hits ? 332
Misses ? 183
Partials ? 0
Continue to review full report at Codecov.
|
Tests sometimes fail due to memory on travis ( #99 ) , reloading the build seems to be a doable workaround for now, |
@error
byerror(msg)
. The former is used for logging, whereas the later throws an Exception. We want the latter.In #91, there are several TODOs concerning which functions still need test coverage. Let's implement them after this PR.
Merging this PR right away, for a reproducible version of the changes and for your comments @YoungFaithful , use #97 .