darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-02-06 11:23:48 -05:00
Code Issues Packages Projects Releases Wiki Activity

Refactor test to use client fixture

Browse Source
This commit is contained in:
Imdad Ahad
2017-01-24 11:01:44 +00:00
parent 343bf9ab98
commit 4d124a5711
1 changed files with 4 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
tests/app/main/views/test_platform_admin.py
View File

@@ -324,7 +324,7 @@ def test_shows_archived_label_instead_of_live_or_research_mode_label(
def test_should_show_correct_sent_totals_for_platform_admin(
app_,
client,
platform_admin_user,
mocker,
mock_get_detailed_services,
@@ -341,11 +341,9 @@ def test_should_show_correct_sent_totals_for_platform_admin(
)
mock_get_detailed_services.return_value = {'data': services}
with app_.test_request_context():
with app_.test_client() as client:
mock_get_user(mocker, user=platform_admin_user)
client.login(platform_admin_user)
response = client.get(url_for('main.platform_admin'))
mock_get_user(mocker, user=platform_admin_user)
client.login(platform_admin_user)
response = client.get(url_for('main.platform_admin'))
assert response.status_code == 200
page = BeautifulSoup(response.data.decode('utf-8'), 'html.parser')