Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update dg section 7 #333

Merged
merged 2 commits into from
Nov 5, 2021
Merged

Update dg section 7 #333

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
fc15893
Update PPP
NaokiShum Nov 5, 2021
1ac29e2
Update Dg section 7
NaokiShum Nov 5, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 33 additions & 1 deletion docs/DeveloperGuide.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -895,7 +895,39 @@ expected to do more *exploratory* testing.
1. Other incorrect edit commands to try: `edit`, `edit -p x`, `...` (where x is larger than the list size)<br>
Expected: Similar to previous.

### 7.5 Saving Data
### 7.5 Listing Clients/Products

1. Listing all clients
1. Test Case: `list -c`<br>
Expected: All clients in the application will be shown.

1. Listing all products
1. Test Case: `list -p`<br>
Expected: All products in the application will be shown.

### 7.6 Finding a Client/Product

1. Finding a client
1. Test case: `find -c ben`<br>
Expected: All clients name containing the keyword `ben` will be shown (case insensitive).
If there are no matches, an empty list will be shown.

1. Test case: `find -c ben alice charlie`<br>
Expected: All clients name containing the keyword `ben`, `alice` or `charlie` will be shown (case insensitive).
If there are no matches, an empty list will be shown.

1. Finding a product
1. Test case: `find -p IPhone`<br>
Expected: All products name containing the keyword `IPhone` will be shown (case insensitive).
If there are no matches, an empty list will be shown.

1. Test case: `find -c IPhone AirPods Macbook`<br>
Expected: All products name containing the keyword `IPhone`, `AirPods`
or `Macbook` will be shown (case insensitive).
If there are no matches, an empty list will be shown.


### 7.7 Saving Data

1. Dealing with missing/corrupted data files

Expand Down
8 changes: 5 additions & 3 deletions docs/team/naokishum.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,8 @@ Given below are my contributions to the project.

* **Enhancements to existing features**:
* Wrote additional tests for existing features to increase coverage (Pull requests
[\#100](https://github.com/AY2122S1-CS2103T-T12-1/tp/pull/100))
[\#100](https://github.com/AY2122S1-CS2103T-T12-1/tp/pull/100),
[\#332](https://github.com/AY2122S1-CS2103T-T12-1/tp/pull/332))
* Added sample data for new users. (Pull requests
[\#155](https://github.com/AY2122S1-CS2103T-T12-1/tp/pull/155))

Expand All @@ -57,9 +58,10 @@ Given below are my contributions to the project.
* Added implementation details of the `find` and `list` feature.(Pull requests
[\#52](https://github.com/AY2122S1-CS2103T-T12-1/tp/pull/52),
[\#159](https://github.com/AY2122S1-CS2103T-T12-1/tp/pull/159))
* Added use cases for `find` and `list` feature. (Pull requests
* Added use cases and manual testing instructions for `find` and `list` feature. (Pull requests
[\#52](https://github.com/AY2122S1-CS2103T-T12-1/tp/pull/52),
[\#159](https://github.com/AY2122S1-CS2103T-T12-1/tp/pull/159))
[\#159](https://github.com/AY2122S1-CS2103T-T12-1/tp/pull/159),
[\#333](https://github.com/AY2122S1-CS2103T-T12-1/tp/pull/333))

* **Community**:
* Reported bugs and suggestions for other teams in the class([link](https://github.com/NaokiShum/ped/issues))
Expand Down