bryanyee33 / pe

0 stars 0 forks source link

Incorrect error message for index 0 #4

Open bryanyee33 opened 4 months ago

bryanyee33 commented 4 months ago

image.png

Doing delete 0 gives an incorrect error message, as the format is correct.

image.png

The above shows the correct error message when delete 7 is done (out of range).

soc-pe-bot commented 4 months ago

Team's Response

Mentioned in the error message that INDEX (Must be a positive number). We singled out invalid index 0 in the invalid command format, hence for the special case of index = 0, we decided to show invalid command format message over the standard invalid index message to provide more help to user.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]


## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.VeryLow`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]