search

kaiyichen / pe

0 stars 0 forks source link

Error message for filter command is wrong #14

Open kaiyichen opened 2 years ago

kaiyichen commented 2 years ago

image.png

faculty given is correct but the prefix is wrong. Error message does not specify the prefix is wrong even when the faculty given is correct

nus-se-bot commented 2 years ago

Team's Response

Hi. ww/ is not a valid prefix, hence it is read as part of the faculty input f/. Hence this error message, implying the f/ input is wrong. This is how Tracey deals with incorrect inputs for predefined constants.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: image.png

Hi, based on CS2103T Website, an error message that does not match the error can be considered as a functionality bug. A user can accidentally type in the wrong prefix frequently. Therefore, Tracey should be able to catch these errors and return a valid error message, telling the user invalid prefix is given. Developers should not assume users (especially first time user like me) to fully understand how Tracey works and displaying the wrong error message will easily confuse the user.