From e2929118c8246c161502fe667a1855198626911a Mon Sep 17 00:00:00 2001 From: Ben Thorner Date: Wed, 23 Mar 2022 15:46:29 +0000 Subject: [PATCH] Remove redundant letter branding UI test This is now covered by the unit tests for the new branding module. Showing the "something else" text box is also already covered by another UI test, so we don't need it for that either. --- tests/app/main/views/test_service_settings.py | 26 ------------------- 1 file changed, 26 deletions(-) diff --git a/tests/app/main/views/test_service_settings.py b/tests/app/main/views/test_service_settings.py index 3b1a02114..da7f82d83 100644 --- a/tests/app/main/views/test_service_settings.py +++ b/tests/app/main/views/test_service_settings.py @@ -4902,32 +4902,6 @@ def test_letter_branding_request_page_back_link( assert back_link[0].attrs['href'] == back_link_url -def test_letter_branding_request_page_when_branding_is_same_as_org( - mocker, - service_one, - client_request, - mock_get_letter_branding_by_id, - mock_get_service_organisation, - active_user_with_permissions, -): - service_one['letter_branding'] = sample_uuid() - mocker.patch( - 'app.organisations_client.get_organisation', - return_value=organisation_json(letter_branding_id=service_one['letter_branding']), - ) - - page = client_request.get( - '.letter_branding_request', service_id=SERVICE_ONE_ID - ) - - # Central government organisations who have their own default - # branding will do so because they’re exempt from GOV.UK. - # We also don’t show their organisation’s branding because they - # have it already. So ‘Something else’ is the only option. - assert not page.select('input[type=radio]') - assert page.select_one('textarea')['name'] == 'something_else' - - @pytest.mark.parametrize('data, org_type, endpoint', ( ( {