From 4eaa482b2d5f1ea8c68f6c69570ad56e4e534efe Mon Sep 17 00:00:00 2001 From: Chris Hill-Scott Date: Fri, 10 Nov 2017 08:42:00 +0000 Subject: [PATCH] Rewrite existing test to use admin_request fixture Makes it consistent with the other, related, test in this PR. --- tests/app/service/test_rest.py | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/tests/app/service/test_rest.py b/tests/app/service/test_rest.py index 5e3fdddb6..2df3ef0da 100644 --- a/tests/app/service/test_rest.py +++ b/tests/app/service/test_rest.py @@ -1553,25 +1553,18 @@ def test_prefixing_messages_based_on_prefix_sms( (False, False, False), ]) def test_set_sms_prefixing_for_service( + admin_request, client, sample_service, posted_value, stored_value, returned_value, ): - data = { - 'prefix_sms': posted_value, - } - - auth_header = create_authorization_header() - - resp = client.post( - '/service/{}'.format(sample_service.id), - data=json.dumps(data), - headers=[('Content-Type', 'application/json'), auth_header] + result = admin_request.post( + 'service.update_service', + service_id=sample_service.id, + _data={'prefix_sms': posted_value}, ) - result = json.loads(resp.get_data(as_text=True)) - assert resp.status_code == 200 assert result['data']['prefix_sms'] == stored_value # This derived value will go away eventually, once we’ve done a migration assert result['data']['prefix_sms_with_service_name'] == returned_value