openstack
/
heat
Code Issues Proposed changes

Merge "Catch Unauthorized in delete_trust()" into stable/newton

This commit is contained in:
Jenkins 2017-04-19 21:02:49 +00:00 committed by Gerrit Code Review
parent d9979236d9 fadca70990
commit 719107fc2e
2 changed files with 19 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
heat/engine/clients/os/keystone/heat_keystoneclient.py
View File

@ -235,7 +235,7 @@ class KsClientWrapper(object):
"""Delete the specified trust."""
try:
self.client.trusts.delete(trust_id)
except ks_exception.NotFound:
except (ks_exception.NotFound, ks_exception.Unauthorized):
pass
def _get_username(self, username):

34
heat/tests/clients/test_heat_client.py
View File

@ -707,36 +707,38 @@ class KeystoneClientTest(common.HeatTestCase):
heat_ks_client = heat_keystoneclient.KeystoneClient(ctx)
self.assertIsNotNone(heat_ks_client._client)
def test_delete_trust(self):
"""Test delete_trust when deleting trust."""
def _test_delete_trust(self, raise_ext=None):
self._stubs_auth()
cfg.CONF.set_override('deferred_auth_method', 'trusts',
enforce_type=True)
self.mock_ks_v3_client.trusts = self.m.CreateMockAnything()
self.mock_ks_v3_client.trusts.delete('atrust123').AndReturn(None)
if raise_ext is None:
self.mock_ks_v3_client.trusts.delete('atrust123').AndReturn(None)
else:
self.mock_ks_v3_client.trusts.delete('atrust123').AndRaise(
raise_ext)
self.m.ReplayAll()
ctx = utils.dummy_context()
heat_ks_client = heat_keystoneclient.KeystoneClient(ctx)
self.assertIsNone(heat_ks_client.delete_trust(trust_id='atrust123'))
def test_delete_trust(self):
"""Test delete_trust when deleting trust."""
self._test_delete_trust()
def test_delete_trust_not_found(self):
"""Test delete_trust when trust already deleted."""
self._stubs_auth()
cfg.CONF.set_override('deferred_auth_method', 'trusts',
enforce_type=True)
self.mock_ks_v3_client.trusts = self.m.CreateMockAnything()
self.mock_ks_v3_client.trusts.delete('atrust123').AndRaise(
kc_exception.NotFound)
self._test_delete_trust(raise_ext=kc_exception.NotFound)
self.m.ReplayAll()
ctx = utils.dummy_context()
heat_ks_client = heat_keystoneclient.KeystoneClient(ctx)
self.assertIsNone(heat_ks_client.delete_trust(trust_id='atrust123'))
def test_delete_trust_unauthorized(self):
"""Test delete_trust when trustor is deleted or trust is expired."""
self._test_delete_trust(raise_ext=kc_exception.Unauthorized)
def test_disable_stack_user(self):