Enforce EmailStr for email fields on waiting list, and fix emails wit…

…h unicode characters.

backend/src/appointment/controller/mailer.py

@@ -100,7 +100,7 @@ def build(self):
                 filename=a.filename
             )
 
-        return message.as_string()
+        return message.as_string().encode('utf-8')
 
     def send(self):
         """actually send the email"""

backend/src/appointment/database/schemas.py

@@ -392,11 +392,11 @@ class TokenData(BaseModel):
 
 
 class SendInviteEmailIn(BaseModel):
-    email: str = Field(title='Email', min_length=1)
+    email: EmailStr = Field(title='Email', min_length=1)
 
 
 class JoinTheWaitingList(BaseModel):
-    email: str = Field(title='Email', min_length=1)
+    email: EmailStr = Field(title='Email', min_length=1)
 
 
 class TokenForWaitingList(BaseModel):

backend/test/integration/test_waiting_list.py

@@ -50,6 +50,21 @@ def test_already_in_list(self, with_db, with_client, make_waiting_list):
             # Ensure we did not send out an email
             mock.assert_not_called()
 
+    def test_bad_emails(self, with_db, with_client, make_waiting_list):
+        # Variety of bad emails
+        emails = ['', 'test', 'test@', '@example.org']
+
+        for email in emails:
+            with patch('fastapi.BackgroundTasks.add_task') as mock:
+                response = with_client.post('/waiting-list/join', json={'email': email})
+
+                # Ensure we hit the email validation error
+                assert response.status_code == 422, response.json()
+                assert 'value is not a valid email address' in response.json()['detail'][0]['msg']
+
+                # Ensure we did not send out an email
+                mock.assert_not_called()
+
 
 class TestWaitingListActionConfirm:
     def assert_email_verified(self, db, waiting_list, success=True):