search

rushilramesh / pe

0 stars 0 forks source link

Different error messages for invalid patient index #5

Open rushilramesh opened 2 years ago

rushilramesh commented 2 years ago

image.png

image.png

The invalid index 'P001' should be identified as an invalid index to identify the correct error to users, as the invalid command format error message does not specify the issue with the index.

nus-pe-bot commented 2 years ago

Team's Response

Thank you for the response :) This does not cause any issue with the usage and the error message is a purely cosmetic issue.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Low]

Reason for disagreement: I believe this is more than a cosmetic issue, as users may be confused by the incorrect error message. The command format in this case is perfectly valid, with an incorrect index. This is definitely a functionality bug as seen below. As the usability of the application does not get affected overall but does cause minor inconvenience to the user, it should be classified as a low severity bug.

image.png