Skip to content

Commit

Permalink
Merge pull request #1686 from tseaver/pubsub-fix_iam_policy_system_tests
Browse files Browse the repository at this point in the history
Fix pubsub IAM policy system tests broken by rename.
  • Loading branch information
tseaver committed Mar 31, 2016
2 parents d85bf2f + c393bf0 commit 659d279
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions system_tests/pubsub.py
Original file line number Diff line number Diff line change
Expand Up @@ -180,9 +180,9 @@ def test_topic_iam_policy(self):
self.assertTrue(topic.exists())
self.to_delete.append(topic)
policy = topic.get_iam_policy()
policy.readers.add(policy.user('jjg@google.com'))
policy.viewers.add(policy.user('jjg@google.com'))
new_policy = topic.set_iam_policy(policy)
self.assertEqual(new_policy.readers, policy.readers)
self.assertEqual(new_policy.viewers, policy.viewers)

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

0 comments on commit 659d279

Please sign in to comment.