-
Notifications
You must be signed in to change notification settings - Fork 5
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
Branch max quantity tests #88
Branch max quantity tests #88
Conversation
Update from organization repo
PR from organization repo
Organization PR
Fix when toAdd item does not have maxQuantity and exisitingItem does it'll replace existingItem maxQuantity.
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 job on tanking the test cases!
docs/UserGuide.md
Outdated
@@ -102,7 +102,7 @@ Format: `list` | |||
|
|||
Edits an existing item in the Inventory book. | |||
|
|||
Format: ` edit INDEX n/NAME q/QUANTITY s/SUPPLIER [t/TAG]…` | |||
Format: ` edit INDEX n/NAME q/QUANTITY s/SUPPLIER [max/MAX_QUANTITY] [t/TAG]…` |
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 all the arguments, except INDEX, should be optional here
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! thanks for pointing it out!
docs/UserGuide.md
Outdated
|**Clear** | `clear` | | ||
|**Delete** | `delete INDEX`<br> e.g., `delete 3` | | ||
|**Edit** | ` edit INDEX n/NAME q/QUANTITY s/SUPPLIER [t/TAG]…`<br> e.g.,`edit 1 n/Chicken q/50` | | ||
|**Edit** | `edit INDEX n/NAME q/QUANTITY s/SUPPLIER [max/MAX_QUANTITY] [t/TAG]…`<br> e.g.,`edit 1 n/Chicken q/50` | |
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.
Likewise for here
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.
Same! Fixed
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
Integrate max quantity into test cases.
Add new test cases to test max quantity
Fix typical items and related test cases.
Previously DUCK item had POULTRY tagged to it.
Fix bug where adding new item without max quantity tag will delete existing item max quantity tag
Update UG.
DG had nothing to be updated yet