search

bchenghi / pe

0 stars 0 forks source link

Invalid index for select command #9

Open bchenghi opened 4 years ago

bchenghi commented 4 years ago

Perhaps for usability, I feel that the error message for select 0 and selecting a card index that is too large should display the same error message. Since both are issues about an invalid index. Below is when select 0 is executed. image.png Below is when index is too large. image.png

nus-se-bot commented 3 years ago

Team's Response

Hi,

Zero is neither a positive nor negative integer. Hence, that error message is appropriate because it asks the user for a positive integer.

The first error message signifies that the input is invalid no matter how many entrie the user inputs but the second error message indicates that the valid could be valid but the current list is too small.

Bug is rejected.

Regards, Melanie

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your reason]

:question: Issue severity

Team chose [severity.VeryLow] Originally [severity.Medium]

Reason for disagreement: [replace this with your reason]