Closed superchilled closed 2 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 98.31%. Comparing base (
35e4214
) to head (17cb17b
).:exclamation: Current head 17cb17b differs from pull request most recent head 22f9655. Consider uploading reports for the commit 22f9655 to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This PR adds some validtion checks to the Verify v2 implementation. Specifically it:
SMS
classfrom
setter to ensure that:WhatsApp
classfrom
setter to ensure that values are between 11 and 15 characters in lengthVerify2#start_verification
method to ensure that thebrand
argument is between 1 and 16 characters in length