diff --git a/app/main/views/manage_users.py b/app/main/views/manage_users.py index 87fa3154d..e1a8f527d 100644 --- a/app/main/views/manage_users.py +++ b/app/main/views/manage_users.py @@ -115,7 +115,7 @@ def delete_user(service_id, user_id): ) -@main.route("/services//cancel-invited-user/", methods=['POST']) +@main.route("/services//cancel-invited-user/", methods=['GET']) def cancel_invited_user(service_id, invited_user_id): invite_api_client.cancel_invited_user(service_id=service_id, invited_user_id=invited_user_id) diff --git a/app/notify_client/invite_api_client.py b/app/notify_client/invite_api_client.py index 4e9dc8639..9f7bd9040 100644 --- a/app/notify_client/invite_api_client.py +++ b/app/notify_client/invite_api_client.py @@ -31,6 +31,6 @@ class InviteApiClient(BaseAPIClient): def cancel_invited_user(self, service_id, invited_user_id): data = {'status': 'cancelled'} - resp = self.post(url='/service/{0}/invite/{0}'.format(service_id, invited_user_id), - data = data) - return resp['data'] \ No newline at end of file + resp = self.post(url='/service/{0}/invite/{1}'.format(service_id, invited_user_id), + data=data) + return resp['data'] diff --git a/tests/app/main/views/test_manage_users.py b/tests/app/main/views/test_manage_users.py index 6a539b7ed..0fe95508a 100644 --- a/tests/app/main/views/test_manage_users.py +++ b/tests/app/main/views/test_manage_users.py @@ -114,8 +114,8 @@ def test_cancel_invited_user_cancels_user_invitations(app_, api_user_active, moc invited_user_id = uuid.uuid4() client.login(api_user_active) service_id = uuid.uuid4() - response = client.post(url_for('main.cancel_invited_user', service_id=service_id, - invited_user_id=invited_user_id)) + response = client.get(url_for('main.cancel_invited_user', service_id=service_id, + invited_user_id=invited_user_id)) assert response.status_code == 302 assert response.location == url_for('main.manage_users', service_id=service_id, _external=True)