search

nus-cs2103-AY2021S2 / pe-dev-response

0 stars 0 forks source link

clear-doctor command accepts invalid arguments #2045

Open nus-pe-bot opened 3 years ago

nus-pe-bot commented 3 years ago

Bug Name: clear-doctor command accepts invalid arguments

Steps to reproduce 1) enter clear-doctor command 2) after typing command, add any argument (eg: dr who)

Expected throw a error saying invalid command, as user could be using clear-doctor to delete a dr rather than delete-doctor. Maybe if invalid argument is detected, clear-doctor throw a warning to the user, instead of clearing everything.

Actual clear-doctor clears all drs regardless of invalid argument

Screenshots

clear doctor.PNG

[original: nus-cs2103-AY2021S2/pe-interim#2045] [original labels: severity.Medium type.FunctionalityBug]

spzj commented 3 years ago

Team's Response

Rejected, duplicate of #2046.

Duplicate status (if any):

Duplicate of #2046