summaryrefslogtreecommitdiff
path: root/nova/tests/unit/test_flavors.py
diff options
context:
space:
mode:
authorHans Lindgren <hanlind@kth.se>2015-11-13 13:18:00 +0100
committerHans Lindgren <hanlind@kth.se>2015-11-13 14:17:06 +0100
commit5df0cc45ee74620141d66e6204470a510128c461 (patch)
tree82a2e16a54530f63e34b9b84c7b31366bb3732e2 /nova/tests/unit/test_flavors.py
parentccc87a0d5bcee7f0aa8b3354809001d5524783ed (diff)
downloadnova-5df0cc45ee74620141d66e6204470a510128c461.tar.gz
Remove flavors.get_flavor() only used in tests
This method is only used in tests and all it really do is calling the object method Flavor.get_by_id(). Lets remove it and call with object method directly instead. Change-Id: I93d57b14cd60fe5c6a24573a74b691c01fb10563
Diffstat (limited to 'nova/tests/unit/test_flavors.py')
-rw-r--r--nova/tests/unit/test_flavors.py19
1 files changed, 1 insertions, 18 deletions
diff --git a/nova/tests/unit/test_flavors.py b/nova/tests/unit/test_flavors.py
index 553010158d..df4e98cd68 100644
--- a/nova/tests/unit/test_flavors.py
+++ b/nova/tests/unit/test_flavors.py
@@ -78,23 +78,6 @@ class InstanceTypeTestCase(test.TestCase):
self.assertRaises(exception.FlavorNotFound,
flavors.get_default_flavor)
- def test_will_get_flavor_by_id(self):
- default_instance_type = flavors.get_default_flavor()
- instance_type_id = default_instance_type.id
- fetched = flavors.get_flavor(instance_type_id)
- self.assertIsInstance(fetched, objects.Flavor)
- self.assertEqual(default_instance_type.flavorid, fetched.flavorid)
-
- def test_will_not_get_flavor_by_unknown_id(self):
- # Ensure get by name returns default flavor with no name.
- self.assertRaises(exception.FlavorNotFound,
- flavors.get_flavor, 10000)
-
- def test_will_not_get_flavor_with_bad_id(self):
- # Ensure get by name returns default flavor with bad name.
- self.assertRaises(exception.FlavorNotFound,
- flavors.get_flavor, 'asdf')
-
def test_flavor_get_by_None_name_returns_default(self):
# Ensure get by name returns default flavor with no name.
default = flavors.get_default_flavor()
@@ -481,7 +464,7 @@ class CreateInstanceTypeTest(test.TestCase):
flavors.destroy('flavor')
self.assertRaises(exception.FlavorNotFound,
- flavors.get_flavor, flavor.id)
+ objects.Flavor.get_by_id, ctxt, flavor.id)
# Deleted instance should not be in list anymore
new_list = objects.FlavorList.get_all(ctxt)