-
Notifications
You must be signed in to change notification settings - Fork 370
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(p/uint25): Increase Test Coverage for uint256
Package
#2931
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2931 +/- ##
==========================================
+ Coverage 60.90% 60.95% +0.05%
==========================================
Files 564 564
Lines 75273 75273
==========================================
+ Hits 45846 45885 +39
+ Misses 26055 26018 -37
+ Partials 3372 3370 -2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -142,7 +142,8 @@ func (z *Uint) fromDecimal(bs string) error { | |||
for i, mult := range multipliers { | |||
if remaining <= 0 { | |||
return nil // Done | |||
} else if remaining > 19 { | |||
} | |||
if remaining > 19 { |
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.
Was this change requested by lint?
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.
This was just my personal preference. should I revert?
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.
I think so. My guess is that core devs will approve the PR more quickly if it just updates the tests. It's up to you.
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.
good. reverted just in case
Description
Increased the test coverage of the
p/demo/uint256
package. Previously, only about 40% was covered, but now it has increased to 90% (checked in go).The existing implementations of uint256 functions have not been modified.
related with: #2846
BREAKING CHANGE: xxx
message was included in the description