search

DarrenCsAcc / pe

0 stars 0 forks source link

Wrong error #6

Open DarrenCsAcc opened 1 year ago

DarrenCsAcc commented 1 year ago

image.png

photo1668133327 (2).jpeg

Error should be invalid index.It only works up till a certain point.

nus-pe-bot commented 1 year ago

Team's Response

This is similar to the linked duplicate issue, which is a bug related to parsing the INDEX input of the command.

The 'Original' Bug

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

Valid index delete

image.png

Did not specify range of valid index.Should throw out error that index given is not valid

[original: nus-cs2103-AY2223S1/pe-interim#926] [original labels: type.FeatureFlaw severity.Medium]

Their Response to the 'Original' Bug

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

This error only occurs when the input index goes beyond the maximum integer 2147483647. We feel that this would not be a likely scenario in which the command would be used, such that this would be a case of extreme user input, which is mentioned on the module website as shown in the screenshot below. Hence, it should not be considered as a bug at all.

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]

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]