search

pranav-ganesh / pe

0 stars 0 forks source link

Error message bug 6 #18

Open pranav-ganesh opened 2 years ago

pranav-ganesh commented 2 years ago

image.png

The error message here is not invalid phone number but rather an invalid command format since there is no space between two arguments

nus-pe-bot commented 2 years ago

Team's Response

Since there is no space, the application regards everything after cp/ as the phone number and therefore the phone number is invalid. This is the standard convention for CLI applications. When there is no space, it is considered as a single input.

image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: While I understand the team's argument, I still continue to believe that the error message thrown is not the most appropriate given the command I entered. The problem with the command is not really the format of the phone number but the fact that there is no space between the cp prefix and ce prefix.

I would have expected an invalid command format error along with the correct format to use rather than an incorrect phone number format. While I agree that the thrown bug is not incorrect by any means, I believe that it is not the most apt and accurate bug. Therefore, this is a feature flaw rather than a functionality flaw.

Hence, this can be considered a bug with low severity.