-
Notifications
You must be signed in to change notification settings - Fork 26
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: Add unit tests for uncovered function #645
Conversation
Signed-off-by: Oleg Kopysov <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #645 +/- ##
============================================
+ Coverage 93.89% 94.39% +0.49%
- Complexity 640 647 +7
============================================
Files 52 52
Lines 2195 2193 -2
Branches 254 253 -1
============================================
+ Hits 2061 2070 +9
+ Misses 58 53 -5
+ Partials 76 70 -6 ☔ View full report in Codecov by Sentry. |
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.
Approved.
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.
Added unit tests. Approved.
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.
LGTM
Pull Request
Description
Current PR contains small fixes in the source code and adds new unit tests for functions that are not covered by unit tests.
Type of change
Please delete options that are not relevant.
Testing
Local run.
Checklist: