yyccbb / pe

0 stars 0 forks source link

Edit-an command not working as specified #11

Open yyccbb opened 5 months ago

yyccbb commented 5 months ago

image.png

Command: edit-an 2 6 d/19-02-2024 t/1230 n/General Flu

Appointment note at index 6 belongs to the second patient. However, there is no way I can edit it through running the command, in the way guided by the UG.

nus-se-script commented 5 months ago

Team's Response

Setup: Patient 2 has 6 Appointment Notes (list-an 2)

Screenshot 2024-04-20 at 1.31.25 PM.png

Tested: edit-an 2 6 d/19-02-2024 t/1230 n/UPDATED Bug Test 5

Successful command for editing the appointment note: Screenshot 2024-04-20 at 1.32.08 PM.png

List-an 2 to further confirm successful edit of the appointment note Screenshot 2024-04-20 at 1.33.38 PM.png

Result: Works as expected. It can be deduced that the appointment note at index 6 DOES NOT belong to the second patient as mentioned by the Tester. Rather, it is likely that it belong to the FIRST patient and the Tester's screen is displaying all notes listed after executing list-an (hence, index is 6).

The tester should execute list-an 2 to view all the appointment notes that belong to the second patients then execute edit-an command with the right index. This flow of action is clearly stated in the User Guide.

Items for the Tester to Verify

:question: Issue response

Team chose [response.CannotReproduce]

Reason for disagreement: ### How to reproduce (Actually you have stated the way to reproduce the bug.)

  1. Currently, my 1st patient has 4 notes and the 2nd patient has 5 notes.
  2. list
  3. list-an
  4. edit-an 2 6 d/19-02-2024 t/1230 n/General Flu

What the UG says

image.png Based on the UG, clearly there is something wrong with what INDEX is referring to. Since INDEX is for appointment records, I reasonably deduced it to be referring to the index in the list of appointment notes. In my case, the appointment note with INDEX 6 belongs to the patient with PATIENT INDEX 2 and I want to update the specified appointment note.