diff --git a/tempest/api/compute/test_extensions.py b/tempest/api/compute/test_extensions.py index 4cc432899f..6e57aff700 100644 --- a/tempest/api/compute/test_extensions.py +++ b/tempest/api/compute/test_extensions.py @@ -32,7 +32,7 @@ class ExtensionsTestJSON(base.BaseV2ComputeTest): # List of all extensions if len(CONF.compute_feature_enabled.api_extensions) == 0: raise self.skipException('There are not any extensions configured') - extensions = self.extensions_client.list_extensions() + extensions = self.extensions_client.list_extensions()['extensions'] ext = CONF.compute_feature_enabled.api_extensions[0] if ext == 'all': self.assertIn('Hosts', map(lambda x: x['name'], extensions)) @@ -49,4 +49,4 @@ class ExtensionsTestJSON(base.BaseV2ComputeTest): def test_get_extension(self): # get the specified extensions extension = self.extensions_client.show_extension('os-consoles') - self.assertEqual('os-consoles', extension['alias']) + self.assertEqual('os-consoles', extension['extension']['alias']) diff --git a/tempest/services/compute/json/extensions_client.py b/tempest/services/compute/json/extensions_client.py index da342a86d7..4741812ba3 100644 --- a/tempest/services/compute/json/extensions_client.py +++ b/tempest/services/compute/json/extensions_client.py @@ -26,9 +26,9 @@ class ExtensionsClient(service_client.ServiceClient): resp, body = self.get(url) body = json.loads(body) self.validate_response(schema.list_extensions, resp, body) - return service_client.ResponseBodyList(resp, body['extensions']) + return service_client.ResponseBody(resp, body) def show_extension(self, extension_alias): resp, body = self.get('extensions/%s' % extension_alias) body = json.loads(body) - return service_client.ResponseBody(resp, body['extension']) + return service_client.ResponseBody(resp, body) diff --git a/tempest/tests/services/compute/test_extensions_client.py b/tempest/tests/services/compute/test_extensions_client.py index aa46efa7ee..20a5b7bbb4 100644 --- a/tempest/tests/services/compute/test_extensions_client.py +++ b/tempest/tests/services/compute/test_extensions_client.py @@ -34,7 +34,7 @@ class TestExtensionsClient(base.TestCase): body = '{"extensions": []}' if bytes_body: body = body.encode('utf-8') - expected = [] + expected = {"extensions": []} response = (httplib2.Response({'status': 200}), body) self.useFixture(mockpatch.Patch( 'tempest.common.service_client.ServiceClient.get', @@ -48,7 +48,7 @@ class TestExtensionsClient(base.TestCase): self._test_list_extensions(bytes_body=True) def _test_show_extension(self, bytes_body=False): - expected = { + expected = {"extension": { "updated": "2011-06-09T00:00:00Z", "name": "Multinic", "links": [], @@ -56,8 +56,8 @@ class TestExtensionsClient(base.TestCase): "http://docs.openstack.org/compute/ext/multinic/api/v1.1", "alias": "NMN", "description": u'\u2740(*\xb4\u25e1`*)\u2740' - } - serialized_body = json.dumps({"extension": expected}) + }} + serialized_body = json.dumps(expected) if bytes_body: serialized_body = serialized_body.encode('utf-8')