search

darshan152 / pe

0 stars 0 forks source link

Invalid index error message not shown when index is too high #9

Open darshan152 opened 2 years ago

darshan152 commented 2 years ago

image.png

When providing a lower index, the error message is:

image.png

Which is what is expected.

Users will not understand why the error message shows an invalid command rather than just an invalid index.

Steps to reproduce: delete 2200000000

nus-pe-bot commented 2 years ago

Team's Response

This is a case of deliberate sabotage. As we have already stated in our NFRs that the application should be able to perform well to handle up to a list of 5000 students. This case of extreme behaviour cannot be accepted.

Screenshot 2022-04-18 at 4.50.48 PM.png

Screenshot 2022-04-18 at 4.49.58 PM.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: NFR not in UG. This is not a deliberate sabotage as there was no mention of limit in UG.

:question: Issue severity

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

Reason for disagreement: This is not simply a cosmetic issue. There is inconsistency in the error message produce which causes minor inconvenience to the users as they did not enter a negative number.