diff options
author | Hans Lindgren <hanlind@kth.se> | 2015-11-13 13:18:00 +0100 |
---|---|---|
committer | Hans Lindgren <hanlind@kth.se> | 2015-11-13 14:17:06 +0100 |
commit | 5df0cc45ee74620141d66e6204470a510128c461 (patch) | |
tree | 82a2e16a54530f63e34b9b84c7b31366bb3732e2 /nova/tests/unit/test_flavors.py | |
parent | ccc87a0d5bcee7f0aa8b3354809001d5524783ed (diff) | |
download | nova-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.py | 19 |
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) |