[PE-D][Tester C] Unexpected behaviour of edit person command for Target User

[nus-pe-bot]

Description:
The edit person command allows users to enter inputs such as numeric statuses instead of having a fixed set of valid inputs such as Recovering, Recovered, Under observation, this may lead to confusion in users utilising the application to input and read details of their patients’ status.

Steps to Reproduce:

1. Launch DocTrack_1.5.1.jar.
2. Attempt the following command: edit person 1 st/123
3. Observe that the command executes successfully with Person chosen with index 1 under Patients is updated with a new Status of 123.

Expected:
Status should only validate a fixed number of valid inputs such as Stable, Recovering, Recovered, Under observation.

Actual:
The command executes successfully with Person chosen using a positive index under Patients is updated with a new Status of 123.

Suggested Improvement:
Change the behaviour of how Status is being parsed to ensure that only valid inputs are allowed. Utilise enums for valid input validation.

---

Labels: type.FunctionalityBug severity.Medium
original: blanklogic/ped#9