nus-cs2103-AY1920S1 / pe-dev-response

0 stars 0 forks source link

Result of `addvisit 1 v/12/12/2012` not matched to the description of test case c #1007

Open nus-pe-bot opened 4 years ago

nus-pe-bot commented 4 years ago

Screenshot 2019-11-15 at 16.23.23.png

Steps to reproduce,

  1. precondition: person #2 has no past records
  2. enter command: addvisit 2 v/12/12/2012
  3. the most recent visit date is updated to the specified date (12/12/2012) instead of the current date according to the description according to test case b (since i should expect similar behavior as test case b)


[original: nus-cs2103-AY1920S1/pe-interim#1016]

SQwQ commented 4 years ago

Team's Response

It is indeed a little ambiguous. However, the similar behavior here actually just means that the Most recent visit date will be updated to the current visit date (12/12/2012 in TestCase c) if the Most recent visit date is earlier than the current visit date/if patient has no prior records.

Duplicate status (if any):

--

SQwQ commented 4 years ago

Team's Response

This is indeed a little ambiguous. The similar behavior here does not mean that test case c will have the exact same behavior as b, but rather similar in the sense that in both cases the Most recent visit date will be updated to the current visit date (12/12/2012 in case C and the current date in case B) if the Most recent visit date is earlier than the current visit date (which is 12/12/2012 in case C and the current date in case B) or if the patient has no past records.

Duplicate status (if any):

--