hauchongtang / pe

0 stars 0 forks source link

Remark command does not handle Integer overflow situation #5

Open hauchongtang opened 1 year ago

hauchongtang commented 1 year ago

Steps to replicate

Run your app
Run command remark 99999999999999999999999999999999999999999999999999999999999999999999999999 testremark
Observe that the CommandResult is not what is expected.

Expected

This should be the expected, where the invalid index is clear to the user.

Actual

Not the expected command result, may confuse the user when in fact its actually index invalid due to not exist.

soc-se-bot commented 1 year ago

Team's Response

Thanks for stress testing our application. However, stated in the CS2103T website, deliberate sabotage to create problems is not counted as a bug. Stated in the second bullet point, bugs related to integer overflow should not be considered a bug. The input that you have used in the second screen shot is an integer overflow bug. Unfortunately, we have to reject it.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

[ ] I disagree

Reason for disagreement: [replace this with your explanation]