Yasuke/moto
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Organisations - Detach policy, and asserts it actually happens (#3759)

Browse Source
This commit is contained in:
Bert Blommers 2021-08-22 10:49:48 +01:00 committed by GitHub
parent bb3cbd0bb4
commit 5f7167ce62
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 28 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
moto/organizations/models.py
View File

@ -589,7 +589,7 @@ class OrganizationsBackend(BaseBackend):
).match(kwargs["TargetId"]):
ou = next((ou for ou in self.ou if ou.id == kwargs["TargetId"]), None)
if ou is not None:
if ou not in ou.attached_policies:
if policy not in ou.attached_policies:
ou.attached_policies.append(policy)
policy.attachments.append(ou)
else:
@ -602,7 +602,7 @@ class OrganizationsBackend(BaseBackend):
(a for a in self.accounts if a.id == kwargs["TargetId"]), None
)
if account is not None:
if account not in account.attached_policies:
if policy not in account.attached_policies:
account.attached_policies.append(policy)
policy.attachments.append(account)
else:
@ -866,7 +866,7 @@ class OrganizationsBackend(BaseBackend):
if re.match(root_id_regex, target_id) or re.match(ou_id_regex, target_id):
ou = next((ou for ou in self.ou if ou.id == target_id), None)
if ou is not None:
if ou in ou.attached_policies:
if policy in ou.attached_policies:
ou.attached_policies.remove(policy)
policy.attachments.remove(ou)
else:
@ -879,7 +879,7 @@ class OrganizationsBackend(BaseBackend):
(account for account in self.accounts if account.id == target_id), None,
)
if account is not None:
if account in account.attached_policies:
if policy in account.attached_policies:
account.attached_policies.remove(policy)
policy.attachments.remove(account)
else:

33
tests/test_organizations/test_organizations_boto3.py
View File

@ -556,15 +556,30 @@ def test_detach_policy():
Name="MockServiceControlPolicy",
Type="SERVICE_CONTROL_POLICY",
)["Policy"]["PolicySummary"]["Id"]
client.attach_policy(PolicyId=policy_id, TargetId=ou_id)
client.attach_policy(PolicyId=policy_id, TargetId=root_id)
client.attach_policy(PolicyId=policy_id, TargetId=account_id)
response = client.detach_policy(PolicyId=policy_id, TargetId=ou_id)
response["ResponseMetadata"]["HTTPStatusCode"].should.equal(200)
response = client.detach_policy(PolicyId=policy_id, TargetId=root_id)
response["ResponseMetadata"]["HTTPStatusCode"].should.equal(200)
response = client.detach_policy(PolicyId=policy_id, TargetId=account_id)
response["ResponseMetadata"]["HTTPStatusCode"].should.equal(200)
# Attach/List/Detach policy
for name, target in [("OU", ou_id), ("Root", root_id), ("Account", account_id)]:
#
with sure.ensure("We should start with 0 policies"):
get_nonaws_policies(target, client).should.have.length_of(0)
#
client.attach_policy(PolicyId=policy_id, TargetId=target)
with sure.ensure("Expecting 1 policy after creation of target={0}", name):
get_nonaws_policies(target, client).should.have.length_of(1)
#
response = client.detach_policy(PolicyId=policy_id, TargetId=target)
response["ResponseMetadata"]["HTTPStatusCode"].should.equal(200)
with sure.ensure("Expecting 0 policies after deletion of target={0}", name):
get_nonaws_policies(target, client).should.have.length_of(0)
def get_nonaws_policies(account_id, client):
return [
p
for p in client.list_policies_for_target(
TargetId=account_id, Filter="SERVICE_CONTROL_POLICY"
)["Policies"]
if not p["AwsManaged"]
]
@mock_organizations