darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-24 16:24:08 -04:00
Code Issues Packages Projects Releases Wiki Activity

Use new API endpoint to get single contact list

Browse Source 
When we first built the contact list feature this endpoint didn’t exist.
Now it does we can remove the slightly cludgy looping-through-all-the-lists
code.
This commit is contained in:
Chris Hill-Scott
2020-03-17 11:30:27 +00:00
parent cb555a4cbe
commit 82634f79de
5 changed files with 42 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -3988,7 +3988,7 @@ def test_send_from_contact_list(
mocker,
client_request,
fake_uuid,
mock_get_contact_lists,
mock_get_contact_list,
):
new_uuid = uuid.uuid4()
mock_download = mocker.patch('app.models.contact_list.s3download', return_value='contents')

6
tests/app/main/views/test_uploads.py
View File

@@ -1183,7 +1183,7 @@ def test_cant_save_bad_contact_list(
def test_view_contact_list(
mocker,
client_request,
mock_get_contact_lists,
mock_get_contact_list,
fake_uuid,
):
mocker.patch('app.models.contact_list.s3download', return_value='\n'.join(
@@ -1225,7 +1225,7 @@ def test_view_contact_list(
def test_view_contact_list_404s_for_non_existing_list(
client_request,
mock_get_contact_lists,
mock_get_no_contact_list,
fake_uuid,
):
client_request.get(
@@ -1240,7 +1240,7 @@ def test_download_contact_list(
mocker,
logged_in_client,
fake_uuid,
mock_get_contact_lists,
mock_get_contact_list,
):
mocker.patch(
'app.models.contact_list.s3download',