diff options
author | Bu Sun Kim <8822365+busunkim96@users.noreply.github.com> | 2021-03-25 05:50:03 -0600 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-03-25 11:50:03 +0000 |
commit | 9eaa7868164a7e98792de24d2be97f79fba22322 (patch) | |
tree | 12ff4af860d6e20fd27300bc000909bef8b9574e /tests | |
parent | 1d76b57d1f218f7885f85dc7c052bad1ad3857ac (diff) | |
download | python-api-core-9eaa7868164a7e98792de24d2be97f79fba22322.tar.gz |
fix: skip empty policy bindings in `len()` and `iter()` (#159)
Exclude empty policy bindings (bindings with no members) in
`Policy.__iter__()` and `Policy.__len__()`
Follow up to #155
Diffstat (limited to 'tests')
-rw-r--r-- | tests/unit/test_iam.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/tests/unit/test_iam.py b/tests/unit/test_iam.py index f9771f0..0da9b23 100644 --- a/tests/unit/test_iam.py +++ b/tests/unit/test_iam.py @@ -32,11 +32,11 @@ class TestPolicy: policy = self._make_one() assert policy.etag is None assert policy.version is None - assert len(policy) == 0 - assert dict(policy) == {} assert policy.owners == empty assert policy.editors == empty assert policy.viewers == empty + assert len(policy) == 0 + assert dict(policy) == {} def test_ctor_explicit(self): VERSION = 1 @@ -45,11 +45,11 @@ class TestPolicy: policy = self._make_one(ETAG, VERSION) assert policy.etag == ETAG assert policy.version == VERSION - assert len(policy) == 0 - assert dict(policy) == {} assert policy.owners == empty assert policy.editors == empty assert policy.viewers == empty + assert len(policy) == 0 + assert dict(policy) == {} def test___getitem___miss(self): policy = self._make_one() @@ -301,10 +301,10 @@ class TestPolicy: policy = klass.from_api_repr(RESOURCE) assert policy.etag == "ACAB" assert policy.version is None - assert dict(policy) == {} assert policy.owners == empty assert policy.editors == empty assert policy.viewers == empty + assert dict(policy) == {} def test_from_api_repr_complete(self): from google.api_core.iam import OWNER_ROLE, EDITOR_ROLE, VIEWER_ROLE |