search

datn02 / pe

0 stars 0 forks source link

Two same kinds of invalid inputs gave two different error message #3

Open datn02 opened 2 years ago

datn02 commented 2 years ago

Command inputted: delete -1; delete a Index in this context is understood by non-negative numbers. Therefore it is not right to give 2 different error messages when a non-numerical value is entered

image.png

image.png

nus-pe-bot commented 2 years ago

Team's Response

Thank you for the feedback.

Due to the suggestion in PED, most testers prefer the current error message as compared to a more specific one.

This is intended by design in order to provide a more generic error message that caters to a wider range of technical expertise level of our audience.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: In the case of the dev team's response, then you should output each error input into 2 cases: one with invalid index and one with invalid number format, so that a wider range of technical expertise users can understand what they have inputted wrongly into the command. Having a general command would make users spend a longer amount of time figuring out what they have inputted wrongly into the command.

:question: Issue severity

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

Reason for disagreement: Would cause minor inconveniences, therefore at least Low severity.