improve test coverage

googleapis · Jan 9, 2020 · ef37fde · ef37fde
1 parent f84ea6f
commit ef37fde
Showing 1 changed file with 8 additions and 5 deletions.
diff --git a/api_core/tests/unit/test_iam.py b/api_core/tests/unit/test_iam.py
@@ -80,15 +80,18 @@ def test___setitem__(self):
         assert dict(policy) == {"rolename": PRINCIPALS}
 
     def test__set_item__overwrite(self):
+        GROUP = "group:test@group.com"
         USER = "user:phred@example.com"
         ALL_USERS = "allUsers"
         MEMBERS = set([ALL_USERS])
+        GROUPS = set([GROUP])
         policy = self._make_one()
-        policy["rolename"] = [USER]
-        policy["rolename"] = [ALL_USERS]
-        assert policy["rolename"] == MEMBERS
-        assert len(policy) == 1
-        assert dict(policy) == {"rolename": MEMBERS}
+        policy["first"] = [GROUP]
+        policy["second"] = [USER]
+        policy["second"] = [ALL_USERS]
+        assert policy["second"] == MEMBERS
+        assert len(policy) == 2
+        assert dict(policy) == {"first": GROUPS, "second": MEMBERS}
 
     def test___setitem___version3(self):
         policy = self._make_one("DEADBEEF", 3)