Fixed policy version's Document type for AWS managed policies (#2234)
* Added checking Document's type in AWS managed policy version response. * Fixed policy version's Document type for AWS managed policies.
This commit is contained in:
parent
3833449b36
commit
ab8a189bbf
@ -131,7 +131,7 @@ class AWSManagedPolicy(ManagedPolicy):
|
|||||||
return cls(name,
|
return cls(name,
|
||||||
default_version_id=data.get('DefaultVersionId'),
|
default_version_id=data.get('DefaultVersionId'),
|
||||||
path=data.get('Path'),
|
path=data.get('Path'),
|
||||||
document=data.get('Document'),
|
document=json.dumps(data.get('Document')),
|
||||||
create_date=datetime.strptime(data.get('CreateDate'), "%Y-%m-%dT%H:%M:%S+00:00"),
|
create_date=datetime.strptime(data.get('CreateDate'), "%Y-%m-%dT%H:%M:%S+00:00"),
|
||||||
update_date=datetime.strptime(data.get('UpdateDate'), "%Y-%m-%dT%H:%M:%S+00:00"))
|
update_date=datetime.strptime(data.get('UpdateDate'), "%Y-%m-%dT%H:%M:%S+00:00"))
|
||||||
|
|
||||||
|
@ -369,6 +369,7 @@ def test_get_aws_managed_policy_version():
|
|||||||
PolicyArn=managed_policy_arn,
|
PolicyArn=managed_policy_arn,
|
||||||
VersionId="v1")
|
VersionId="v1")
|
||||||
retrieved['PolicyVersion']['CreateDate'].replace(tzinfo=None).should.equal(managed_policy_version_create_date)
|
retrieved['PolicyVersion']['CreateDate'].replace(tzinfo=None).should.equal(managed_policy_version_create_date)
|
||||||
|
retrieved['PolicyVersion']['Document'].should.be.an(dict)
|
||||||
|
|
||||||
|
|
||||||
@mock_iam
|
@mock_iam
|
||||||
@ -384,6 +385,7 @@ def test_get_aws_managed_policy_v4_version():
|
|||||||
PolicyArn=managed_policy_arn,
|
PolicyArn=managed_policy_arn,
|
||||||
VersionId="v4")
|
VersionId="v4")
|
||||||
retrieved['PolicyVersion']['CreateDate'].replace(tzinfo=None).should.equal(managed_policy_version_create_date)
|
retrieved['PolicyVersion']['CreateDate'].replace(tzinfo=None).should.equal(managed_policy_version_create_date)
|
||||||
|
retrieved['PolicyVersion']['Document'].should.be.an(dict)
|
||||||
|
|
||||||
|
|
||||||
@mock_iam
|
@mock_iam
|
||||||
|
Loading…
Reference in New Issue
Block a user