search

sebin0817 / pe

0 stars 0 forks source link

Invalid Email accepted #5

Open sebin0817 opened 2 months ago

sebin0817 commented 2 months ago

Screenshot 2024-04-19 at 4.25.02 PM.png

Tested: test@example Expected: Raise error that invalid email is provided Actual: Accepted and edited the person detail with an invalid email

nus-se-bot commented 2 months ago

Team's Response

Thank you for reporting this bug.

Our team acknowledges this issue that there has some oversight. We feel that this is a feature flaw rather than a functionality bug because our email validation is "incomplete".

Items for the Tester to Verify

:question: Issue type

Team chose [type.FeatureFlaw] Originally [type.FunctionalityBug]

Reason for disagreement: image.png

Disagree with team's modification in the issue type, this is a functionality bug as defined in the course page.test@example is a legitimate user behaviour as they may have forgotten to type in the conventional .com at the end, hence the application should flag this out either as a warning or error.