search

natosy / pe

0 stars 0 forks source link

Parameter limitations for dog's Breed (b/) undefined #4

Open natosy opened 3 years ago

natosy commented 3 years ago

When I tried adding a dog with breed A, the error message shows Breeds should only contain alphabet characters and spaces, and it should not be blank., which does not cover the error which I'm producing. Subsequently, the breed argument passes when it is increased to 3 characters.

It is unstated in the documentation that breeds need to meet a minimum of 3 characters, and on top of that, the error message shown is not all-encompassing.

Screenshot 2021-04-16 at 2.27.29 PM.png

nus-pe-bot commented 3 years ago

Team's Response

As far as we know, there is no dog breeds that consist of only 1 or 2 letters, thus this is a non-typical user behaviour. As what you have mentioned, the 3 character requirement is unspecified, this is a documentation bug.

Items for the Tester to Verify

:question: Issue type

Team chose [type.DocumentationBug] Originally [type.FunctionalityBug]

Reason for disagreement: Hi, although the 3-letter requirement has not been documented, that is not the only issue here. I'm saying that the error message was irrelevant to the causation of the invalid input. This is both a DocumentationBug (limitation not stated) and a FunctionalityBug (the error message is incorrect), but as I'm only able to pick one, I chose FunctionalityBug instead.

Validation on Functionality Bug: image.png

Also, I don't think your team should dismiss errors just because the input is of "non-typical user behaviour".. Isn't this the purpose of bug testing? We're supposed to find the corner cases and identify errors from them.

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]