-
Notifications
You must be signed in to change notification settings - Fork 4
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
Fix quantity overflow #146
Fix quantity overflow #146
Conversation
…into fix-quantity-overflow
Codecov Report
@@ Coverage Diff @@
## master #146 +/- ##
============================================
- Coverage 75.09% 75.03% -0.06%
Complexity 864 864
============================================
Files 125 125
Lines 2554 2556 +2
Branches 288 289 +1
============================================
Hits 1918 1918
- Misses 534 535 +1
- Partials 102 103 +1
Continue to review full report at Codecov.
|
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
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 after unittest added
@@ -35,15 +35,15 @@ | |||
public static final String MESSAGE_ITEM_NOT_PROVIDED = "Item name must be provided."; | |||
public static final String MESSAGE_QUANTITY_NOT_PROVIDED = "Quantity must be provided."; | |||
public static final String MESSAGE_NEGATIVE_QUANTITY = "Quantity will become negative."; | |||
|
|||
private static final String MESSAGE_OVERFLOW_QUANTITY = "Quantity will overflow."; |
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.
Needs a test case for this
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.
fixed quantity overflow