ngnigel99 / pe

0 stars 0 forks source link

Incorrect user message given when listing lesson #8

Open ngnigel99 opened 3 years ago

ngnigel99 commented 3 years ago

image.png

correct syntax, wrong parameter

nus-pe-script commented 3 years ago

Team's Response

There is nothing wrong here. Our error message did inform the user of the correct format, which from the example provided is list lesson {PERIOD}.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Again let me be more precise

, the format is correct since the prefix is correct: list lesson

the parameter is wrong (mo should be mon)

A more appropriate error message would be to address a wrong parameter PERIOD could be mon, ...


:question: Issue severity

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

Reason for disagreement: not entirely cosmetic, as this message appears to be a general exception when an incorrect command is being parsed image.png

It's not cosmetic as it does cause a minor inconvenience by not informing the user ( a fast typist ) of the correct parameters. This would result in the inexperienced user having to make some reference to another document when entering the command.

image.png