search

DingBao-sys / pe

0 stars 0 forks source link

Error Messages not Indicative of the specified error for all commands #4

Open DingBao-sys opened 4 months ago

DingBao-sys commented 4 months ago

Steps to reproduce: addv n/John Doe d/25/2/2024 c/High Fever v/High

expected: indicates missing phone number

actual: throws same error message Invalid command format! addv: adds a visit to the specified patient. Parameters: INDEX (must be a positive integer) [d/DATE OF VISIT] [c/CONDITION] [v/SEVERITY] OR [n/NAME] [p/PHONE] [d/DATE OF VISIT] [c/CONDITION] [v/SEVERITY] Example: addv 1 d/25/2/2024 c/Mild Fever v/Low

I feel since it is quite manageable to catch the missing argument, like the effort to do so is pretty minimal we may try to catch it so that the user do not have to find out what the missing arguments is.

Screenshot 2024-04-19 at 4.42.37 PM.png

soc-se-bot commented 4 months ago

Your response not required for this bug as the team has accepted the bug as it is.