search

LimZiJia / pe

0 stars 0 forks source link

Inconsistent errors for edit #2

Open LimZiJia opened 4 months ago

LimZiJia commented 4 months ago

Description of Bug

When editing a person, different invalid indexes give different errors

Expected Result

Both -ve and +ve invalid index should give the same resulting error message.

Actual Result

-ve gives a generic message, and +ve gives invalid index message

Steps to Reproduce

edit -1 t\ vs edit 99 t\

Screenshots

image.png

image.png

soc-se-bot commented 4 months ago

Team's Response

Screenshot 2024-04-21 at 21.18.35.png

When given a negative integer, the error message correctly states that a positive integer should be provided.

When given a positive integer that is out of bounds, the program then throws an error that the index is invalid.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I guess my point is that The person index provided is invalid can apply to many things. The user will experience different behaviors for the same type of errors they face, which is that an error in the index will provide two different messages.