summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKen Pepple <ken.pepple@gmail.com>2011-08-20 12:43:50 -0700
committerKen Pepple <ken.pepple@gmail.com>2011-08-20 12:43:50 -0700
commit65b30ad73338fa481d1ab9155153b8265fbe8f90 (patch)
tree73b880e5372fd88ac475fab8b9300b0c0a015fd3
parent37508da788c5b2c2eadb36ef61d58836d93a3365 (diff)
pep8
-rw-r--r--nova/tests/test_instance_types.py20
1 files changed, 10 insertions, 10 deletions
diff --git a/nova/tests/test_instance_types.py b/nova/tests/test_instance_types.py
index 989c6f32f..556ba91a9 100644
--- a/nova/tests/test_instance_types.py
+++ b/nova/tests/test_instance_types.py
@@ -51,20 +51,20 @@ class InstanceTypeTestCase(test.TestCase):
"""return an instance type name not in the DB"""
nonexistant_flavor = "sdfsfsdf"
flavors = instance_types.get_all_types()
- while flavors.has_key(nonexistant_flavor):
- nonexistant_flavor = nonexistant_flavor.join("z")
+ while nonexistant_flavor in flavors:
+ nonexistant_flavor = nonexistant_flavor.join("z")
else:
- return nonexistant_flavor
+ return nonexistant_flavor
def _nonexistant_flavor_id(self):
"""return an instance type ID not in the DB"""
nonexistant_flavor = 2700
- flavor_ids = [ value["id"] for key, value in\
- instance_types.get_all_types().iteritems() ]
+ flavor_ids = [value["id"] for key, value in\
+ instance_types.get_all_types().iteritems()]
while nonexistant_flavor in flavor_ids:
- nonexistant_flavor += 1
+ nonexistant_flavor += 1
else:
- return nonexistant_flavor
+ return nonexistant_flavor
def _existing_flavor(self):
"""return first instance type name"""
@@ -127,12 +127,12 @@ class InstanceTypeTestCase(test.TestCase):
self.assertRaises(exception.ApiError,
instance_types.destroy,
self._nonexistant_flavor_name())
-
+
def test_will_not_purge_without_name(self):
"""Ensure purge without a name raises error"""
self.assertRaises(exception.InvalidInstanceType,
instance_types.purge, None)
-
+
def test_will_not_purge_with_wrong_name(self):
"""Ensure purge without correct name raises error"""
self.assertRaises(exception.ApiError,
@@ -149,7 +149,7 @@ class InstanceTypeTestCase(test.TestCase):
"""Ensure get by name returns default flavor with no name"""
self.assertEqual(instance_types.get_default_instance_type(),
instance_types.get_instance_type_by_name(None))
-
+
def test_will_not_get_instance_type_with_bad_name(self):
"""Ensure get by name returns default flavor with bad name"""
self.assertRaises(exception.InstanceTypeNotFound,