-
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
Add enhancements to UG and DG #199
Add enhancements to UG and DG #199
Conversation
Codecov Report
@@ Coverage Diff @@
## master #199 +/- ##
=========================================
Coverage 74.27% 74.27%
Complexity 528 528
=========================================
Files 83 83
Lines 1555 1555
Branches 154 154
=========================================
Hits 1155 1155
Misses 340 340
Partials 60 60
Continue to review full report at Codecov.
|
docs/DeveloperGuide.md
Outdated
@@ -91,7 +91,7 @@ The `UI` component, | |||
|
|||
1. `Logic` uses the `AddressBookParser` class to parse the user command. | |||
1. This results in a `Command` object which is executed by the `LogicManager`. | |||
1. The command execution can affect the `Model` (e.g. adding a person). | |||
1. The command execution can affect the `Model` (e.g. adding a animal). |
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.
"an animal"
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
docs/DeveloperGuide.md
Outdated
|
||
1. Other incorrect delete commands to try: `delete`, `delete x`, `...` (where x is larger than the list size)<br> | ||
1. Other incorrect delete commands to try: `delete`, `delete x`, `...` (where x is smaller than the requisite size)<br> |
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.
Think this one is for the delete index?
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.
Updated it to 'where x is smaller than all 3 digit numbers' since these are examples of incorrect delete commands
docs/DeveloperGuide.md
Outdated
1. Deleting an animal while not all animals are being shown | ||
|
||
1. Prerequisites: Use `find 455` command to list animals that contain 455 in any of their fields. | ||
2 animals are listed: an anaconda named 455 and a lion named Simba with an ID of 455. | ||
|
||
1. Test case: `delete 455`<br> | ||
Expected: Simba with an ID of 455 is deleted from the list. Details of the deleted animal shown in the status message. Timestamp in the status bar is updated. | ||
|
||
1. Test case: `delete 123`<br> | ||
Expected: No animal is deleted. Error details shown in the status message. Status bar remains the same. | ||
|
||
1. Other incorrect delete commands to try: `delete`, `delete x`, `...` (where x is smaller than the requisite size)<br> | ||
Expected: Similar to previous. |
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.
May have to modify this since delete should work without finding anymore.
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.
Deleted
No description provided.