diff --git a/app/main/views/send.py b/app/main/views/send.py index dab41646e..180ad4c3a 100644 --- a/app/main/views/send.py +++ b/app/main/views/send.py @@ -275,19 +275,11 @@ def remove_notify_from_sender_options(sender_details): def verify_sender_options(sender): - if sender.get("sms_sender") in ["Notify.gov", "US Notify"] and sender["is_default"]: - return True - if ( - sender.get("sms_sender") not in ["Notify.gov", "US Notify"] - and not sender["is_default"] - ): - return True - if ( - sender.get("sms_sender") not in ["Notify.gov", "US Notify"] - and sender["is_default"] - ): + if sender.get("sms_sender") not in ["Notify.gov", "US Notify"]: return True + return False + def get_sender_context(sender_details, template_type): context = { diff --git a/tests/app/main/views/test_send.py b/tests/app/main/views/test_send.py index 0d9c896e4..2870f4599 100644 --- a/tests/app/main/views/test_send.py +++ b/tests/app/main/views/test_send.py @@ -308,7 +308,7 @@ def test_usnotify_and_notifygov_sms_sender_removal_if_default(sender_data, mocke sender_details = remove_notify_from_sender_options(sender_data) - assert len(sender_details) == 3 + assert len(sender_details) == 2 def test_that_test_files_exist():