search

Preshita01 / pe

0 stars 0 forks source link

Error message when multiple arguments are passed in for `mlist /e 1 1` does not seem to match the error #6

Open Preshita01 opened 2 years ago

Preshita01 commented 2 years ago

The error message for the command mlist /e 1 1 does not appear to match the error. I had expected it to highlight a duplicate argument.

Screenshot 2021-11-12 at 4.44.45 PM.png

nus-pe-bot commented 2 years ago

Team's Response

The input mlist /e 1 1 will see 1 1 as the EVENT_INDEX, which is a string with a space in between 1 and not a integer, therefore it is seen as an error. To specify another parameter, it should /e 1 /e 1.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: While I better understand the meaning behind the error message now, I still do not think that the error message displayed is very helpful for users.

I had tried out this command in the first place as since mdel can take in multiple arguments, such as /e 1 /e 1, users could possibly forget to include the /e prefix in front of every index. In such cases, I had expected the system's error message to either show users the command format or to highlight a duplicate index, so that it is easier for users to understand the error and correct themselves.