yyccbb / pe

0 stars 0 forks source link

Delete-an command not working as specified #10

Open yyccbb opened 2 months ago

yyccbb commented 2 months ago

image.png

Command: delete-an 2 6

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

soc-se-bot commented 2 months ago

Team's Response

Set up: Patient 2 has 6 appointment notes (list-an 2)

Screenshot 2024-04-20 at 1.35.33 PM.png

Tested: delete-an 2 6

Correct command message to show successful deletion: Screenshot 2024-04-20 at 1.36.02 PM.png

list-an 2 to further confirm the deletion of Patient 2's appointment note at index 6 Screenshot 2024-04-20 at 1.36.17 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 delete-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. delete-an 2 6

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 delete the specified appointment note.