parth-io / pe

0 stars 0 forks source link

Large indices throw a different error message #11

Open parth-io opened 1 year ago

parth-io commented 1 year ago

Pre-req: 1 message exists

To reproduce: Type m d 11111111111111111111111

Expected: The error message should read "The message index provided is invalid."

image.png

nus-pe-script commented 1 year ago

Team's Response

This was mentioned in the PE brief. Testing problems caused by integer overflow and intentional sabotage is not in the scope of the PE.

image.png

The 'Original' Bug

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

Show command shows different/wrong error message for same kind of error

Show command shows different/wrong error message for same kind of error

When I choose a relatively small out-of-bounds index (e.g. 100) for the show command, the application shows invalid index error, which is correct, because the index is out of bounds.

However, when I choose a larger out-of-bounds index (e.g. 10000000000000000000) for the show command, the application is shown invalid command format.

Example

show 100

Screenshot 2022-11-11 at 4.31.25 PM.png

show 10000000000000000000

Screenshot 2022-11-11 at 4.31.07 PM.png

This causes the error message to become generic, hence I put it under functionalitybug.


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

Their Response to the 'Original' Bug

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

This was mentioned in the PE brief. Testing problems caused by integer overflow and intentional sabotage is not in the scope of the PE.

image.png

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]