chiralcentre / pe

0 stars 0 forks source link

Inconsistent error messages for invalid indices in edit command #7

Open chiralcentre opened 6 months ago

chiralcentre commented 6 months ago

The error messages when a negative index and when a positive index exceeding length of menu used differ, when both should be The product index provided is invalid.

image.png image.png

nus-se-bot commented 6 months ago

Team's Response

Here, the index -1 produces an invalid command format because it is negative, and negative indexes will never exist in the menu. The command requires a positive index, hence invalid command format. For index 10, the index can potentially be on the menu, depending on the number of items on it. In this case, there is no index 10 on the menu, hence the program outputs that the product index provided is invalid.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]