search

weiquu / pe

0 stars 0 forks source link

Wrong error message for deletem #9

Open weiquu opened 2 years ago

weiquu commented 2 years ago

deletem with a big number as the index shows the wrong error message (should be out of range of the currently displayed list rather than invalid command format):

Incorrect error message:

Screenshot 2021-11-12 at 4.39.36 PM.png

Correct error message:

Screenshot 2021-11-12 at 4.39.44 PM.png

nus-pe-bot commented 2 years ago

Team's Response

This seems to be an unrealistic input as it is unlikely for a CCA to have that many members and can be considered an intentional sabotage. From the screenshot below, we do not consider this a bug.

image.png

Duplicate issue with the rest as it all involves the same "defect", which is not accounting parsing values above maximum integer in java.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]