Skip to content

Commit 659d279

Browse files
committed
Merge pull request #1686 from tseaver/pubsub-fix_iam_policy_system_tests
Fix pubsub IAM policy system tests broken by rename.
2 parents d85bf2f + c393bf0 commit 659d279

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

system_tests/pubsub.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -180,9 +180,9 @@ def test_topic_iam_policy(self):
180180
self.assertTrue(topic.exists())
181181
self.to_delete.append(topic)
182182
policy = topic.get_iam_policy()
183-
policy.readers.add(policy.user('jjg@google.com'))
183+
policy.viewers.add(policy.user('jjg@google.com'))
184184
new_policy = topic.set_iam_policy(policy)
185-
self.assertEqual(new_policy.readers, policy.readers)
185+
self.assertEqual(new_policy.viewers, policy.viewers)
186186

187187
def test_subscription_iam_policy(self):
188188
topic_name = 'test-sub-iam-policy-topic-%d' % (1000 * time.time(),)
@@ -204,6 +204,6 @@ def test_subscription_iam_policy(self):
204204
self.assertTrue(subscription.exists())
205205
self.to_delete.insert(0, subscription)
206206
policy = subscription.get_iam_policy()
207-
policy.readers.add(policy.user('jjg@google.com'))
207+
policy.viewers.add(policy.user('jjg@google.com'))
208208
new_policy = subscription.set_iam_policy(policy)
209-
self.assertEqual(new_policy.readers, policy.readers)
209+
self.assertEqual(new_policy.viewers, policy.viewers)

0 commit comments

Comments
 (0)