search

LimZiJia / pe

0 stars 0 forks source link

Delete index error message #3

Open LimZiJia opened 4 months ago

LimZiJia commented 4 months ago

Description of Bug

When deleting 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

delete 99 vs delete -1

Screenshots

image.png

image.png

soc-se-bot commented 4 months ago

Team's Response

When given a negative integer, the error message correctly displays that a positive integer has to be provided

Screenshot 2024-04-21 at 21.25.49.png

When given a positive integer that is out of bounds, the program will throw the 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.