openstack
/
barbican
Code Issues Proposed changes

Fix multiple backend test 

This patch fixes the secret tests for multiple backends.  The tests
were assuming that there would be at least two configured backends
when multiple backends is enabled.   All these test fail when
testing a deployment that has multiple backends = True but only
one configured backend.

Change-Id: I177096155592e14bfe617dd4b79b1e052de3e8c5
This commit is contained in:
Douglas Mendizábal 2019-02-01 08:57:51 -06:00
parent 8eeff8d03e
commit f725e6d97b
1 changed files with 12 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
functionaltests/api/v1/functional/test_secrets.py
View File

@ -1609,18 +1609,23 @@ class SecretsMultipleBackendTestCase(base.TestCase):
self.client) self.client)
self.default_secret_create_data = get_default_data() self.default_secret_create_data = get_default_data()
if base.conf_multiple_backends_enabled: if base.conf_multiple_backends_enabled:
# set preferred secret store for admin_a (project a) user
# and don't set preferred secret store for admin_b (project b) user
resp, stores = self.ss_behaviors.get_all_secret_stores( resp, stores = self.ss_behaviors.get_all_secret_stores(
user_name=admin_a) user_name=admin_a)
self.assertEqual(200, resp.status_code) self.assertEqual(200, resp.status_code)
secret_store_ref = None global_ss = None
first_non_global_ss = None
for store in stores['secret_stores']: for store in stores['secret_stores']:
if not store['global_default']: if store['global_default']:
secret_store_ref = store['secret_store_ref'] global_ss = store['secret_store_ref']
else:
first_non_global_ss = store['secret_store_ref']
break break
# set preferred secret store for admin_a (project a) user self.ss_behaviors.set_preferred_secret_store(
# and don't set preferred secret store for admin_b (project b) user first_non_global_ss or global_ss,
self.ss_behaviors.set_preferred_secret_store(secret_store_ref, user_name=admin_a
user_name=admin_a) )
def tearDown(self): def tearDown(self):
self.behaviors.delete_all_created_secrets() self.behaviors.delete_all_created_secrets()