weiquu / pe

0 stars 0 forks source link

Wrong error message for editf #11

Open weiquu opened 2 years ago

weiquu commented 2 years ago

editf 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):

soc-se-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.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Wrong error message for deletem

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


[original: nus-cs2103-AY2122S1/pe-interim#496] [original labels: severity.Low type.FunctionalityBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

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 duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]


: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]