tyx021 / pe

0 stars 0 forks source link

Clear all vaccination data #10

Open tyx021 opened 1 year ago

tyx021 commented 1 year ago

Expected: Only deletion of vaccinations that cause some appointment to become invalid should be rejected.

Actual: Only 2 of 12 vaccination cleared, while I only have one of them in appointment (Does 2 (Moderna))

image.png

nus-se-bot commented 1 year ago

Team's Response

Below shows the description of vaccination clear command:

clear - Clear all vaccination data

image.png

From this, the clear command is meant to wipe all vaccination data from the system.

However, to protect the user's data and prevent unintended further losses, the clear command will stop its process if it detects that deleting a vaccination will make an appointment invalid.

If the user's intention was to intentionally clear all vaccination data for real, he would have intentionally added the --force true argument as well. From the screenshot, the eeerror message states this as well.

However, if your bug is to delete all vaccinations that would not result in an invalid appointment, then this is not in scope as this feature is

For this reason, we have classified this under NotInScope, FeatureFlaw as the described clear is a new feature. The serverity has also been reduced to Low as only a minority of users will want to clear such vaccination data (vaccinations that will not make an appointment invalid) and as it is, without this clear feature, the application is still usuable.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]


## :question: Issue type Team chose [`type.FeatureFlaw`] Originally [`type.FunctionalityBug`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]
## :question: Issue severity Team chose [`severity.Low`] Originally [`severity.Medium`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]