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

add new get_detailed_service mock to tests that the dashboard

Browse Source
This commit is contained in:
Leo Hemsted
2016-07-20 13:55:08 +01:00
parent 3ffd6c744c
commit fade656e3b
4 changed files with 24 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
tests/app/main/views/test_accept_invite.py
View File

@@ -314,7 +314,10 @@ def test_new_invited_user_verifies_and_added_to_service(app_,
mock_get_service_statistics,
mock_get_template_statistics,
mock_get_jobs,
mock_has_permissions):
mock_has_permissions,
mock_get_users_by_service,
mock_get_detailed_service,
mock_get_usage):
with app_.test_request_context():
with app_.test_client() as client:

16
tests/app/main/views/test_dashboard.py
View File

@@ -69,6 +69,7 @@ def test_get_started(
mock_login,
mock_get_jobs,
mock_has_permissions,
mock_get_detailed_service,
mock_get_usage
):
@@ -98,11 +99,11 @@ def test_get_started_is_hidden_once_templates_exist(
mock_login,
mock_get_jobs,
mock_has_permissions,
mock_get_detailed_service,
mock_get_usage
):
mock_template_stats = mocker.patch('app.template_statistics_client.get_template_statistics_for_service',
return_value=copy.deepcopy(stub_template_stats))
with app_.test_request_context(), app_.test_client() as client:
client.login(api_user_active)
response = client.get(url_for('main.service_dashboard', service_id=SERVICE_ONE_ID))
@@ -117,13 +118,13 @@ def test_should_show_recent_templates_on_dashboard(app_,
api_user_active,
mock_get_service,
mock_get_service_templates,
mock_get_service_statistics,
mock_get_aggregate_service_statistics,
mock_get_user,
mock_get_user_by_email,
mock_login,
mock_get_jobs,
mock_has_permissions,
mock_get_detailed_service,
mock_get_usage):
mock_template_stats = mocker.patch('app.template_statistics_client.get_template_statistics_for_service',
@@ -136,7 +137,6 @@ def test_should_show_recent_templates_on_dashboard(app_,
assert response.status_code == 200
response.get_data(as_text=True)
mock_get_service_statistics.assert_called_once_with(SERVICE_ONE_ID, limit_days=7)
mock_template_stats.assert_called_once_with(SERVICE_ONE_ID, limit_days=7)
page = BeautifulSoup(response.data.decode('utf-8'), 'html.parser')
@@ -210,6 +210,7 @@ def test_should_show_recent_jobs_on_dashboard(
mock_get_user_by_email,
mock_login,
mock_get_template_statistics,
mock_get_detailed_service,
mock_get_jobs,
mock_has_permissions,
mock_get_usage
@@ -261,6 +262,7 @@ def test_menu_send_messages(mocker,
mock_get_service_templates,
mock_get_jobs,
mock_get_template_statistics,
mock_get_detailed_service,
mock_get_usage):
with app_.test_request_context():
@@ -295,6 +297,7 @@ def test_menu_manage_service(mocker,
mock_get_service_templates,
mock_get_jobs,
mock_get_template_statistics,
mock_get_detailed_service,
mock_get_usage):
with app_.test_request_context():
resp = _test_dashboard_menu(
@@ -327,6 +330,7 @@ def test_menu_manage_api_keys(mocker,
mock_get_service_templates,
mock_get_jobs,
mock_get_template_statistics,
mock_get_detailed_service,
mock_get_usage):
with app_.test_request_context():
resp = _test_dashboard_menu(
@@ -358,6 +362,7 @@ def test_menu_all_services_for_platform_admin_user(mocker,
mock_get_service_templates,
mock_get_jobs,
mock_get_template_statistics,
mock_get_detailed_service,
mock_get_usage):
with app_.test_request_context():
resp = _test_dashboard_menu(
@@ -386,6 +391,7 @@ def test_route_for_service_permissions(mocker,
mock_get_jobs,
mock_get_service_statistics,
mock_get_template_statistics,
mock_get_detailed_service,
mock_get_usage):
routes = [
'main.service_dashboard']
@@ -424,10 +430,10 @@ def test_service_dashboard_updates_gets_dashboard_totals(mocker,
mock_get_user,
mock_get_service_templates,
mock_get_template_statistics,
mock_get_detailed_service,
mock_get_jobs,
mock_get_service_statistics,
mock_get_usage,
mock_get_detailed_service):
mock_get_usage):
dashboard_totals = mocker.patch('app.main.views.dashboard.get_dashboard_totals', return_value={
'email': {'requested': 0, 'delivered': 0, 'failed': 0},
'sms': {'requested': 0, 'delivered': 0, 'failed': 0}

1
tests/app/main/views/test_sign_out.py
View File

@@ -21,6 +21,7 @@ def test_sign_out_user(app_,
mock_get_jobs,
mock_has_permissions,
mock_get_template_statistics,
mock_get_detailed_service,
mock_get_usage):
with app_.test_request_context():
with app_.test_client() as client:

15
tests/conftest.py
View File

@@ -67,14 +67,15 @@ def mock_get_service(mocker, api_user_active):
@pytest.fixture(scope='function')
def mock_get_detailed_service(mocker, api_user_active):
def _get(service_id):
service = service_json(
service_id, "Test Service", [api_user_active.id], message_limit=1000,
active=False, restricted=True)
service['statistics'] = {
'email': {'requested': 0, 'delivered': 0, 'failed': 0},
'sms': {'requested': 0, 'delivered': 0, 'failed': 0}
return {
'data': {
'id': service_id,
'statistics': {
'email': {'requested': 0, 'delivered': 0, 'failed': 0},
'sms': {'requested': 0, 'delivered': 0, 'failed': 0}
}
}
}
return {'data': service}
return mocker.patch('app.service_api_client.get_detailed_service', side_effect=_get)