search

nicknamenic / pe

0 stars 0 forks source link

delete error message consistency #8

Open nicknamenic opened 5 months ago

nicknamenic commented 5 months ago

image.png

image.png

delete 0 should show the error message in the second picture, rather than some 'gotcha' to testers. For one, it's for clarity and user feedback in case it's a genuine error. Second, it's a bit annoying.

nus-se-script commented 5 months ago

Team's Response

Thanks for pointing this out. We overlooked this error.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: Again, severity.VeryLow rating is reserved for cosmetic flaws. Wrong error messages are not cosmetic flaws, so this rating is not acceptable. In fact, this is the same problem as the other bug relating to the expense command error message, so I believe it should be severity.Medium.