Change method to a get.

Fix path param in invite_api_client.cancel_invited_user
This commit is contained in:
Rebecca Law
2016-03-01 17:00:01 +00:00
parent 219c740071
commit 8e6bd2471d
3 changed files with 6 additions and 6 deletions
+1 -1
View File
@@ -115,7 +115,7 @@ def delete_user(service_id, user_id):
) )
@main.route("/services/<service_id>/cancel-invited-user/<invited_user_id>", methods=['POST']) @main.route("/services/<service_id>/cancel-invited-user/<invited_user_id>", methods=['GET'])
def cancel_invited_user(service_id, invited_user_id): 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) invite_api_client.cancel_invited_user(service_id=service_id, invited_user_id=invited_user_id)
+3 -3
View File
@@ -31,6 +31,6 @@ class InviteApiClient(BaseAPIClient):
def cancel_invited_user(self, service_id, invited_user_id): def cancel_invited_user(self, service_id, invited_user_id):
data = {'status': 'cancelled'} data = {'status': 'cancelled'}
resp = self.post(url='/service/{0}/invite/{0}'.format(service_id, invited_user_id), resp = self.post(url='/service/{0}/invite/{1}'.format(service_id, invited_user_id),
data = data) data=data)
return resp['data'] return resp['data']
+2 -2
View File
@@ -114,8 +114,8 @@ def test_cancel_invited_user_cancels_user_invitations(app_, api_user_active, moc
invited_user_id = uuid.uuid4() invited_user_id = uuid.uuid4()
client.login(api_user_active) client.login(api_user_active)
service_id = uuid.uuid4() service_id = uuid.uuid4()
response = client.post(url_for('main.cancel_invited_user', service_id=service_id, response = client.get(url_for('main.cancel_invited_user', service_id=service_id,
invited_user_id=invited_user_id)) invited_user_id=invited_user_id))
assert response.status_code == 302 assert response.status_code == 302
assert response.location == url_for('main.manage_users', service_id=service_id, _external=True) assert response.location == url_for('main.manage_users', service_id=service_id, _external=True)