Allow full stops in SMS senders

We have a team who want their (short) web address as the text message
sender. This commit updates the validation of text message senders to
allow `.` as a valid character, which is currently blocking them from
doing this.

We can be fairly confident this works because:

- the team are sending large volumes of messages already with their
  existing provider
- we’ve tested it with all combinations of
  - both our text message providers
  - an Android phone and n iPhone

app/main/forms.py

@@ -35,7 +35,7 @@ from app.main.validators import (
     ValidGovEmail,
     NoCommasInPlaceHolders,
     OnlyGSMCharacters,
-    LettersAndNumbersOnly,
+    LettersNumbersAndFullStopsOnly,
 )
 
 
@@ -560,7 +560,7 @@ class ServiceSmsSenderForm(StripWhitespaceForm):
         validators=[
             DataRequired(message="Can’t be empty"),
             Length(max=11, message="Enter 11 characters or fewer"),
-            LettersAndNumbersOnly(),
+            LettersNumbersAndFullStopsOnly(),
         ]
     )
     is_default = BooleanField("Make this text message sender the default")

app/main/validators.py

@@ -66,9 +66,9 @@ class OnlyGSMCharacters:
             )
 
 
-class LettersAndNumbersOnly:
+class LettersNumbersAndFullStopsOnly:
 
-    regex = re.compile(r'^[a-zA-Z0-9\s]+$')
+    regex = re.compile(r'^[a-zA-Z0-9\s\.]+$')
 
     def __init__(self, message='Use letters and numbers only'):
         self.message = message

tests/app/main/views/test_service_settings.py

@@ -814,7 +814,7 @@ def test_incorrect_letter_contact_block_input(
     ('', 'Can’t be empty'),
     ('abcdefghijkhgkg', 'Enter 11 characters or fewer'),
     (' ¯\_(ツ)_/¯ ', 'Use letters and numbers only'),
-    ('blood.co.uk', 'Use letters and numbers only'),
+    ('blood.co.uk', None),
 ])
 def test_incorrect_sms_sender_input(
     sms_sender_input,