search

cristoforows / pe

0 stars 0 forks source link

Not suitable error message #4

Open cristoforows opened 2 years ago

cristoforows commented 2 years ago

the error was too many parameter was inputted instead of lacking of parameter as the error message suggested

image.png

nus-se-bot commented 2 years ago

Team's Response

Both point to the same error messages

The 'Original' Bug

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

Incorrect error message printed when extra parameters are provided

Steps to reproduce: 1) Run the following command: add doctor /info s1234567a, alice a, 22, F, 1 NUS Road, 2000-01-01, ent, extra parameter

Expected outcome: the user is warned about the presence of an extra unnecessary parameter.

Actual outcome: the user is warned about missing parameters instead.

image.png

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

Their Response to the 'Original' Bug

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

Yes, I believe a better error message could have been thrown so that the user understand what is going on.

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 severity

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

Reason for disagreement: this is not a cosmetics problem and would cause inconvenience for the user because they would be confused on which part of the input was wrong. Instead of lowering the severity, this could even result in higher one (medium) instead. I believe the severity lays between low and medium