search

Asuraxsoul / pe

0 stars 0 forks source link

Different error messages shown for cancel command for same invalid group #5

Open Asuraxsoul opened 4 years ago

Asuraxsoul commented 4 years ago

image.png

image.png

image.png

Similar to delete command index issue, Both inputs belongs to the same partition of test cases, so I feel that I would be better for them to have the same error message to reduce confusion.

The positive number error message for 'cancel 9999999999999' does not seem to be a fitting error message as well.

nus-se-bot commented 3 years ago

Team's Response

Brought over bug from AB3. The integer exceed MAX_INT.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I do not believe that this is a brought over bug from AB3 because there is no such feature and function in AB3. There is no stating that we must use functions and APIs defined from AB3 unmodified. So I feel that the error message can be better improved.

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]