From 86a8a5a2091dc0a36ccbe05f46609c82b1060291 Mon Sep 17 00:00:00 2001
From: Michael Gundlach <michael.gundlach@rackspace.com>
Date: Tue, 19 Oct 2010 19:37:42 -0400
Subject: Fixes https://bugs.launchpad.net/nova/+bug/663551 by catching
 exceptions at the top level of the API and turning them into Faults.

---
 nova/tests/api/openstack/test_api.py | 65 ++++++++++++++++++++++++++++++++++++
 1 file changed, 65 insertions(+)
 create mode 100644 nova/tests/api/openstack/test_api.py

(limited to 'nova/tests')

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)
+
+
-- 
cgit