aboutsummaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorBu Sun Kim <8822365+busunkim96@users.noreply.github.com>2021-03-25 05:50:03 -0600
committerGitHub <noreply@github.com>2021-03-25 11:50:03 +0000
commit9eaa7868164a7e98792de24d2be97f79fba22322 (patch)
tree12ff4af860d6e20fd27300bc000909bef8b9574e /tests
parent1d76b57d1f218f7885f85dc7c052bad1ad3857ac (diff)
downloadpython-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.py10
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