diff --git a/app/schemas.py b/app/schemas.py index 10cf6e90f..c985d1812 100644 --- a/app/schemas.py +++ b/app/schemas.py @@ -235,6 +235,18 @@ class ServiceSchema(BaseSchema): 'letter_contacts', 'complaints', 'data_retention', + 'all_template_folders', + 'annual_billing', + 'contact_list', + 'crown', + 'inbound_number', + 'inbound_sms', + 'letter_logo_filename', + 'rate_limit', + 'returned_letters', + 'users', + 'version', + 'whitelist', ) strict = True @@ -289,7 +301,18 @@ class DetailedServiceSchema(BaseSchema): 'sms_sender', 'permissions', 'inbound_number', - 'inbound_sms' + 'inbound_sms', + 'all_template_folders', + 'annual_billing', + 'contact_list', + 'created_by', + 'crown', + 'letter_logo_filename', + 'rate_limit', + 'returned_letters', + 'users', + 'version', + 'whitelist', ) diff --git a/tests/app/service/test_rest.py b/tests/app/service/test_rest.py index 31e27460c..159e45117 100644 --- a/tests/app/service/test_rest.py +++ b/tests/app/service/test_rest.py @@ -235,7 +235,6 @@ def test_get_service_by_id(admin_request, sample_service): assert json_resp['data']['email_branding'] is None assert 'branding' not in json_resp['data'] assert json_resp['data']['prefix_sms'] is True - assert json_resp['data']['letter_logo_filename'] is None @pytest.mark.parametrize('detailed', [True, False]) @@ -347,7 +346,6 @@ def test_create_service( assert json_resp['data']['name'] == 'created service' assert json_resp['data']['email_from'] == 'created.service' assert not json_resp['data']['research_mode'] - assert json_resp['data']['rate_limit'] == 3000 assert json_resp['data']['letter_branding'] is None assert json_resp['data']['count_as_live'] is expected_count_as_live @@ -1225,7 +1223,6 @@ def test_add_existing_user_to_another_service_with_all_permissions( ) assert resp.status_code == 200 json_resp = resp.json - assert str(user_to_add.id) in json_resp['data']['users'] # check user has all permissions auth_header = create_authorization_header()