Extra unit-test

uktrade · Feb 6, 2024 · 6d265e2 · 6d265e2
1 parent 1a610ba
commit 6d265e2
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/mail/tests/test_tasks.py b/mail/tests/test_tasks.py
@@ -46,6 +46,16 @@ def test_locking_prevents_multiple_executions(self, mock_cache, mock_smtp_send):
         self.assertEqual(mock_cache.add.call_count, 2)
         mock_cache.delete.assert_called_once_with("global_send_email_lock")
 
+    @mock.patch("mail.tasks.smtp_send")
+    def test_send_email_task_insufficient_parameters(self, mock_smtp_send):
+        mock_smtp_send.side_effect = lambda *args, **kwargs: None
+
+        with self.assertRaises(ValueError) as context:
+            send_email_task.apply(kwargs={}).get()
+
+        self.assertTrue("Insufficient parameters to build email." in str(context.exception))
+        mock_smtp_send.assert_not_called()
+
 
 class NotifyUsersOfRejectedMailTests(TestCase):
     @override_settings(EMAIL_USER="[email protected]", NOTIFY_USERS=["[email protected]"])  # /PS-IGNORE