Fix up client init

Commit c9114b7eb8 introduced a critical
bug where service_type is not being passed through to the client making
it fail.

Change-Id: I6c19199785109cacef6d8b06e08d926efc6d4890
Closes-Bug: #1942330
This commit is contained in:
Sam Morrison 2021-09-02 11:55:18 +10:00 committed by Pierre Riteau
parent d2eb2f6583
commit cfabd241df
2 changed files with 7 additions and 3 deletions

View File

@ -34,4 +34,6 @@ def Client(version=1, service_type='reservation', *args, **kwargs):
})
raise exception.UnsupportedVersion(msg)
return importutils.import_object(client_path, *args, **kwargs)
return importutils.import_object(client_path,
service_type=service_type,
*args, **kwargs)

View File

@ -32,12 +32,14 @@ class BaseClientTestCase(tests.TestCase):
def test_with_v1(self):
self.client.Client()
self.import_obj.assert_called_once_with(
'blazarclient.v1.client.Client')
'blazarclient.v1.client.Client',
service_type='reservation')
def test_with_v1a0(self):
self.client.Client(version='1a0')
self.import_obj.assert_called_once_with(
'blazarclient.v1.client.Client')
'blazarclient.v1.client.Client',
service_type='reservation')
def test_with_wrong_vers(self):
self.assertRaises(exception.UnsupportedVersion,