-
Notifications
You must be signed in to change notification settings - Fork 16
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
Improve test coverage #56
Conversation
ef48be0
to
eff5729
Compare
Codecov Report
@@ Coverage Diff @@
## master #56 +/- ##
==========================================
+ Coverage 94.81% 97.00% +2.19%
==========================================
Files 4 4
Lines 771 769 -2
==========================================
+ Hits 731 746 +15
+ Misses 40 23 -17
Continue to review full report at Codecov.
|
6db8d48
to
dbafadf
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.
Let's merge this ! 👍
d9e9e58
to
e838db7
Compare
I added some tests to cover some cases that were not tested before. There are still a few lines that are not covered by tests.
Related to #12.