From 70315fd67ce92ca723ca22457171c7d5d7444056 Mon Sep 17 00:00:00 2001 From: Mike Grima Date: Fri, 27 Feb 2015 12:22:31 -0800 Subject: [PATCH] Fixed how parameters are passed in following clarification on GitHub comments. --- moto/core/responses.py | 5 +---- moto/iam/responses.py | 3 ++- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/moto/core/responses.py b/moto/core/responses.py index 5558e9a02..a99ac78c8 100644 --- a/moto/core/responses.py +++ b/moto/core/responses.py @@ -137,10 +137,7 @@ class BaseResponse(_TemplateEnvironmentMixin): if action in method_names: method = getattr(self, action) try: - if action != 'list_instance_profiles_for_role': - response = method() - else: - response = method(role_name=self.querystring['RoleName'][0]) + response = method() except HTTPException as http_error: response = http_error.description, dict(status=http_error.code) if isinstance(response, six.string_types): diff --git a/moto/iam/responses.py b/moto/iam/responses.py index e1722162b..4ebfb74ec 100644 --- a/moto/iam/responses.py +++ b/moto/iam/responses.py @@ -87,7 +87,8 @@ class IamResponse(BaseResponse): template = self.response_template(LIST_INSTANCE_PROFILES_TEMPLATE) return template.render(instance_profiles=profiles) - def list_instance_profiles_for_role(self, role_name=None): + def list_instance_profiles_for_role(self): + role_name = self._get_param('RoleName') profiles = iam_backend.get_instance_profiles_for_role(role_name=role_name) template = self.response_template(LIST_INSTANCE_PROFILES_FOR_ROLE_TEMPLATE)