darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-06-17 19:56:47 -04:00
Code Issues Packages Projects Releases Wiki Activity

Revert missing logic

Browse Source
This commit is contained in:
Andrew Shumway
2023-11-22 11:55:40 -07:00
parent 9bc3602461
commit 3d0460a9ee
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/main/views/send.py
View File

@@ -275,6 +275,8 @@ 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"]:
return True

2
tests/app/main/views/test_send.py
View File

@@ -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) == 2
assert len(sender_details) == 3
def test_that_test_files_exist():