search

avellinwong01 / pe

0 stars 0 forks source link

Error message could be clearer #5

Open avellinwong01 opened 2 years ago

avellinwong01 commented 2 years ago

image.png

Error message could be clearer to tell the user which attribute needs a positive integer here

nus-pe-script commented 2 years ago

Team's Response

Based on the input in your screenshot, there are only two attributes you have entered: description of the fluid & calorie count of the fluid. The error message clearly indicates there is an issue with your integer input which could only refer to your second attribute (calorie count) obviously.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Even if it can be deduced by the user, I believe it would still be better to make the error message more explicit. Though I would agree that the severity is very low.

image.png

In addition, in your manual testing section in the DG, it is written that the positive integer value should be for calories when a negative calorie value is keyed in. I believe that the same error message should be displayed when some garbage non-integer value for calories are keyed in (since the invalid inputs are of a similar type)

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]