purivirakarin / pe

0 stars 0 forks source link

Invalid Email Validation: e.g. `invalid@invalid` is accepted #7

Open purivirakarin opened 4 months ago

purivirakarin commented 4 months ago

How to reproduce Type edit 1 e/invalid@invalid

Actual The application accept invalid@invalid as a valid email

Expected According to this image.png

The application should reject invalid@invalid email as the email is not in (XX@XX.XX) format

Screenshot 2567-04-19 at 17.05.51.png Screenshot 2567-04-19 at 17.05.57.png

nus-se-script commented 4 months ago

Team's Response

"This outcome is correct. The domain part of the email does not need to have a period inserted, as it is not stated under ""domain name must.

If there is only one domain label which in the tester's case was ""invalid"", there is no domain label to seperate and thus no period should be used. The use of the period is to seperate domain labels as mentioned in the error message. "

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: [replace this with your explanation]