openstack
/
tacker
Code Issues Proposed changes

Merge "Enable skipped test cases for context"

This commit is contained in:
Zuul 2019-02-04 08:01:18 +00:00 committed by Gerrit Code Review
parent d18afb1d0a 3ed8064011
commit 4efb48cfa2
1 changed files with 2 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
tacker/tests/unit/test_context.py
View File

@ -25,7 +25,6 @@ class TestTackerContext(base.BaseTestCase):
def setUp(self):
super(TestTackerContext, self).setUp()
self.skip("Not ready yet")
db_api = 'tacker.db.api.get_session'
self._db_api_session_patcher = mock.patch(db_api)
self.db_api_session = self._db_api_session_patcher.start()
@ -41,11 +40,6 @@ class TestTackerContext(base.BaseTestCase):
self.assertIsNone(ctx.user_name)
self.assertIsNone(ctx.tenant_name)
def test_tacker_context_create_logs_unknown_kwarg(self):
with mock.patch.object(context.LOG, 'debug') as mock_log:
context.Context('user_id', 'tenant_id', foo=None)
self.assertEqual(1, mock_log.call_count)
def test_tacker_context_create_with_name(self):
ctx = context.Context('user_id', 'tenant_id',
tenant_name='tenant_name', user_name='user_name')
@ -96,12 +90,9 @@ class TestTackerContext(base.BaseTestCase):
self.assertIsNone(ctx_dict['tenant_id'])
self.assertFalse(hasattr(ctx, 'session'))
def test_tacker_context_with_load_roles_true(self):
def test_tacker_context_admin_context(self):
ctx = context.get_admin_context()
self.assertIn('admin', ctx.roles)
def test_tacker_context_with_load_roles_false(self):
ctx = context.get_admin_context(load_admin_roles=False)
self.assertTrue(ctx.is_admin)
self.assertFalse(ctx.roles)
def test_tacker_context_elevated_retains_request_id(self):