daknam2001 / pe

0 stars 0 forks source link

Error in one field may sometimes be reported as invalid format. #6

Open daknam2001 opened 2 years ago

daknam2001 commented 2 years ago

image.png

Format is correct, phone number is wrong, but error message reports as invalid format.

Error message should say something like "Invalid phone number"

Other instances of this type of error

image.png

soc-se-bot commented 2 years ago

Team's Response

The function works as expected.

First phone number input only has 1 number, instead of 8 numbers (as specified by the UG). So error message shown is "Invalid input detected" and the format error message, to remind the user to follow the specified format.

Second phone number input has 8 numbers, hence will be accepted. UG does not specify that invalid phone numbers will be rejected as the function only requires 8 numbers to work, checking for validity is not in scope for this iteration.

Screenshot 2021-11-13 at 2.01.50 PM.png

Set blood pressure is a separate matter and should be reported as a separate bug, it is a duplicate of another bug we received and has been handled accordingly.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: While I agree with your point on the first example I gave, the second example, with the bug on 'setbloodpressure' still matches the title of the bug report, "error in one field may sometimes be reported as invalid format". The numbers inputted for the systemic and diabolic pressures are instances of 'error in one field' as the numbers are inhumanly low, but an 'invalid format' error is shown to the user.