KN-CY / pe

0 stars 0 forks source link

Incorrect error message for reps #9

Open KN-CY opened 1 year ago

KN-CY commented 1 year ago

Incorrect error message is given. Reps is said to be invalid even though a valid integer of 8 is given.

image.png

nus-se-script commented 1 year ago

Team's Response

image.png In the exercise_name it should be an exercise_name without any kg as no exercise_name would have a kg in its name. As per the picture above, after exercise_name it should be whitespace followed by weight_used_weight_unit

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: As mentioned, the main issue which I am trying to address is the "Incorrect error message for reps". The number of reps given is 8, which is indeed a valid integer.

However, the error message states that there is an "invalid input for reps", which is obviously an incorrect and misleading error message.

Instead, an appropriate error message could be exactly what the developer said in the response. The user should be informed that the exercise_name given is invalid...