From 9a2e7ad8b04af065331f8fa772b717f9e9ec2535 Mon Sep 17 00:00:00 2001 From: Simon-Pierre Gingras Date: Mon, 29 Jun 2015 11:04:40 -0400 Subject: [PATCH] IAM: unnecessary check for role --- moto/iam/models.py | 19 +++++-------------- 1 file changed, 5 insertions(+), 14 deletions(-) diff --git a/moto/iam/models.py b/moto/iam/models.py index ed9be53b5..195c2fce4 100644 --- a/moto/iam/models.py +++ b/moto/iam/models.py @@ -256,26 +256,17 @@ class IAMBackend(BaseBackend): def put_role_policy(self, role_name, policy_name, policy_json): role = self.get_role(role_name) - if role: - role.put_policy(policy_name, policy_json) - else: - raise BotoServerError(404, 'Not Found') + role.put_policy(policy_name, policy_json) def get_role_policy(self, role_name, policy_name): role = self.get_role(role_name) - if role: - for p, d in role.policies.items(): - if p == policy_name: - return p, d - else: - raise BotoServerError(404, 'Not Found') + for p, d in role.policies.items(): + if p == policy_name: + return p, d def list_role_policies(self, role_name): role = self.get_role(role_name) - if role: - return role.policies.keys() - else: - raise BotoServerError(404, 'Not Found') + return role.policies.keys() def create_instance_profile(self, name, path, role_ids): instance_profile_id = random_resource_id()