search

botr99 / pe

0 stars 0 forks source link

Address cannot be all non-alphanumeric #17

Open botr99 opened 2 years ago

botr99 commented 2 years ago

Steps to replicate:

  1. add n/C e/aaa.b@aa-bbd a/@@@ p/00000001

Expceted: fails doesnt make sense as address cannot be all non-alphanumeric

Actual: added

nus-pe-bot commented 2 years ago

Team's Response

Addresses can take any values, and it should not be blank. There is no specification in our guides that "address cannot be all non-alphanumeric".

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: I feel that the input validation can be more strict. Even though it's not explicitly stated in the UG, addresses with all non-alphanumeric characters doesn't make sense and shouldn't be allowed to pass the input validation.