zacharyxzz / pe

0 stars 0 forks source link

Invalid find command followed by valid find command breaks Appointments list #5

Open zacharyxzz opened 1 week ago

zacharyxzz commented 1 week ago

Steps:

Ensure that there are some appointments in the appointments list, in this case I have added two appointments Enter command: find appt n/ Enter command: find appt n/ x Behavior:

In step 2, an error message "Unknown command" was given. Then in step 3, the message "0 appointments listed!" was given. However, the appt list still shows all appts. Now, if we enter the list appt command, duplicate appts are shown, and the appt list is basically broken.

image.png

nus-se-bot commented 1 week ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Invalid find command followed by valid find command breaks patients list

Steps:

  1. Ensure that there are some patients in the patients list, in this case I have added two patients: bellam and jack
  2. Enter command: find person n/
  3. Enter command: find person n/ s

Behavior:

  1. In step 2, an error message "Unknown command" was given.
  2. Then in step 3, the message "0 persons listed!" was given.
  3. However, the patients list still shows all patients.
  4. Now, if we enter the list person command, duplicate patients are shown, and the patients list is basically broken.

image.png

image.png

Rationale:

  1. This sequence of commands is decently likely to happen as it involves just 1 typo (entering the command find person n/ ).
  2. This then proceeds to cause the patients list to show inaccurate results.

[original: nus-cs2103-AY2425S1/pe-interim#1666] [original labels: severity.High type.FunctionalityBug]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

No details provided by team.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]