heat engine : raise appropriate error for incorrect stack_name

Raise an AttributeError instead of returning a string when passed
an incorrect stack name, this then maps to the appropriate error
in the API.

Fixes #182

Change-Id: I7fe80496883bfbe515d9c235de812aaa14c41c12
Signed-off-by: Steven Hardy <shardy@redhat.com>

diff --git a/heat/engine/manager.py b/heat/engine/manager.py
index e86ee632e21e8bcf54575d6d546a3ca6253706c8..2e1c26ce9dc1e969d9aa0aa9b219c297543814ec 100644 (file)
--- a/heat/engine/manager.py
+++ b/heat/engine/manager.py
@@ -102,7 +102,7 @@ class EngineManager(manager.Manager):
         auth.authenticate(context)
 
         if db_api.stack_get_by_name(None, stack_name):
-            return {'Error': 'Stack already exists with that name.'}
+            raise AttributeError('Stack already exists with that name')
 
         tmpl = parser.Template(template)
 
@@ -141,7 +141,7 @@ class EngineManager(manager.Manager):
         # Get the database representation of the existing stack
         db_stack = db_api.stack_get_by_name(None, stack_name)
         if not db_stack:
-            return {'Error': 'No stack exists with that name.'}
+            raise AttributeError('No stack exists with that name')
 
         current_stack = parser.Stack.load(context, db_stack.id)