search

charmainehly / pe

0 stars 0 forks source link

mail-x command does not search across existing contacts in the system for a specific name. #8

Open charmainehly opened 2 years ago

charmainehly commented 2 years ago

steps to reproduce:

  1. use add command "add n/ Peter pan a/A1188777L e/abhhhhe@gmail.com m/ACC3709 t/needshalp"
  2. use find command "find n/Alex"
  3. use mail-x command "mail-x e/alexyeoh@example.com n/Peter pan"

image.png

expected: mail should open up with all instances of entries with "Peter pan" OR minimally have a mail open up with the entry for "alexyeoh@example.com" - given as it is stated in UG that anyone who matches at least one of the specified arguments will be included.

actual: error message pops up with no further execution.

nus-se-script commented 2 years ago

Team's Response

mail-x command only promises creating a mailing group based on the parameters provided. The contacts present in the contact list must be adhering to the parameter. This does not mean that at least one of the parameters is being matched.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: As mentioned in the steps to reproduce, it is expected that the contact previously added - Peter pan should appear in the mail app when it pops up but it does not.

I understand it might not be the case that all argument parameters match at least one entry as justified by the team. But in this particular case and report, the parameter "Peter pan" should match the candidate that had been added.