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

removed comments and updated test-service_settings

Browse Source
This commit is contained in:
samathad2023
2024-03-13 10:28:07 -07:00
parent 7ff7455022
commit 37f6579ca8
2 changed files with 1 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
tests/app/main/views/service_settings/test_service_settings.py
View File

@@ -2307,7 +2307,7 @@ def test_resume_service_after_confirm(
) )
mock_api.assert_called_once_with( mock_api.assert_called_once_with(
"/service/{}/resume".format(SERVICE_ONE_ID), data=None f"/service/{SERVICE_ONE_ID}/resume", data=None
) )
mock_event.assert_called_once_with( mock_event.assert_called_once_with(
service_id=SERVICE_ONE_ID, resumed_by_id=user["id"] service_id=SERVICE_ONE_ID, resumed_by_id=user["id"]

11
tests/app/main/views/test_find_users.py
View File

@@ -23,7 +23,6 @@ def test_find_users_by_email_displays_users_found(
mocker.patch( mocker.patch(
"app.user_api_client.find_users_by_full_or_partial_email", "app.user_api_client.find_users_by_full_or_partial_email",
return_value={"data": [user_json()]}, return_value={"data": [user_json()]},
# autospec=True, # not need with 3.12
) )
document = client_request.post( document = client_request.post(
"main.find_users_by_email", "main.find_users_by_email",
@@ -57,7 +56,6 @@ def test_find_users_by_email_displays_multiple_users(
return_value={ return_value={
"data": [user_json(name="Apple Jack"), user_json(name="Apple Bloom")] "data": [user_json(name="Apple Jack"), user_json(name="Apple Bloom")]
}, },
# autospec=True, # not need with 3.12
) )
document = client_request.post( document = client_request.post(
"main.find_users_by_email", _data={"search": "apple"}, _expected_status=200 "main.find_users_by_email", _data={"search": "apple"}, _expected_status=200
@@ -80,7 +78,6 @@ def test_find_users_by_email_displays_message_if_no_users_found(
mocker.patch( mocker.patch(
"app.user_api_client.find_users_by_full_or_partial_email", "app.user_api_client.find_users_by_full_or_partial_email",
return_value={"data": []}, return_value={"data": []},
# autospec=True, # not need with 3.12
) )
document = client_request.post( document = client_request.post(
"main.find_users_by_email", "main.find_users_by_email",
@@ -125,7 +122,6 @@ def test_user_information_page_shows_information_about_user(
side_effect=[ side_effect=[
user_json(name="Apple Bloom", services=[user_service_one, user_service_two]) user_json(name="Apple Bloom", services=[user_service_one, user_service_two])
], ],
# autospec=True, # not need with 3.12
) )
mocker.patch( mocker.patch(
@@ -141,7 +137,6 @@ def test_user_information_page_shows_information_about_user(
{"id": user_service_two, "name": "Nature Therapy", "restricted": False}, {"id": user_service_two, "name": "Nature Therapy", "restricted": False},
], ],
}, },
# autospec=True, # not need with 3.12
) )
page = client_request.get("main.user_information", user_id=fake_uuid) page = client_request.get("main.user_information", user_id=fake_uuid)
@@ -184,7 +179,6 @@ def test_user_information_page_shows_change_auth_type_link(
id_=api_user_active["id"], name="Apple Bloom", auth_type="sms_auth" id_=api_user_active["id"], name="Apple Bloom", auth_type="sms_auth"
) )
], ],
# autospec=True, # not need with 3.12
) )
page = client_request.get("main.user_information", user_id=api_user_active["id"]) page = client_request.get("main.user_information", user_id=api_user_active["id"])
@@ -209,7 +203,6 @@ def test_change_user_auth_preselects_current_auth_type(
auth_type=current_auth_type, auth_type=current_auth_type,
) )
], ],
# autospec=True, # not need with 3.12
) )
checked_radios = client_request.get( checked_radios = client_request.get(
@@ -231,7 +224,6 @@ def test_change_user_auth(client_request, platform_admin_user, api_user_active,
id_=api_user_active["id"], name="Apple Bloom", auth_type="sms_auth" id_=api_user_active["id"], name="Apple Bloom", auth_type="sms_auth"
) )
], ],
# autospec=True, # not need with 3.12
) )
mock_update = mocker.patch("app.user_api_client.update_user_attribute") mock_update = mocker.patch("app.user_api_client.update_user_attribute")
@@ -261,13 +253,11 @@ def test_user_information_page_displays_if_there_are_failed_login_attempts(
mocker.patch( mocker.patch(
"app.user_api_client.get_user", "app.user_api_client.get_user",
side_effect=[user_json(name="Apple Bloom", failed_login_count=2)], side_effect=[user_json(name="Apple Bloom", failed_login_count=2)],
# autospec=True, # not need with 3.12
) )
mocker.patch( mocker.patch(
"app.user_api_client.get_organizations_and_services_for_user", "app.user_api_client.get_organizations_and_services_for_user",
return_value={"organizations": [], "services": []}, return_value={"organizations": [], "services": []},
# autospec=True, # not need with 3.12
) )
page = client_request.get("main.user_information", user_id=fake_uuid) page = client_request.get("main.user_information", user_id=fake_uuid)
@@ -302,7 +292,6 @@ def test_user_information_page_does_not_show_archive_link_for_inactive_users(
mocker.patch( mocker.patch(
"app.user_api_client.get_user", "app.user_api_client.get_user",
side_effect=[platform_admin_user, inactive_user], side_effect=[platform_admin_user, inactive_user],
# autospec=True, # not need with 3.12
) )
page = client_request.get("main.user_information", user_id=inactive_user_id) page = client_request.get("main.user_information", user_id=inactive_user_id)