Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 1 | # Copyright (c) 2016 Hewlett-Packard Enterprise Development Company, L.P. |
| 2 | # |
| 3 | # Licensed under the Apache License, Version 2.0 (the "License"); you may not |
| 4 | # use this file except in compliance with the License. You may obtain a copy of |
| 5 | # the License at |
| 6 | # |
| 7 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 8 | # |
| 9 | # Unless required by applicable law or agreed to in writing, software |
| 10 | # distributed under the License is distributed on an "AS IS" BASIS, WITHOUT |
| 11 | # WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the |
| 12 | # License for the specific language governing permissions and limitations under |
| 13 | # the License. |
| 14 | |
Andrea Frittoli (andreaf) | de5fb0c | 2016-06-13 12:15:00 +0100 | [diff] [blame] | 15 | import fixtures |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 16 | import mock |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 17 | import testtools |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 18 | import types |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 19 | |
| 20 | from tempest.lib import auth |
| 21 | from tempest.lib import exceptions |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 22 | from tempest.lib.services import clients |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 23 | from tempest.tests import base |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 24 | from tempest.tests.lib import fake_auth_provider |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 25 | from tempest.tests.lib import fake_credentials |
| 26 | |
| 27 | |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 28 | has_attribute = testtools.matchers.MatchesPredicateWithParams( |
| 29 | lambda x, y: hasattr(x, y), '{0} does not have an attribute {1}') |
| 30 | |
| 31 | |
| 32 | class TestClientsFactory(base.TestCase): |
| 33 | |
| 34 | def setUp(self): |
| 35 | super(TestClientsFactory, self).setUp() |
| 36 | self.classes = [] |
| 37 | |
| 38 | def _setup_fake_module(self, class_names=None, extra_dict=None): |
| 39 | class_names = class_names or [] |
| 40 | fake_module = types.ModuleType('fake_service_client') |
| 41 | _dict = {} |
| 42 | # Add fake classes to the fake module |
| 43 | for name in class_names: |
| 44 | _dict[name] = type(name, (object,), {}) |
| 45 | # Store it for assertions |
| 46 | self.classes.append(_dict[name]) |
| 47 | if extra_dict: |
| 48 | _dict[extra_dict] = extra_dict |
| 49 | fake_module.__dict__.update(_dict) |
| 50 | fixture_importlib = self.useFixture(fixtures.MockPatch( |
| 51 | 'importlib.import_module', return_value=fake_module)) |
| 52 | return fixture_importlib.mock |
| 53 | |
| 54 | def test___init___one_class(self): |
| 55 | fake_partial = 'fake_partial' |
| 56 | partial_mock = self.useFixture(fixtures.MockPatch( |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 57 | 'tempest.lib.services.clients.ClientsFactory._get_partial_class', |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 58 | return_value=fake_partial)).mock |
| 59 | class_names = ['FakeServiceClient1'] |
| 60 | mock_importlib = self._setup_fake_module(class_names=class_names) |
| 61 | auth_provider = fake_auth_provider.FakeAuthProvider() |
| 62 | params = {'k1': 'v1', 'k2': 'v2'} |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 63 | factory = clients.ClientsFactory('fake_path', class_names, |
| 64 | auth_provider, **params) |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 65 | # Assert module has been imported |
| 66 | mock_importlib.assert_called_once_with('fake_path') |
| 67 | # All attributes have been created |
| 68 | for client in class_names: |
| 69 | self.assertThat(factory, has_attribute(client)) |
| 70 | # Partial have been invoked correctly |
| 71 | partial_mock.assert_called_once_with( |
| 72 | self.classes[0], auth_provider, params) |
| 73 | # Get the clients |
| 74 | for name in class_names: |
| 75 | self.assertEqual(fake_partial, getattr(factory, name)) |
| 76 | |
| 77 | def test___init___two_classes(self): |
| 78 | fake_partial = 'fake_partial' |
| 79 | partial_mock = self.useFixture(fixtures.MockPatch( |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 80 | 'tempest.lib.services.clients.ClientsFactory._get_partial_class', |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 81 | return_value=fake_partial)).mock |
| 82 | class_names = ['FakeServiceClient1', 'FakeServiceClient2'] |
| 83 | mock_importlib = self._setup_fake_module(class_names=class_names) |
| 84 | auth_provider = fake_auth_provider.FakeAuthProvider() |
| 85 | params = {'k1': 'v1', 'k2': 'v2'} |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 86 | factory = clients.ClientsFactory('fake_path', class_names, |
| 87 | auth_provider, **params) |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 88 | # Assert module has been imported |
| 89 | mock_importlib.assert_called_once_with('fake_path') |
| 90 | # All attributes have been created |
| 91 | for client in class_names: |
| 92 | self.assertThat(factory, has_attribute(client)) |
| 93 | # Partial have been invoked the right number of times |
| 94 | partial_mock.call_count = len(class_names) |
| 95 | # Get the clients |
| 96 | for name in class_names: |
| 97 | self.assertEqual(fake_partial, getattr(factory, name)) |
| 98 | |
| 99 | def test___init___no_module(self): |
| 100 | auth_provider = fake_auth_provider.FakeAuthProvider() |
| 101 | class_names = ['FakeServiceClient1', 'FakeServiceClient2'] |
| 102 | with testtools.ExpectedException(ImportError, '.*fake_module.*'): |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 103 | clients.ClientsFactory('fake_module', class_names, |
| 104 | auth_provider) |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 105 | |
| 106 | def test___init___not_a_class(self): |
| 107 | class_names = ['FakeServiceClient1', 'FakeServiceClient2'] |
| 108 | extended_class_names = class_names + ['not_really_a_class'] |
| 109 | self._setup_fake_module( |
| 110 | class_names=class_names, extra_dict='not_really_a_class') |
| 111 | auth_provider = fake_auth_provider.FakeAuthProvider() |
| 112 | expected_msg = '.*not_really_a_class.*str.*' |
| 113 | with testtools.ExpectedException(TypeError, expected_msg): |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 114 | clients.ClientsFactory('fake_module', extended_class_names, |
| 115 | auth_provider) |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 116 | |
| 117 | def test___init___class_not_found(self): |
| 118 | class_names = ['FakeServiceClient1', 'FakeServiceClient2'] |
| 119 | extended_class_names = class_names + ['not_really_a_class'] |
| 120 | self._setup_fake_module(class_names=class_names) |
| 121 | auth_provider = fake_auth_provider.FakeAuthProvider() |
| 122 | expected_msg = '.*not_really_a_class.*fake_service_client.*' |
| 123 | with testtools.ExpectedException(AttributeError, expected_msg): |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 124 | clients.ClientsFactory('fake_module', extended_class_names, |
| 125 | auth_provider) |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 126 | |
| 127 | def test__get_partial_class_no_later_kwargs(self): |
| 128 | expected_fake_client = 'not_really_a_client' |
| 129 | self._setup_fake_module(class_names=[]) |
| 130 | auth_provider = fake_auth_provider.FakeAuthProvider() |
| 131 | params = {'k1': 'v1', 'k2': 'v2'} |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 132 | factory = clients.ClientsFactory( |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 133 | 'fake_path', [], auth_provider, **params) |
| 134 | klass_mock = mock.Mock(return_value=expected_fake_client) |
| 135 | partial = factory._get_partial_class(klass_mock, auth_provider, params) |
| 136 | # Class has not be initialised yet |
| 137 | klass_mock.assert_not_called() |
| 138 | # Use partial and assert on parameters |
| 139 | client = partial() |
| 140 | self.assertEqual(expected_fake_client, client) |
| 141 | klass_mock.assert_called_once_with(auth_provider=auth_provider, |
| 142 | **params) |
| 143 | |
| 144 | def test__get_partial_class_later_kwargs(self): |
| 145 | expected_fake_client = 'not_really_a_client' |
| 146 | self._setup_fake_module(class_names=[]) |
| 147 | auth_provider = fake_auth_provider.FakeAuthProvider() |
| 148 | params = {'k1': 'v1', 'k2': 'v2'} |
| 149 | later_params = {'k2': 'v4', 'k3': 'v3'} |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 150 | factory = clients.ClientsFactory( |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 151 | 'fake_path', [], auth_provider, **params) |
| 152 | klass_mock = mock.Mock(return_value=expected_fake_client) |
| 153 | partial = factory._get_partial_class(klass_mock, auth_provider, params) |
| 154 | # Class has not be initialised yet |
| 155 | klass_mock.assert_not_called() |
| 156 | # Use partial and assert on parameters |
| 157 | client = partial(**later_params) |
| 158 | params.update(later_params) |
| 159 | self.assertEqual(expected_fake_client, client) |
| 160 | klass_mock.assert_called_once_with(auth_provider=auth_provider, |
| 161 | **params) |
| 162 | |
Andrea Frittoli (andreaf) | f9d9a9d | 2016-06-30 17:35:38 +0100 | [diff] [blame] | 163 | def test__get_partial_class_with_alias(self): |
| 164 | expected_fake_client = 'not_really_a_client' |
| 165 | client_alias = 'fake_client' |
| 166 | self._setup_fake_module(class_names=[]) |
| 167 | auth_provider = fake_auth_provider.FakeAuthProvider() |
| 168 | params = {'k1': 'v1', 'k2': 'v2'} |
| 169 | later_params = {'k2': 'v4', 'k3': 'v3'} |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 170 | factory = clients.ClientsFactory( |
Andrea Frittoli (andreaf) | f9d9a9d | 2016-06-30 17:35:38 +0100 | [diff] [blame] | 171 | 'fake_path', [], auth_provider, **params) |
| 172 | klass_mock = mock.Mock(return_value=expected_fake_client) |
| 173 | partial = factory._get_partial_class(klass_mock, auth_provider, params) |
| 174 | # Class has not be initialised yet |
| 175 | klass_mock.assert_not_called() |
| 176 | # Use partial and assert on parameters |
| 177 | client = partial(alias=client_alias, **later_params) |
| 178 | params.update(later_params) |
| 179 | self.assertEqual(expected_fake_client, client) |
| 180 | klass_mock.assert_called_once_with(auth_provider=auth_provider, |
| 181 | **params) |
| 182 | self.assertThat(factory, has_attribute(client_alias)) |
| 183 | self.assertEqual(expected_fake_client, getattr(factory, client_alias)) |
| 184 | |
Andrea Frittoli (andreaf) | 73dd51d | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 185 | |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 186 | class TestServiceClients(base.TestCase): |
| 187 | |
Andrea Frittoli (andreaf) | de5fb0c | 2016-06-13 12:15:00 +0100 | [diff] [blame] | 188 | def setUp(self): |
| 189 | super(TestServiceClients, self).setUp() |
| 190 | self.useFixture(fixtures.MockPatch( |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 191 | 'tempest.lib.services.clients.tempest_modules', return_value={})) |
Andrea Frittoli (andreaf) | 40c9d6b | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 192 | self.useFixture(fixtures.MockPatch( |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 193 | 'tempest.lib.services.clients._tempest_internal_modules', |
Andrea Frittoli (andreaf) | 40c9d6b | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 194 | return_value=set(['fake_service1']))) |
Andrea Frittoli (andreaf) | de5fb0c | 2016-06-13 12:15:00 +0100 | [diff] [blame] | 195 | |
| 196 | def test___init___creds_v2_uri(self): |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 197 | # Verify that no API request is made, since no mock |
| 198 | # is required to run the test successfully |
| 199 | creds = fake_credentials.FakeKeystoneV2Credentials() |
| 200 | uri = 'fake_uri' |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 201 | _manager = clients.ServiceClients(creds, identity_uri=uri) |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 202 | self.assertIsInstance(_manager.auth_provider, |
| 203 | auth.KeystoneV2AuthProvider) |
| 204 | |
Andrea Frittoli (andreaf) | de5fb0c | 2016-06-13 12:15:00 +0100 | [diff] [blame] | 205 | def test___init___creds_v3_uri(self): |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 206 | # Verify that no API request is made, since no mock |
| 207 | # is required to run the test successfully |
| 208 | creds = fake_credentials.FakeKeystoneV3Credentials() |
| 209 | uri = 'fake_uri' |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 210 | _manager = clients.ServiceClients(creds, identity_uri=uri) |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 211 | self.assertIsInstance(_manager.auth_provider, |
| 212 | auth.KeystoneV3AuthProvider) |
| 213 | |
Andrea Frittoli (andreaf) | de5fb0c | 2016-06-13 12:15:00 +0100 | [diff] [blame] | 214 | def test___init___base_creds_uri(self): |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 215 | creds = fake_credentials.FakeCredentials() |
| 216 | uri = 'fake_uri' |
| 217 | with testtools.ExpectedException(exceptions.InvalidCredentials): |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 218 | clients.ServiceClients(creds, identity_uri=uri) |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 219 | |
Andrea Frittoli (andreaf) | de5fb0c | 2016-06-13 12:15:00 +0100 | [diff] [blame] | 220 | def test___init___invalid_creds_uri(self): |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 221 | creds = fake_credentials.FakeKeystoneV2Credentials() |
| 222 | delattr(creds, 'username') |
| 223 | uri = 'fake_uri' |
| 224 | with testtools.ExpectedException(exceptions.InvalidCredentials): |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 225 | clients.ServiceClients(creds, identity_uri=uri) |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 226 | |
Andrea Frittoli (andreaf) | de5fb0c | 2016-06-13 12:15:00 +0100 | [diff] [blame] | 227 | def test___init___creds_uri_none(self): |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 228 | creds = fake_credentials.FakeKeystoneV2Credentials() |
Andrea Frittoli (andreaf) | de5fb0c | 2016-06-13 12:15:00 +0100 | [diff] [blame] | 229 | msg = ("Invalid Credentials\nDetails: ServiceClients requires a " |
| 230 | "non-empty") |
Andrea Frittoli (andreaf) | 2395014 | 2016-06-13 12:39:29 +0100 | [diff] [blame] | 231 | with testtools.ExpectedException(exceptions.InvalidCredentials, |
| 232 | value_re=msg): |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 233 | clients.ServiceClients(creds, None) |
Andrea Frittoli (andreaf) | de5fb0c | 2016-06-13 12:15:00 +0100 | [diff] [blame] | 234 | |
| 235 | def test___init___creds_uri_params(self): |
| 236 | creds = fake_credentials.FakeKeystoneV2Credentials() |
| 237 | expeted_params = {'fake_param1': 'fake_value1', |
| 238 | 'fake_param2': 'fake_value2'} |
| 239 | params = {'fake_service1': expeted_params} |
| 240 | uri = 'fake_uri' |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 241 | _manager = clients.ServiceClients(creds, identity_uri=uri, |
| 242 | client_parameters=params) |
Joe H. Rahme | a72f2c6 | 2016-07-11 16:28:19 +0200 | [diff] [blame] | 243 | self.assertIn('fake_service1', _manager.parameters) |
| 244 | for _key in expeted_params: |
Andrea Frittoli (andreaf) | de5fb0c | 2016-06-13 12:15:00 +0100 | [diff] [blame] | 245 | self.assertIn(_key, _manager.parameters['fake_service1'].keys()) |
| 246 | self.assertEqual(expeted_params[_key], |
| 247 | _manager.parameters['fake_service1'].get(_key)) |
| 248 | |
| 249 | def test___init___creds_uri_params_unknown_services(self): |
| 250 | creds = fake_credentials.FakeKeystoneV2Credentials() |
| 251 | fake_params = {'fake_param1': 'fake_value1'} |
| 252 | params = {'unknown_service1': fake_params, |
| 253 | 'unknown_service2': fake_params} |
| 254 | uri = 'fake_uri' |
| 255 | msg = "(?=.*{0})(?=.*{1})".format(*list(params.keys())) |
| 256 | with testtools.ExpectedException( |
| 257 | exceptions.UnknownServiceClient, value_re=msg): |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 258 | clients.ServiceClients(creds, identity_uri=uri, |
| 259 | client_parameters=params) |
Andrea Frittoli (andreaf) | de5fb0c | 2016-06-13 12:15:00 +0100 | [diff] [blame] | 260 | |
| 261 | def _get_manager(self, init_region='fake_region'): |
| 262 | # Get a manager to invoke _setup_parameters on |
| 263 | creds = fake_credentials.FakeKeystoneV2Credentials() |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 264 | return clients.ServiceClients(creds, identity_uri='fake_uri', |
| 265 | region=init_region) |
Andrea Frittoli (andreaf) | de5fb0c | 2016-06-13 12:15:00 +0100 | [diff] [blame] | 266 | |
| 267 | def test__setup_parameters_none_no_region(self): |
| 268 | kwargs = {} |
| 269 | _manager = self._get_manager(init_region=None) |
| 270 | _params = _manager._setup_parameters(kwargs) |
| 271 | self.assertNotIn('region', _params) |
| 272 | |
| 273 | def test__setup_parameters_none(self): |
| 274 | kwargs = {} |
| 275 | _manager = self._get_manager() |
| 276 | _params = _manager._setup_parameters(kwargs) |
| 277 | self.assertIn('region', _params) |
| 278 | self.assertEqual('fake_region', _params['region']) |
| 279 | |
| 280 | def test__setup_parameters_all(self): |
| 281 | expected_params = {'region': 'fake_region1', |
| 282 | 'catalog_type': 'fake_service2_mod', |
| 283 | 'fake_param1': 'fake_value1', |
| 284 | 'fake_param2': 'fake_value2'} |
| 285 | _manager = self._get_manager() |
| 286 | _params = _manager._setup_parameters(expected_params) |
| 287 | for _key in _params.keys(): |
| 288 | self.assertEqual(expected_params[_key], |
| 289 | _params[_key]) |
Andrea Frittoli (andreaf) | 6d4d85a | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 290 | |
| 291 | def test_register_service_client_module(self): |
Andrea Frittoli (andreaf) | 6d4d85a | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 292 | expected_params = {'fake_param1': 'fake_value1', |
| 293 | 'fake_param2': 'fake_value2'} |
| 294 | _manager = self._get_manager(init_region='fake_region_default') |
Andrea Frittoli (andreaf) | f2affcc | 2016-06-28 21:41:47 +0100 | [diff] [blame] | 295 | # Mock after the _manager is setup to preserve the call count |
| 296 | factory_mock = self.useFixture(fixtures.MockPatch( |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 297 | 'tempest.lib.services.clients.ClientsFactory')).mock |
Andrea Frittoli (andreaf) | 6d4d85a | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 298 | _manager.register_service_client_module( |
| 299 | name='fake_module', |
| 300 | service_version='fake_service', |
| 301 | module_path='fake.path.to.module', |
| 302 | client_names=[], |
| 303 | **expected_params) |
| 304 | self.assertThat(_manager, has_attribute('fake_module')) |
| 305 | # Assert called once, without check for exact parameters |
| 306 | self.assertTrue(factory_mock.called) |
| 307 | self.assertEqual(1, factory_mock.call_count) |
| 308 | # Assert expected params are in with their values |
| 309 | actual_kwargs = factory_mock.call_args[1] |
| 310 | self.assertIn('region', actual_kwargs) |
| 311 | self.assertEqual('fake_region_default', actual_kwargs['region']) |
| 312 | for param in expected_params: |
| 313 | self.assertIn(param, actual_kwargs) |
| 314 | self.assertEqual(expected_params[param], actual_kwargs[param]) |
| 315 | # Assert the new service is registered |
| 316 | self.assertIn('fake_service', _manager._registered_services) |
| 317 | |
| 318 | def test_register_service_client_module_override_default(self): |
Andrea Frittoli (andreaf) | 6d4d85a | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 319 | new_region = 'new_region' |
| 320 | expected_params = {'fake_param1': 'fake_value1', |
| 321 | 'fake_param2': 'fake_value2', |
| 322 | 'region': new_region} |
| 323 | _manager = self._get_manager(init_region='fake_region_default') |
Andrea Frittoli (andreaf) | f2affcc | 2016-06-28 21:41:47 +0100 | [diff] [blame] | 324 | # Mock after the _manager is setup to preserve the call count |
| 325 | factory_mock = self.useFixture(fixtures.MockPatch( |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 326 | 'tempest.lib.services.clients.ClientsFactory')).mock |
Andrea Frittoli (andreaf) | 6d4d85a | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 327 | _manager.register_service_client_module( |
| 328 | name='fake_module', |
| 329 | service_version='fake_service', |
| 330 | module_path='fake.path.to.module', |
| 331 | client_names=[], |
| 332 | **expected_params) |
| 333 | self.assertThat(_manager, has_attribute('fake_module')) |
| 334 | # Assert called once, without check for exact parameters |
| 335 | self.assertTrue(factory_mock.called) |
| 336 | self.assertEqual(1, factory_mock.call_count) |
| 337 | # Assert expected params are in with their values |
| 338 | actual_kwargs = factory_mock.call_args[1] |
| 339 | self.assertIn('region', actual_kwargs) |
| 340 | self.assertEqual(new_region, actual_kwargs['region']) |
| 341 | for param in expected_params: |
| 342 | self.assertIn(param, actual_kwargs) |
| 343 | self.assertEqual(expected_params[param], actual_kwargs[param]) |
| 344 | # Assert the new service is registered |
| 345 | self.assertIn('fake_service', _manager._registered_services) |
| 346 | |
| 347 | def test_register_service_client_module_duplicate_name(self): |
| 348 | self.useFixture(fixtures.MockPatch( |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 349 | 'tempest.lib.services.clients.ClientsFactory')).mock |
Andrea Frittoli (andreaf) | 6d4d85a | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 350 | _manager = self._get_manager() |
| 351 | name_owner = 'this_is_a_string' |
| 352 | setattr(_manager, 'fake_module', name_owner) |
| 353 | expected_error = '.*' + name_owner |
| 354 | with testtools.ExpectedException( |
| 355 | exceptions.ServiceClientRegistrationException, expected_error): |
| 356 | _manager.register_service_client_module( |
| 357 | name='fake_module', module_path='fake.path.to.module', |
| 358 | service_version='fake_service', client_names=[]) |
| 359 | |
| 360 | def test_register_service_client_module_duplicate_service(self): |
| 361 | self.useFixture(fixtures.MockPatch( |
Andrea Frittoli (andreaf) | e07579c | 2016-08-05 07:27:02 +0100 | [diff] [blame] | 362 | 'tempest.lib.services.clients.ClientsFactory')).mock |
Andrea Frittoli (andreaf) | 6d4d85a | 2016-06-21 17:20:31 +0100 | [diff] [blame] | 363 | _manager = self._get_manager() |
| 364 | duplicate_service = 'fake_service1' |
| 365 | expected_error = '.*' + duplicate_service |
| 366 | with testtools.ExpectedException( |
| 367 | exceptions.ServiceClientRegistrationException, expected_error): |
| 368 | _manager.register_service_client_module( |
| 369 | name='fake_module', module_path='fake.path.to.module', |
| 370 | service_version=duplicate_service, client_names=[]) |