From 330c23ad3061f48506f9872d08459946d903929f Mon Sep 17 00:00:00 2001 From: Ben Thorner Date: Wed, 23 Mar 2022 14:17:47 +0000 Subject: [PATCH] Remove excess tests for other org types These all behave the same as each other so there's little value in testing all of them - if we had 100 org types we wouldn't test them all, but it's easy to get carried away when there are fewer. --- tests/app/utils/test_branding.py | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/tests/app/utils/test_branding.py b/tests/app/utils/test_branding.py index 8027f261c..25603a165 100644 --- a/tests/app/utils/test_branding.py +++ b/tests/app/utils/test_branding.py @@ -12,10 +12,6 @@ from tests import organisation_json ('central', []), ('local', []), ('nhs_central', [('nhs', 'NHS')]), - ('nhs_local', [('nhs', 'NHS')]), - ('nhs_gp', [('nhs', 'NHS')]), - ('emergency_service', []), - ('other', []), ]) def test_get_choices_service_not_assigned_to_org( service_one, @@ -37,10 +33,6 @@ def test_get_choices_service_not_assigned_to_org( ]), ('local', [('organisation', 'Test Organisation')]), ('nhs_central', [('nhs', 'NHS')]), - ('nhs_local', [('nhs', 'NHS')]), - ('nhs_gp', [('nhs', 'NHS')]), - ('emergency_service', [('organisation', 'Test Organisation')]), - ('other', [('organisation', 'Test Organisation')]), ]) def test_get_email_choices_service_assigned_to_org( mocker, @@ -89,10 +81,6 @@ def test_get_email_choices_central_org_includes_govuk( ('central', [('organisation', 'Test Organisation')]), ('local', [('organisation', 'Test Organisation')]), ('nhs_central', [('nhs', 'NHS')]), - ('nhs_local', [('nhs', 'NHS')]), - ('nhs_gp', [('nhs', 'NHS')]), - ('emergency_service', [('organisation', 'Test Organisation')]), - ('other', [('organisation', 'Test Organisation')]), ]) def test_get_letter_choices_service_assigned_to_org( mocker,