Skip to content

Commit

Permalink
Minor fixes/typos
Browse files Browse the repository at this point in the history
  • Loading branch information
itsme-zeix committed Nov 12, 2024
1 parent 8b7f22c commit ad53765
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions docs/DeveloperGuide.md
Original file line number Diff line number Diff line change
Expand Up @@ -437,7 +437,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 @@ -692,7 +692,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>
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 @@ -703,7 +703,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

0 comments on commit ad53765

Please sign in to comment.