tkhahns / pe

0 stars 0 forks source link

"save meal" does not handle special characters in string #3

Open tkhahns opened 2 weeks ago

tkhahns commented 2 weeks ago

I suppose that the meal menu should only contain upper case or lower case alphabetical characters from "a" to "z". A wrong string format exception should be thrown And the string after "/" in the meal string is not handled in this case

save meal mat-cha /c800
      _____________________________________________________________________________
      Added to options: mat-cha with 800 calories
      _____________________________________________________________________________
save meal hot/dog /c1000
      _____________________________________________________________________________
      Added to options: hot with 1000 calories
      _____________________________________________________________________________
nus-se-bot commented 1 week ago

Team's Response

Thank you for spotting the bug. We should have specified a constraint on meal names not having '/' inside of them. However, as this bug does not cause the programme to break, and is a rare occurrence, we have thus reclassified it to 'low'.

Items for the Tester to Verify

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]