Created new class ProjectSettingsError. 29/37929/1
authorspisarski <s.pisarski@cablelabs.com>
Fri, 21 Jul 2017 14:08:30 +0000 (08:08 -0600)
committerspisarski <s.pisarski@cablelabs.com>
Fri, 21 Jul 2017 14:08:30 +0000 (08:08 -0600)
Raising ProjectSettingsError in ProjectSettings instead of Exception.

JIRA: SNAPS-135

Change-Id: I5169dd196775453466ea8d7fe9fa0c8d82b2382a
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
snaps/openstack/create_project.py
snaps/openstack/tests/create_project_tests.py

index 0384ccc..a20033e 100644 (file)
@@ -131,5 +131,11 @@ class ProjectSettings:
             self.enabled = True
 
         if not self.name:
-            raise Exception(
+            raise ProjectSettingsError(
                 "The attribute name is required for ProjectSettings")
+
+
+class ProjectSettingsError(Exception):
+    """
+    Exception to be thrown when project settings attributes are incorrect
+    """
index 3c6b2d1..f2af0d9 100644 (file)
@@ -15,7 +15,8 @@
 import unittest
 import uuid
 
-from snaps.openstack.create_project import OpenStackProject, ProjectSettings
+from snaps.openstack.create_project import (
+    OpenStackProject, ProjectSettings, ProjectSettingsError)
 from snaps.openstack.create_security_group import OpenStackSecurityGroup
 from snaps.openstack.create_security_group import SecurityGroupSettings
 from snaps.openstack.create_user import OpenStackUser
@@ -32,11 +33,11 @@ class ProjectSettingsUnitTests(unittest.TestCase):
     """
 
     def test_no_params(self):
-        with self.assertRaises(Exception):
+        with self.assertRaises(ProjectSettingsError):
             ProjectSettings()
 
     def test_empty_config(self):
-        with self.assertRaises(Exception):
+        with self.assertRaises(ProjectSettingsError):
             ProjectSettings(**dict())
 
     def test_name_only(self):