Skip to content

Commit c39867f

Browse files
Jenkinsopenstack-gerrit
authored andcommitted
Merge "Fix issue token for v3"
2 parents f0c57e1 + 1b3c7ec commit c39867f

4 files changed

Lines changed: 13 additions & 4 deletions

File tree

‎functional/tests/test_identity.py‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,7 @@ class IdentityV3Tests(test.TestCase):
7676

7777
DOMAIN_FIELDS = ['description', 'enabled', 'id', 'name', 'links']
7878
GROUP_FIELDS = ['description', 'domain_id', 'id', 'name', 'links']
79+
TOKEN_FIELDS = ['expires', 'id', 'project_id', 'user_id']
7980

8081
def _create_dummy_group(self):
8182
name = uuid.uuid4().hex
@@ -139,3 +140,8 @@ def test_domain_show(self):
139140
raw_output = self.openstack('domain show ' + name)
140141
items = self.parse_show(raw_output)
141142
self.assert_show_fields(items, self.DOMAIN_FIELDS)
143+
144+
def test_token_issue(self):
145+
raw_output = self.openstack('token issue')
146+
items = self.parse_show(raw_output)
147+
self.assert_show_fields(items, self.TOKEN_FIELDS)

‎openstackclient/identity/v3/token.py‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -159,9 +159,9 @@ def get_parser(self, prog_name):
159159

160160
def take_action(self, parsed_args):
161161
self.log.debug('take_action(%s)', parsed_args)
162-
identity_client = self.app.client_manager.identity
162+
session = self.app.client_manager.identity.session
163163

164-
token = identity_client.service_catalog.get_token()
164+
token = session.auth.auth_ref.service_catalog.get_token()
165165
if 'tenant_id' in token:
166166
token['project_id'] = token.pop('tenant_id')
167167
return zip(*sorted(six.iteritems(token)))

‎openstackclient/tests/identity/v3/fakes.py‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,9 @@ def __init__(self, **kwargs):
292292
self.roles.resource_class = fakes.FakeResource(None, {})
293293
self.services = mock.Mock()
294294
self.services.resource_class = fakes.FakeResource(None, {})
295-
self.service_catalog = mock.Mock()
295+
self.session = mock.Mock()
296+
self.session.auth.auth_ref.service_catalog.resource_class = \
297+
fakes.FakeResource(None, {})
296298
self.users = mock.Mock()
297299
self.users.resource_class = fakes.FakeResource(None, {})
298300
self.role_assignments = mock.Mock()

‎openstackclient/tests/identity/v3/test_token.py‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,8 @@ def setUp(self):
2323
super(TestToken, self).setUp()
2424

2525
# Get a shortcut to the Service Catalog Mock
26-
self.sc_mock = self.app.client_manager.identity.service_catalog
26+
session = self.app.client_manager.identity.session
27+
self.sc_mock = session.auth.auth_ref.service_catalog
2728
self.sc_mock.reset_mock()
2829

2930

0 commit comments

Comments
 (0)