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

Update DG #342

Closed
wants to merge 2 commits into from
Closed

Update DG #342

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c128ca2
Update DG
FionaQY Nov 11, 2024
f2de2b8
Fix capitalisation
FionaQY Nov 12, 2024
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
6 changes: 3 additions & 3 deletions docs/DeveloperGuide.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -424,7 +424,7 @@ Priorities: High (must have) - `* * *`, Medium (nice to have) - `* *`, Low (unli

### Use cases

(For all use cases below, the **System** is the `AgentAssist` and the **Actor** is the `user`, unless specified otherwise)
(For all use cases below, the **System** is the `AgentAssist` and the **Actor** is the `User`, unless specified otherwise)

**Use case: U1 - List all clients**

Expand Down Expand Up @@ -679,7 +679,7 @@ testers are expected to do more *exploratory* testing.
2. Test case: `add n/ <NAME> p/ <PHONE> e/ <EMAIL> a/ <ADDRESS> j/ <JOB> i/ <INCOME>`<br>
Expected: A client with the given details is added to the end of the list of clients. Details are shown in the results box and any details that are truncated can be seen by using the `view` command with the client's displayed index.

3. Test case: `add n/ <NAME> p/ <PHONE> e/ <EMAIL> a/ <ADDRESS> j/ <JOB> i/ <INCOME> t/ <TIER> r/ <REMARK> s/ Status`<br>
3. Test case: `add n/ <NAME> p/ <PHONE> e/ <EMAIL> a/ <ADDRESS> j/ <JOB> i/ <INCOME> t/ <TIER> r/ <REMARK> s/ <Status>`<br>
iamdiluxedbutcooler marked this conversation as resolved.
Show resolved Hide resolved
Expected: A client with the given details, including the optional ones, is added to the end of the list of clients. Similar to previous.

4. Missing compulsory fields: `add n/ <NAME> p/ <PHONE>`, `add n/ <NAME> p/ <PHONE> e/ <EMAIL>`, `...` <br>
Expand All @@ -690,7 +690,7 @@ testers are expected to do more *exploratory* testing.

2. Add an existing client
1. Prerequisites: There exists a client in AddressBook which has the same name, phone number and email as the client to be added.
2. Test case: `add n/ <SAME_NAME> p/ <SAME_PHONE> e/ <SAME_EMAIL> a/ <ADDRESS> j/ <JOB> i/ <INCOME>` (where <SAME_NAME>, <SAME_PHONE> and <SAME_EMAIL> are the name, phone number and email of an existing client) <br>
2. Test case: `add n/ <SAME_NAME> p/ <SAME_PHONE> e/ <SAME_EMAIL> a/ <ADDRESS> j/ <JOB> i/ <INCOME>` (where `<SAME_NAME>`, `<SAME_PHONE>` and `<SAME_EMAIL>` are the name, phone number and email of an existing client) <br>
Expected: An error message indicating that the client already exists is shown to user. No client is added.

### Deleting a client
Expand Down