Finished release 0.34.

Changelog.md

@@ -2,6 +2,9 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.34] - 2023-08-27
+- Fixed bug with "To" address for single email recipient
+
 ## [0.33] - 2023-08-27
 - Fixed utf-8 issues with sending emails
 - Fixed bug with applying migrations disabling all logging

compendium_v2/email/__init__.py

@@ -54,6 +54,8 @@ def send_mail(
 
     if not recipients:
         recipients = admin_emails
+    if isinstance(recipients, str):
+        recipients = [recipients]
 
     message = MIMEMultipart('alternative')
     message['Subject'] = subject

setup.py

@@ -2,7 +2,7 @@ from setuptools import setup, find_packages
 
 setup(
     name='compendium-v2',
-    version="0.33",
+    version="0.34",
     author='GEANT',
     author_email='swd@geant.org',
     description='Flask and React project for displaying '

test/test_send_mail.py

 from typing import List
 from email.mime.multipart import MIMEMultipart
-from email.mime.text import MIMEText  # noqa: F401
+from email.mime.text import MIMEText
 from contextlib import contextmanager
 from compendium_v2.db.auth_model import User
 from compendium_v2.email import send_admin_signup_notification, send_user_signup_notification
 
 
+class MockedThread(object):
+    def __init__(self, target, args):
+        self.target = target
+        self.args = args
+
+    def start(self):
+        self.target(*self.args)
+
+
 def decode_message(msg: MIMEMultipart):
     message: List[MIMEText] = msg.get_payload()
     text: MIMEText = message[0]
@@ -30,10 +39,15 @@ def test_signup_email_admin(app, mocked_admin_user, mocker):
     def _send_mail(*args, **kwargs):
         _msg = args[-1]
         decoded = decode_message(_msg)
+        _from = _msg['From']
+        _to = _msg['To']
+        assert _from == 'fakesender123@test.local'
+        assert _to == 'testemail123@email.local'
         message = 'testname has just signed up with the email testmail321@email.com and provider ID testsub'
         assert decoded == message
 
     mocker.patch('compendium_v2.email._send_mail', _send_mail)
+    mocker.patch('threading.Thread', MockedThread)
     with test_user(app) as user:
         send_admin_signup_notification(user)
 
@@ -42,9 +56,14 @@ def test_signup_email_user(app, mocker):
 
     def _send_mail(*args, **kwargs):
         _msg = args[-1]
+        _from = _msg['From']
+        _to = _msg['To']
+        assert _from == 'fakesender123@test.local'
+        assert _to == 'testmail321@email.com'
         decoded = decode_message(_msg)
         assert len(decoded) > 50  # check that there's a message
 
     mocker.patch('compendium_v2.email._send_mail', _send_mail)
+    mocker.patch('threading.Thread', MockedThread)
     with test_user(app) as user:
         send_user_signup_notification(user, 'testname')