AY2324S2-CS2103T-F13-2 / tp

https://nus-cs2103-ay2324s2.github.io/tp/
MIT License
0 stars 4 forks source link

[PE-D][Tester B] Inconsistent error message for invalid index input #138

Closed nus-pe-script closed 7 months ago

nus-pe-script commented 8 months ago

Normally, one would expect a somewhat consistent error message for a command in the correct format but with invalid input. But, when testing FApro, for commands that takes index as one of their parameters, there is different error message being displayed depending whether if the input index is less than or equal to 0 or if the input index is greater than the total number of contacts.

The following is a screenshot of the output of delete command with -1 as input index

image.png

The following is a screenshot of the output of delete command with an input index greater than the total number of contacts

image.png

The edit command, the addtags command and the select command also has the same output like that of the delete command for the two types of invalid index.


Labels: severity.VeryLow type.FunctionalityBug original: JukeBoxK/ped#4

ziyang27 commented 7 months ago

Not fixing