From 1adc6047f0dc56caffb3ab552de948c720b50682 Mon Sep 17 00:00:00 2001 From: Brian Ober Date: Fri, 13 May 2016 16:43:50 -0500 Subject: [PATCH] Bypass Account Mgmt Functional Cases When Disabled The testPUT case is failing when keystone was enabled and allow_account_management is set to True. There were a few issues needing addressed. First the case was renamed to call out what it was actually doing which is verifying an error scenario for which a PUT on a storage account was not allowed. Second the case was running even when allow_account_management is enabled, which is incorrect. It "accidently" works with TempAuth because it requires a reseller permission, so the Keystone failure here has more to do with not requiring a reseller permission to do a PUT on a storage account for which a user has an operator role on. The common sense fix here is to not execute this test case when allow_account_management is enabled. Change-Id: Id29f5ca48f92cd139535be7064107b8a61b02856 --- test/functional/tests.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/test/functional/tests.py b/test/functional/tests.py index fa5b0188bf..9c81c8fc86 100644 --- a/test/functional/tests.py +++ b/test/functional/tests.py @@ -173,7 +173,9 @@ class TestAccount(Base): finally: self.env.account.conn.storage_url = was_url - def testPUT(self): + def testPUTError(self): + if load_constraint('allow_account_management'): + raise SkipTest("Allow account management is enabled") self.env.account.conn.make_request('PUT') self.assert_status([403, 405])