search

sogggy / pe

0 stars 0 forks source link

Integer Overflow for indexes for commands #5

Open sogggy opened 3 years ago

sogggy commented 3 years ago

Perhaps integer overflow cases should be handled for indexes for commands such as add, edit etc.

image.png Error message if there is overflow.

image.png Error message if there is no overflow.

Even though they are both postive numbers, their error messages are different, which implies that overflow was not handled properly.

nus-pe-bot commented 3 years ago

Team's Response

Error presents in AB3.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: This case should have been covered by developers though, as it leads to misleading error messages thrown.

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: This is a functionality bug as it causes the application to work unexpectedly. (App is not expected to throw error that appears when negative number is entered).