Skip to content

Commit

Permalink
Drop exercising extra bucket / blob ACL failures under BPO.
Browse files Browse the repository at this point in the history
  • Loading branch information
tseaver committed Jan 17, 2019
1 parent 3b43092 commit 693838f
Showing 1 changed file with 0 additions and 13 deletions.
13 changes: 0 additions & 13 deletions storage/tests/system.py
Original file line number Diff line number Diff line change
Expand Up @@ -1543,19 +1543,6 @@ def test_bpo_set_unset_preserves_acls(self):
with self.assertRaises(exceptions.BadRequest):
bucket.acl.reload()

# XXX This should raise, but doesn't as of 2018-01-08T19:00:00Z
#with self.assertRaises(exceptions.BadRequest):
# bucket.acl.clear()

# XXX The blob ACL get / set stuff raises 403, rather than 400.
#with self.assertRaises(exceptions.BadRequest):
with self.assertRaises(exceptions.Forbidden):
blob.acl.reload()

#with self.assertRaises(exceptions.BadRequest):
with self.assertRaises(exceptions.Forbidden):
blob.acl.clear()

# Clear BPO
bucket.iam_configuration.bucket_policy_only = False
bucket.patch()
Expand Down

0 comments on commit 693838f

Please sign in to comment.