search

Bubbl3T / pe

0 stars 0 forks source link

`delete 0` returns a wrong error message #3

Open Bubbl3T opened 1 year ago

Bubbl3T commented 1 year ago

When the user enters the delete command with an invalid index that is greater than the number of items in the list, (e.g. list contains 5 items, user tries to enter command delete 6), the error message shown is "The itinerary index provided is invalid." However, when the user tries to enter the delete command with an index of 0 which is also an invalid index, the error message popped up was "Invalid command format!" followed by the command usage instructions for the delete command.

nus-pe-bot commented 1 year ago

Team's Response

The index provided in the command needs to be positive, hence in this case, providing "0" violates the command format, and it is intended for the application to throw the error for invalid command format to show the user the command usage instruction for delete which once again notifies the user that index needs to be positive.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]