]> review.fuel-infra Code Review - openstack-build/heat-build.git/commitdiff
heat tests : update_allowed_keys should be tuple
authorSteven Hardy <shardy@redhat.com>
Mon, 4 Feb 2013 13:35:05 +0000 (13:35 +0000)
committerSteven Hardy <shardy@redhat.com>
Mon, 4 Feb 2013 18:05:57 +0000 (18:05 +0000)
In test_resource.py update_allowed_keys should be a tuple not
a list, so it matches the expected usage in the resource classes

Change-Id: Ic7428e3790ea2c8c9887b464daf303c4f5d563fc
Signed-off-by: Steven Hardy <shardy@redhat.com>
heat/tests/test_resource.py

index 71531b8ec5b7bb6ceb6edb4bb4806ab295f10bc3..eac55c2d22e5eb5e196f40b96ad39640c10cd47e 100644 (file)
@@ -133,7 +133,7 @@ class ResourceTest(unittest.TestCase):
         tmpl = {'Type': 'Foo', 'Metadata': {'foo': 123}}
         update_snippet = {'Type': 'Foo', 'Metadata': {'foo': 456}}
         res = resource.GenericResource('test_resource', tmpl, self.stack)
-        res.update_allowed_keys = ['Metadata']
+        res.update_allowed_keys = ('Metadata',)
         diff = res.update_template_diff(json_snippet=update_snippet)
         self.assertEqual(diff, {'Metadata': {'foo': 456}})
 
@@ -141,7 +141,7 @@ class ResourceTest(unittest.TestCase):
         tmpl = {'Type': 'Foo'}
         update_snippet = {'Type': 'Foo', 'Metadata': {'foo': 123}}
         res = resource.GenericResource('test_resource', tmpl, self.stack)
-        res.update_allowed_keys = ['Metadata']
+        res.update_allowed_keys = ('Metadata',)
         diff = res.update_template_diff(json_snippet=update_snippet)
         self.assertEqual(diff, {'Metadata': {'foo': 123}})
 
@@ -149,7 +149,7 @@ class ResourceTest(unittest.TestCase):
         tmpl = {'Type': 'Foo', 'Metadata': {'foo': 123}}
         update_snippet = {'Type': 'Foo'}
         res = resource.GenericResource('test_resource', tmpl, self.stack)
-        res.update_allowed_keys = ['Metadata']
+        res.update_allowed_keys = ('Metadata',)
         diff = res.update_template_diff(json_snippet=update_snippet)
         self.assertEqual(diff, {'Metadata': None})