search

theeugenechong / pe

0 stars 0 forks source link

Error message does not match the error (less specific) #6

Open theeugenechong opened 2 years ago

theeugenechong commented 2 years ago

image.png

The error message for invalid format of the delimiter (/a) does not match the error / does not tell me that my delimiter is wrongly entered.

nus-pe-script commented 2 years ago

Team's Response

Ui format related.

This is not a bug. Error message is valid.

In the list_marks command, the parser first looks for the c/ parameter and the value associated to it will be CS2113T /aOP1. The parser prioritizes checking for invalid characters in the value before missing parameter(s) as parameters are processed in sequence. The error message is shown (in the image) as there is an illegal character / in CS2113T /aOP1.

Note: The parser does not look for / as a delimiter and different commands have different delimiters.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]