search

ramapriyan912001 / pe

0 stars 0 forks source link

Misleading error command for empty params #11

Open ramapriyan912001 opened 3 years ago

ramapriyan912001 commented 3 years ago

Screenshot 2021-11-12 at 5.01.54 PM.png

Incorrect error msg for the command entered in the picture. My format is correct, but my params cannot be empty However error suggests former is the issue, not the latter.

nus-pe-bot commented 3 years ago

Team's Response

Successive fields need to be separated by a whitespace, so typing add c/p/... would be interpreted as an attempt to add a company with name p/. As a result, the command is lacking the Position field which is a compulsory field that must be specified when adding an application entry. Missing of any compulsory fields would be considered as invalid command format, thus the error message given is exactly as expected.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The details of your response have not been made clear anywhere in your UG. Given that to be the case, I believe this is clearly a bug.