Zhouyirui / pe

0 stars 0 forks source link

Ambiguous Error Message #3

Open Zhouyirui opened 2 years ago

Zhouyirui commented 2 years ago

On entering add n/Liya n/9119 e/e@e.com i/NUS c/cs y/03/2022 j/SE g/2.00

An error message about an invalid command format is given. The error message only states the correct format of the command. It is difficult to identify the error in the command.

Actual:

image.png

nus-pe-bot commented 2 years ago

Team's Response

Duplicate of #817

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Specificity of error message can be better

[Description] When an invalid argument is being supplied, error message returned is too generic.

[Steps to Reproduce]

  1. edit 1 grad/ 234234

[Expected] Expected an error message to be thrown stating that invalid argument supplied or "grad/" is not a valid argument.

[Actual] Generic error message of "Invalid command format!" returned.

[Screenshot]

image.png


[original: nus-cs2103-AY2122S1/pe-interim#781] [original labels: severity.Low type.FeatureFlaw]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

The error message is sufficient to let the users know that the command format is wrong.

To check specifically for wrong prefixes is not trivial and we already have error message for incorrect field input. image.png

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: [replace this with your explanation]


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]