DreamerDragon / pe

0 stars 0 forks source link

Exception thrown does not match the issue #2

Open DreamerDragon opened 4 years ago

DreamerDragon commented 4 years ago

When i enter add d/2020-10-14 w/-12, the exception tells me Weight should only contain numbers, and it can only take up to at most 2 decimal points . However, this does not point it the problem but should in fact be it should be a postive number with the exception given, it should also remind the user about the unit of weight like kg. Coz it will affect the value entered as well. This may not really happen to enter negative number, but if user accidentally do this and the error should be able to match. Screenshot 2020-11-13 at 4.25.09 PM.png

nus-se-bot commented 4 years ago

Team's Response

Our user guide mentions that the weight the application takes in is in KG, thus our application does explicitly state this.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Although the answer given by the team is correct, it still does not answer my question directly, as my question is the incorrect error message shown and the kg unit is just a suggestion to make the error message clearer. The error message should be clearer and state that an invalid weight number is entered ( please enter a valid postive number with up to at most two decimals).


:question: Issue severity

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

Reason for disagreement: Although this will be a rare case, the error mismatch will still affect the user experience and the rigour of the app. It is not an error that can be ignored and should be low.