]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
normalize the json output of show a given extension.
authorYong Sheng Gong <gongysh@cn.ibm.com>
Sat, 11 Aug 2012 23:50:22 +0000 (07:50 +0800)
committerYong Sheng Gong <gongysh@cn.ibm.com>
Sat, 11 Aug 2012 23:51:44 +0000 (07:51 +0800)
we should use {'extension':{}} for a given extension query to comply with
our API design.

Change-Id: Id0b04b29bed4be92498c04178e268c31e9a1d635

quantum/extensions/extensions.py
quantum/tests/unit/test_extensions.py

index af7f91d41346cce90042ba138c3df7a96d16cd8d..f7dd55cb2b92db6160c0e5584b42661d1fc914c5 100644 (file)
@@ -221,7 +221,7 @@ class ExtensionController(wsgi.Controller):
         if not ext:
             raise webob.exc.HTTPNotFound(
                 _("Extension with alias %s does not exist") % id)
-        return self._translate(ext)
+        return dict(extension=self._translate(ext))
 
     def delete(self, request, id):
         raise webob.exc.HTTPNotFound()
index 777c745f4d2bb4b1472bf8196421140952686e2a..d86c4784f8158da0166f18535e5ac3edf8e8e271 100644 (file)
@@ -454,7 +454,7 @@ class ExtensionControllerTest(unittest.TestCase):
 
     def test_extension_can_be_accessed_by_alias(self):
         foxnsox_extension = self.test_app.get("/extensions/FOXNSOX").json
-
+        foxnsox_extension = foxnsox_extension['extension']
         self.assertEqual(foxnsox_extension["alias"], "FOXNSOX")
         self.assertEqual(foxnsox_extension["namespace"],
                          "http://www.fox.in.socks/api/ext/pie/v1.0")