search

leeraiyan / pe

0 stars 0 forks source link

Generic error messages #2

Open leeraiyan opened 4 years ago

leeraiyan commented 4 years ago

image.png

+ +

image.png

+ +

image.png

+ +

Error messages for alot of error handling throws the same message to the user, suggesting limited error catching.

This might also not allow the user to know what is wrong with the command

nus-pe-bot commented 4 years ago

Team's Response

For the error messages shown in this bug report, the reason why they are all the same is that the commands entered contains a character that is not allowed by the program. This means that these commands all fail the same first level check by the parser. The whole program will not accept characters other than alphabets, numbers, spaces and underscores (unless required for the input eg. dashes to indicate parameters and slashes for date).

image.png

As this was expected behaviour and documented in the user guide, we are marking this issue as Rejected.

You can refer to issue #874 where we clarified this further with another tester.

Additionally, if there was another issue related to the input of commands, like time format not correct or too many 'doafter' tasks, then it will be specifically mentioned (as shown below):

image.png

image.png

Thank you for the bug report and helping us test our product.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]