search

mamayuan / pe

0 stars 0 forks source link

Inconsistent Error Message for Invalid Indices #1

Open mamayuan opened 11 months ago

mamayuan commented 11 months ago

Upon entering delete 0 and delete 200 (200 is larger than the size of the data). Both are considered invalid indices but the error messages are different and inconsistent. This can potentially confuse the user.

image.png

image.png

soc-se-bot commented 11 months ago

Team's Response

Thank you for your input. We believe that this is the correct behaviour as the first error message is trying to say that the INDEX has to be a positive number while the second error message is trying to say that 200 is not a valid index as there is no employee with the index 200. We believe that this should be considered a feature flaw instead.

Under the Feature Flaw bug:

image.png

Items for the Tester to Verify

:question: Issue type

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

Reason for disagreement: [replace this with your explanation]