From cfbe767dff29d518daca39f1133f108dec6e7a5d Mon Sep 17 00:00:00 2001 From: Chris Hill-Scott Date: Wed, 12 Jul 2017 14:39:50 +0100 Subject: [PATCH] Test permissions on the inbox updates --- tests/app/main/views/test_dashboard.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/tests/app/main/views/test_dashboard.py b/tests/app/main/views/test_dashboard.py index a4560fbfa..4a0074bd2 100644 --- a/tests/app/main/views/test_dashboard.py +++ b/tests/app/main/views/test_dashboard.py @@ -189,12 +189,17 @@ def test_empty_inbox( ) +@pytest.mark.parametrize('endpoint', [ + 'main.inbox', + 'main.inbox_updates', +]) def test_inbox_not_accessible_to_service_without_permissions( logged_in_client, service_one, + endpoint, ): service_one['permissions'] = [] - response = logged_in_client.get(url_for('main.inbox', service_id=SERVICE_ONE_ID)) + response = logged_in_client.get(url_for(endpoint, service_id=SERVICE_ONE_ID)) assert response.status_code == 403 @@ -228,8 +233,6 @@ def test_view_inbox_updates( mock_get_inbound_sms_with_no_messages, ): - service_one['permissions'] = ['inbound_sms'] - mock_get_partials = mocker.patch( 'app.main.views.dashboard.get_inbox_partials', return_value={'messages': 'foo'},