search

avrilgk / pe

0 stars 0 forks source link

Wrong error message thrown when entering wrong command for BMI count #3

Open avrilgk opened 2 months ago

avrilgk commented 2 months ago

image.png

When entering the command:

health /h:bmi /height 2.76 /weight: 64.23 /date:20-02-2024

The input is invalid due to the missing semicolons/ additional whitespaces. Not due to the 'Insufficient parameters for bmi!'. Hence, this might confuse the users.

Reproducing the error:

health /h:bmi /height 2.76 /weight: 64.23 /date:20-02-2024 Exception Caught!

nus-pe-bot commented 2 months ago

Team's Response

UG stated to follow syntax exactly, else errors will be thrown. By not having the :, application does not read the height input, and thus thinks it is a missing parameter. This error is as intended.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]