From eaea9d14150e9c6cb283599ecf7773b7d639d436 Mon Sep 17 00:00:00 2001 From: Kenneth Kehl <@kkehl@flexion.us> Date: Thu, 2 Oct 2025 12:20:03 -0700 Subject: [PATCH] fix tests --- tests/app/delivery/test_send_to_providers.py | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/tests/app/delivery/test_send_to_providers.py b/tests/app/delivery/test_send_to_providers.py index e2040d6c0..5d67c16ce 100644 --- a/tests/app/delivery/test_send_to_providers.py +++ b/tests/app/delivery/test_send_to_providers.py @@ -142,9 +142,7 @@ def test_should_send_personalised_template_to_correct_email_provider_and_persist template=sample_email_template_with_html, ) db_notification.personalisation = {"name": "Jo"} - mock_ses_client = MagicMock() - mock_ses_client.send_email.return_value = "reference" - mocker.patch("app.aws_ses_client", mock_ses_client) + mocker.patch("app.aws_ses_client.send_email", return_value="reference") send_to_providers.send_email_to_provider(db_notification) app.aws_ses_client.send_email.assert_called_once_with( f'"Sample service" ', @@ -178,9 +176,7 @@ def test_should_not_send_email_message_when_service_is_inactive_notifcation_is_i sample_service, sample_notification, mocker ): sample_service.active = False - mock_ses_client = MagicMock() - mock_ses_client.send_email.return_value = "reference" - send_mock = mocker.patch("app.aws_ses_client", mock_ses_client) + send_mock = mocker.patch("app.aws_ses_client.send_email", return_value="reference") mock_s3 = mocker.patch("app.delivery.send_to_providers.get_phone_number_from_s3") mock_s3.return_value = "2028675309"