michaeldera / simple-regex-ts Goto Github PK

Describe the bug
If the number that needs to be tested contains whitespace, the regex tes will give a false result.

To Reproduce
Steps to reproduce the behavior:

1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If possible include screenshots to help explain your problem.

Platform: Where did you see this bug (please remove those what does not apply):

• Google Chrome
• Safari
• Microsoft Edge

Additional context
Please add any other context about the problem here (like the Operating System where you witnessed the bug)

There are scenarios when the SimpleRegex tests will be run multiple times, for example, testing phone numbers during typing, after a user blurs input and just before sending an HTTP request to the server.

In these scenarios, the regex test return alternating results for the exact same phone number. After doing some digging (thanks to stack overflow), I realized that the global regex search starts from the previous match for each invocation of .test and that causes subsequent tests to have a different result even with the same input.

Example: Running SimpleRegex.MobileNumber.Econet.test(phoneNumber) twice, gives you different results

I would like to have an option to remove the global matching of the regexes.

At the moment, testing if a value is a mobile number involves testing all three different RegExp SimpleRegex.MobileNumber.Econet.test(VALUE), SimpleRegex.MobileNumber.Netone.test(VALUE) and SimpleRegex.MobileNumber.Telecel.test(VALUE)...

A developer friendly way would be to have another single way to test all three line SimpleRegex.MobileNumber.test(VALUE), then all three can be can be tested at once.