Merge "Fix context from_dict() for system_scope"

This commit is contained in:
Zuul 2021-10-04 19:55:17 +00:00 committed by Gerrit Code Review
commit 57a4a5f576
3 changed files with 8 additions and 1 deletions

View File

@ -419,6 +419,7 @@ class RequestContext(object):
values.get('project_domain_name'))
kwargs.setdefault('is_admin_project',
values.get('is_admin_project', True))
kwargs.setdefault('system_scope', values.get('system_scope'))
for key in cls.FROM_DICT_EXTRA_KEYS:
kwargs.setdefault(key, values.get(key))
return cls(**kwargs)

View File

@ -143,7 +143,8 @@ class ContextTest(test_base.BaseTestCase):
"request_id": "request1",
"global_request_id": "req-uuid",
"resource_uuid": "instance1",
"extra_data": "foo"
"extra_data": "foo",
"system_scope": "all"
}
ctx = context.RequestContext.from_dict(dct)
self.assertEqual(dct['auth_token'], ctx.auth_token)
@ -163,6 +164,7 @@ class ContextTest(test_base.BaseTestCase):
self.assertEqual(dct['domain_name'], ctx.domain_name)
self.assertEqual(dct['user_domain_name'], ctx.user_domain_name)
self.assertEqual(dct['project_domain_name'], ctx.project_domain_name)
self.assertEqual(dct['system_scope'], ctx.system_scope)
def test_from_dict_unknown_keys(self):
dct = {

View File

@ -0,0 +1,4 @@
---
fixes:
- |
Fixes context from_dict() to properly handle system_scope.