search

YingxuH / pe

0 stars 0 forks source link

No error message given wrong fields while finding persons #10

Open YingxuH opened 4 years ago

YingxuH commented 4 years ago

Steps:

  1. Launch
  2. find a person by name and birthday. "(ab)find n/alex b/05-34"

image.png

  1. Try find by a birthday of 10 May "(ab)find n/alex b/10 May"

image.png

Though the find by birthday feature may not be implemented, no error message is given to warn the user.

nus-pe-bot commented 4 years ago

Team's Response

In the first place you aren't supposed to find by birthday, and there should be no prefix "b/".

As long as you provide one of the required fields, and the keyword does exist, the app will return something.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: Looks like the command will consider everything behind the "n/" as the name field. At least some error message should be provided to warn the users the command is in wrong format.

This is a bug as stated in the website:

image.png

extra parameters Not handled in their code.