search

mechastriker3 / pe

0 stars 0 forks source link

Wrong error mesage shown for find_person command #7

Open mechastriker3 opened 3 years ago

mechastriker3 commented 3 years ago

When I ran find_person n/bob lee p/8888, the error message shown below was displayed.

image.pngfindPersonInvalidErr.png

If the intended behavior was to tell users they can only search by one field (perphaps you could have made it clearer in the UG or thrown a better error message)

nus-pe-bot commented 3 years ago

Team's Response

This was the intended error, the app is saying that the parameter that you input to find a person is not even valid. As stated in the UG (and in the error message), we don't even allow phone numbers with only 4 digits.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]

:question: Issue severity

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

Reason for disagreement: [replace this with your explanation]