search

nus-cs2103-AY2223S1 / pe-dev-response

0 stars 0 forks source link

Wrong error message for list\ #5696

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

Reproduction instructions

list\

Expected

Unknown command

Actual

Invalid command format! 
help: Shows program usage instructions.
Example: help

[original: nus-cs2103-AY2223S1/pe-interim#5650] [original labels: type.FunctionalityBug severity.Low]

yeoyujie commented 1 year ago

Team's Response

It is indeed an invalid command format. list list list list 123 would have been accepted as they do follow the command format, as the invalid parameters after list would simply be ignored as the parser will only read list, but not when its list\ as it does not conform to the format of our commands specified in our UG or the application.

In fact, our error message is even more specific as we showed a help page to our UG with all the available commands, the prefix required (if any) and its format.

image.png

Duplicate status (if any):

--