rizemon / pe

0 stars 0 forks source link

Case-sensitivity of category for `check_budget` #12

Open rizemon opened 3 years ago

rizemon commented 3 years ago

image.png

The image states what values CATEGORY can be. However, if I specified an upper case version, it will still accept it, despite not following strictly the given values.

image.png

nus-pe-script commented 3 years ago

Team's Response

image.png

All versions of the category names are accepted.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: This issue is clearly not a cosmetic issue as this impacts how your program interprets the user's command, followed by its behavior to this command, therefore it should not be severity.VeryLow. Since this bug causes only minor inconvenience, I think severity.Low is suitable.

Also, using code to explain your program works is a weak argument because as a tester I might not have knowledge of Java syntax and your simple explanation (All versions of the category names are accepted.) is actually proving that my reported issue is indeed a bug because that's exactly what I am reporting!