search

danielonges / pe

0 stars 0 forks source link

Error message for add-appt with empty patient list does not indicate that there are no patients #4

Open danielonges opened 3 years ago

danielonges commented 3 years ago

In the UG, it is specified that "There must be at least 1 patient and doctor in the patient records and doctor records respectively." However, when I try to add an appointment provided that the patient list is empty, I get an error message warning me about an invalid index, which does not specify the pre-requisite for this command to work. Here is the command I used:

  1. clear-patient
  2. add-appt pt/1 dr/1 at/2021-01-01 00:00 to/2021-01-01 01:30

Here is the result obtained:

image.png

nus-pe-bot commented 3 years ago

Team's Response

It is very clear from the GUI that there are no patients displayed. the patient list is empty. There does not need to be an error to tell the user this, because it is clearly visible.

Therefore, the error is intended, as the patient index is invalid.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]