Skip to content

Commit

Permalink
Merge pull request #2284 from wndhydrnt/fix-iam-get-role-policy-not-f…
Browse files Browse the repository at this point in the history
…ound

Raise exception if a role policy is not found
  • Loading branch information
spulec committed Jul 8, 2019
2 parents 4e6eb25 + 67326ac commit 6efd558
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 0 deletions.
1 change: 1 addition & 0 deletions moto/iam/models.py
Expand Up @@ -685,6 +685,7 @@ def get_role_policy(self, role_name, policy_name):
for p, d in role.policies.items():
if p == policy_name:
return p, d
raise IAMNotFoundException("Policy Document {0} not attached to role {1}".format(policy_name, role_name))

def list_role_policies(self, role_name):
role = self.get_role(role_name)
Expand Down
9 changes: 9 additions & 0 deletions tests/test_iam/test_iam.py
Expand Up @@ -311,6 +311,15 @@ def test_put_role_policy():
policy.should.equal("test policy")


@mock_iam
def test_get_role_policy():
conn = boto3.client('iam', region_name='us-east-1')
conn.create_role(
RoleName="my-role", AssumeRolePolicyDocument="some policy", Path="my-path")
with assert_raises(conn.exceptions.NoSuchEntityException):
conn.get_role_policy(RoleName="my-role", PolicyName="does-not-exist")


@mock_iam_deprecated()
def test_update_assume_role_policy():
conn = boto.connect_iam()
Expand Down

0 comments on commit 6efd558

Please sign in to comment.