DreamerDragon / pe

0 stars 0 forks source link

Insufficient information for the exception thrown #1

Open DreamerDragon opened 3 years ago

DreamerDragon commented 3 years ago

When the profiles list has 8 entries and when i enter login 9, the error shown is not valid index, however, the error shown should be more specific of how to correct the behaviour, like say it is more than the length of the list, or out of range to tell user what exacly is the problem to avoid confusion. Screenshot 2020-11-13 at 4.20.43 PM.png

nus-se-bot commented 3 years ago

Team's Response

Thanks for the suggestion but my team feels that this is not considered a bug.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: The purpose of the error message is not only for warning but also specifying the error for the user to correct the mistakes. "Invalid" should be followed by more specific information to inform the users, which forms a better error message as shown in the original AB3 where incorrect command will be followed by its correct format and the usage.


:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]