search

JoanneJo / pe

0 stars 0 forks source link

Could have a more specific error message #3

Open JoanneJo opened 7 months ago

JoanneJo commented 7 months ago

Steps:

  1. add /n Panadol /d 40/02/2024 /s ACD123 /q 52 /p 19.90

Expected: Invalid date.

Actual: Invalid command format.

image.png

nus-pe-bot commented 7 months ago

Team's Response

The behaviour could be improve, but it is not wrong currently.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: image.png

image.png

VeryLow is only for cosmetic problems. An invalid date could appear in rare situations, like mistyping, and causes a minor inconvenience. The output says "Invalid command format!" which is a general error message without pointing out the specific reason. The date "40/02/2024" is a format error, specific to the date, and should be severity low.