Adds test scripts for _validate_values

Shortened _validate_values logic
Wrote unittests for _validate_values

Change-Id: I0f947d20b73aa96524121257784f9ba42489e477

diff --git a/quantum/api/v2/attributes.py b/quantum/api/v2/attributes.py
index 9076657630aa636fa83abd2e92ac749c4162634d..77e8e4692ea2871ab665cba2ac44066c2bcc5773 100644 (file)
--- a/quantum/api/v2/attributes.py
+++ b/quantum/api/v2/attributes.py
@@ -42,11 +42,8 @@ def _validate_boolean(data, valid_values=None):
 
 
 def _validate_values(data, valid_values=None):
-    if data in valid_values:
-        return
-    else:
-        msg_dict = dict(data=data, values=valid_values)
-        msg = _("%(data)s is not in %(values)s") % msg_dict
+    if data not in valid_values:
+        msg = _("'%(data)s' is not in %(valid_values)s") % locals()
         LOG.debug("validate_values: %s", msg)
         return msg
 

diff --git a/quantum/tests/unit/test_attributes.py b/quantum/tests/unit/test_attributes.py
index b53a420274aeaebb751752b2719c700651c3f6c7..d73aeaa2e39d748148dd57b8095916bcbba5688e 100644 (file)
--- a/quantum/tests/unit/test_attributes.py
+++ b/quantum/tests/unit/test_attributes.py
@@ -63,6 +63,19 @@ class TestAttributes(unittest2.TestCase):
         msg = attributes._validate_boolean(None)
         self.assertEquals(msg, "'None' is not boolean")
 
+    def test_values(self):
+        msg = attributes._validate_values(4, [4, 6])
+        self.assertIsNone(msg)
+
+        msg = attributes._validate_values(4, (4, 6))
+        self.assertIsNone(msg)
+
+        msg = attributes._validate_values(7, [4, 6])
+        self.assertEquals(msg, "'7' is not in [4, 6]")
+
+        msg = attributes._validate_values(7, (4, 6))
+        self.assertEquals(msg, "'7' is not in (4, 6)")
+
     def test_strings(self):
         msg = attributes._validate_string(None, None)
         self.assertEquals(msg, "'None' is not a valid string")