search

kokerinks / pe

0 stars 0 forks source link

`edit` command allows phone number to be more than 16 characters long (insufficient validation) #3

Open kokerinks opened 4 months ago

kokerinks commented 4 months ago

How to recreate: run the command edit 1 p/1234567890123456

Expected: the application should reject this phone number as no international phone numbers will be more than 15 numbers long.

Actual: image.png

the application accepts the phone number, the UG also does not state that this phone number is considered invalid:

image.png

nus-se-bot commented 4 months ago

Team's Response

image.png

Included in our planned enhancement section.

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

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]