search

tau-bar / pe

0 stars 0 forks source link

Misleading error when archiving appointment at index 0. #10

Open tau-bar opened 2 years ago

tau-bar commented 2 years ago

Expected:

• Expected Behaviour: For error to tell me that index is out of range.

Actual:

• Actual Behaviour: States that input is blank.

• Why that severity? Low, as despite the first line being misleading, the line below states that must be a positive integer. Users who are staff from clinics may not know what that means.

Steps to reproduce:

  1. Run apmt archive 0

Screenshots:

Screenshot 2021-11-12 at 4.51.35 PM.png

nus-pe-bot commented 2 years ago

Team's Response

No details provided by team.

The 'Original' Bug

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

Error message does not match the error

Error message does not match. The input is not blank but the index for the patient is invalid.

Screen Shot 2021-11-12 at 5.06.26 PM.png

[original: nus-cs2103-AY2122S1/pe-interim#4887] [original labels: severity.Low type.FunctionalityBug]

Their Response to the 'Original' Bug

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

The error message partially matches the user input. Under the Parameters: INDEX (must be a positive integer) i/INDEX OF MEDICAL ENTRY.

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]