nus-cs2113-AY2223S2 / pe-dev-response

0 stars 0 forks source link

Error message does not match error when invalid flag for tags is used #1639

Open nus-pe-bot opened 1 year ago

nus-pe-bot commented 1 year ago

image.png

Error thrown should be Tag edit commands must have exactly one of -edit, -add, or -del but instead user is told that there are missing arguments.


[original: nus-cs2113-AY2223S2/pe-interim#1273] [original labels: type.FunctionalityBug severity.Low]

clement559 commented 1 year ago

Team's Response

Our parser checks if arguments are valid for each provided flag before checking if the command is missing flags, and the error message is based on the first detected problem, which is the requirement of an additional argument after the flag done. However, we may improve the parser in the future to catch missing flags first.

Duplicate status (if any):

--