nws321 / pe

0 stars 0 forks source link

Error message for deleting large index inaccurate #5

Open nws321 opened 1 week ago

nws321 commented 1 week ago

The error message for a large enough index says Index is not a non-zero unsigned integer., but this is not exactly true as it is a non-zero unsigned integer.

Suggestion: The message for such an index should be The person index provided is invalid instead.

Screenshot 2024-11-15 at 4.51.05 PM.png

nus-se-script commented 1 week ago

Team's Response

Hi tester, Thank you for your comment. The term "non-zero unsigned integer" is clearly defined in our glossary, and the User Guide for the delete command links this term to the glossary for reference. The error message shown is intended and accurately conveys that the provided index exceeds the valid range. In fact, this would be classified as deliberate sabotage as our target audience is NUS CCA leaders and the largest CCA in NUS will not have 10000000000 (ten billion) members, a mistype is also extremely unlikely.

From UG glossary:

image.png

From delete command

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]