Fixes https://bugs.launchpad.net/nova/+bug/663551 by catching exceptions at the top level of the API and turning them into Faults.

1 files changed, 65 insertions, 0 deletions

diff --git a/nova/tests/api/openstack/test_api.py b/nova/tests/api/openstack/test_api.py
new file mode 100644
index 000000000..171818806
--- /dev/null
+++ b/nova/tests/api/openstack/test_api.py
@@ -0,0 +1,65 @@
+# vim: tabstop=4 shiftwidth=4 softtabstop=4
+
+# Copyright 2010 OpenStack LLC.
+# All Rights Reserved.
+#
+#    Licensed under the Apache License, Version 2.0 (the "License"); you may
+#    not use this file except in compliance with the License. You may obtain
+#    a copy of the License at
+#
+#         http://www.apache.org/licenses/LICENSE-2.0
+#
+#    Unless required by applicable law or agreed to in writing, software
+#    distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+#    License for the specific language governing permissions and limitations
+#    under the License.
+
+import unittest
+import webob.exc
+import webob.dec
+
+import nova.api.openstack
+from nova.api.openstack import API
+from nova.api.openstack import faults
+from webob import Request
+
+class APITest(unittest.TestCase):
+
+    def test_exceptions_are_converted_to_faults(self):
+        @webob.dec.wsgify
+        def succeed(req):
+            return 'Succeeded'
+        @webob.dec.wsgify
+        def raise_webob_exc(req):
+            raise webob.exc.HTTPNotFound(explanation='Raised a webob.exc')
+        @webob.dec.wsgify
+        def fail(req):
+            raise Exception("Threw an exception")
+        @webob.dec.wsgify
+        def raise_api_fault(req):
+            exc = webob.exc.HTTPNotFound(explanation='Raised a webob.exc')
+            return faults.Fault(exc)
+        api = API()
+
+        api.application = succeed
+        resp = Request.blank('/').get_response(api)
+        self.assertFalse('cloudServersFault' in resp.body, resp.body)
+        self.assertEqual(resp.status_int, 200, resp.body)
+
+        api.application = raise_webob_exc
+        resp = Request.blank('/').get_response(api)
+        self.assertFalse('cloudServersFault' in resp.body, resp.body)
+        self.assertEqual(resp.status_int, 404, resp.body)
+
+        api.application = raise_api_fault
+        resp = Request.blank('/').get_response(api)
+        self.assertTrue('itemNotFound' in resp.body, resp.body)
+        self.assertEqual(resp.status_int, 404, resp.body)
+
+        api.application = fail
+        resp = Request.blank('/').get_response(api)
+        self.assertTrue('cloudServersFault' in resp.body, resp.body)
+        self.assertEqual(resp.status_int, 500, resp.body)
+
+