db9b47a0c7
Python 3 does not support indexing keys or items iterators. Removes unit tests from tests-py3.txt blacklist. Partially implements blueprint: nova-python3-mitaka Change-Id: I3ec1e775d91e7f56356f744fd80f5d9df36cae1d
489 lines
20 KiB
Python
489 lines
20 KiB
Python
# Copyright 2011 Ken Pepple
|
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License. You may obtain
|
|
# a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
# License for the specific language governing permissions and limitations
|
|
# under the License.
|
|
"""
|
|
Unit Tests for flavors code
|
|
"""
|
|
from nova.compute import flavors
|
|
from nova import context
|
|
from nova import db
|
|
from nova import exception
|
|
from nova import objects
|
|
from nova.objects import base as obj_base
|
|
from nova import test
|
|
|
|
|
|
DEFAULT_FLAVORS = [
|
|
{'memory_mb': 512, 'root_gb': 1, 'deleted_at': None, 'name': 'm1.tiny',
|
|
'deleted': 0, 'created_at': None, 'ephemeral_gb': 0, 'updated_at': None,
|
|
'disabled': False, 'vcpus': 1, 'extra_specs': {}, 'swap': 0,
|
|
'rxtx_factor': 1.0, 'is_public': True, 'flavorid': '1',
|
|
'vcpu_weight': None, 'id': 2},
|
|
{'memory_mb': 2048, 'root_gb': 20, 'deleted_at': None, 'name': 'm1.small',
|
|
'deleted': 0, 'created_at': None, 'ephemeral_gb': 0, 'updated_at': None,
|
|
'disabled': False, 'vcpus': 1, 'extra_specs': {}, 'swap': 0,
|
|
'rxtx_factor': 1.0, 'is_public': True, 'flavorid': '2',
|
|
'vcpu_weight': None, 'id': 5},
|
|
{'memory_mb': 4096, 'root_gb': 40, 'deleted_at': None, 'name': 'm1.medium',
|
|
'deleted': 0, 'created_at': None, 'ephemeral_gb': 0, 'updated_at': None,
|
|
'disabled': False, 'vcpus': 2, 'extra_specs': {}, 'swap': 0,
|
|
'rxtx_factor': 1.0, 'is_public': True, 'flavorid': '3',
|
|
'vcpu_weight': None, 'id': 1},
|
|
{'memory_mb': 8192, 'root_gb': 80, 'deleted_at': None, 'name': 'm1.large',
|
|
'deleted': 0, 'created_at': None, 'ephemeral_gb': 0, 'updated_at': None,
|
|
'disabled': False, 'vcpus': 4, 'extra_specs': {}, 'swap': 0,
|
|
'rxtx_factor': 1.0, 'is_public': True, 'flavorid': '4',
|
|
'vcpu_weight': None, 'id': 3},
|
|
{'memory_mb': 16384, 'root_gb': 160, 'deleted_at': None,
|
|
'name': 'm1.xlarge', 'deleted': 0, 'created_at': None, 'ephemeral_gb': 0,
|
|
'updated_at': None, 'disabled': False, 'vcpus': 8, 'extra_specs': {},
|
|
'swap': 0, 'rxtx_factor': 1.0, 'is_public': True, 'flavorid': '5',
|
|
'vcpu_weight': None, 'id': 4}
|
|
]
|
|
|
|
CONTEXT = context.RequestContext('fake', 'fake', is_admin=False)
|
|
|
|
DEFAULT_FLAVOR_OBJS = [
|
|
objects.Flavor._obj_from_primitive(CONTEXT, objects.Flavor.VERSION,
|
|
{'nova_object.data': flavor})
|
|
for flavor in DEFAULT_FLAVORS
|
|
]
|
|
|
|
|
|
class InstanceTypeTestCase(test.TestCase):
|
|
"""Test cases for flavor code."""
|
|
def test_non_existent_inst_type_should_not_delete(self):
|
|
# Ensures that flavor creation fails with invalid args.
|
|
self.assertRaises(exception.FlavorNotFoundByName,
|
|
flavors.destroy,
|
|
'unknown_flavor')
|
|
|
|
def test_will_not_destroy_with_no_name(self):
|
|
# Ensure destroy said path of no name raises error.
|
|
self.assertRaises(exception.FlavorNotFoundByName,
|
|
flavors.destroy, None)
|
|
|
|
def test_will_not_get_bad_default_instance_type(self):
|
|
# ensures error raised on bad default flavor.
|
|
self.flags(default_flavor='unknown_flavor')
|
|
self.assertRaises(exception.FlavorNotFound,
|
|
flavors.get_default_flavor)
|
|
|
|
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()
|
|
actual = flavors.get_flavor_by_name(None)
|
|
self.assertIsInstance(default, objects.Flavor)
|
|
self.assertIsInstance(actual, objects.Flavor)
|
|
self.assertEqual(default.flavorid, actual.flavorid)
|
|
|
|
def test_will_not_get_flavor_with_bad_name(self):
|
|
# Ensure get by name returns default flavor with bad name.
|
|
self.assertRaises(exception.FlavorNotFound,
|
|
flavors.get_flavor_by_name, 10000)
|
|
|
|
def test_will_not_get_instance_by_unknown_flavor_id(self):
|
|
# Ensure get by flavor raises error with wrong flavorid.
|
|
self.assertRaises(exception.FlavorNotFound,
|
|
flavors.get_flavor_by_flavor_id,
|
|
'unknown_flavor')
|
|
|
|
def test_will_get_instance_by_flavor_id(self):
|
|
default_instance_type = flavors.get_default_flavor()
|
|
flavorid = default_instance_type.flavorid
|
|
fetched = flavors.get_flavor_by_flavor_id(flavorid)
|
|
self.assertIsInstance(fetched, objects.Flavor)
|
|
self.assertEqual(default_instance_type.flavorid, fetched.flavorid)
|
|
|
|
def test_can_read_deleted_types_using_flavor_id(self):
|
|
# Ensure deleted flavors can be read when querying flavor_id.
|
|
inst_type_name = "test"
|
|
inst_type_flavor_id = "test1"
|
|
|
|
inst_type = flavors.create(inst_type_name, 256, 1, 120, 100,
|
|
inst_type_flavor_id)
|
|
self.assertEqual(inst_type_name, inst_type.name)
|
|
|
|
# NOTE(jk0): The deleted flavor will show up here because the context
|
|
# in get_flavor_by_flavor_id() is set to use read_deleted by
|
|
# default.
|
|
flavors.destroy(inst_type.name)
|
|
deleted_inst_type = flavors.get_flavor_by_flavor_id(
|
|
inst_type_flavor_id)
|
|
self.assertEqual(inst_type_name, deleted_inst_type.name)
|
|
|
|
def test_read_deleted_false_converting_flavorid(self):
|
|
"""Ensure deleted flavors are not returned when not needed (for
|
|
example when creating a server and attempting to translate from
|
|
flavorid to instance_type_id.
|
|
"""
|
|
flavors.create("instance_type1", 256, 1, 120, 100, "test1")
|
|
flavors.destroy("instance_type1")
|
|
flavors.create("instance_type1_redo", 256, 1, 120, 100, "test1")
|
|
|
|
instance_type = flavors.get_flavor_by_flavor_id(
|
|
"test1", read_deleted="no")
|
|
self.assertEqual("instance_type1_redo", instance_type.name)
|
|
|
|
def test_get_all_flavors_sorted_list_sort(self):
|
|
# Test default sort
|
|
all_flavors = flavors.get_all_flavors_sorted_list()
|
|
self.assertEqual(len(DEFAULT_FLAVORS), len(all_flavors))
|
|
for i in range(len(all_flavors)):
|
|
f = all_flavors[i]
|
|
self.assertIsInstance(f, objects.Flavor)
|
|
self.assertEqual(DEFAULT_FLAVORS[i]['flavorid'], f.flavorid)
|
|
|
|
# Test sorted by name
|
|
all_flavors = flavors.get_all_flavors_sorted_list(sort_key='name')
|
|
expected = sorted(DEFAULT_FLAVORS, key=lambda item: item['name'])
|
|
self.assertEqual(len(expected), len(all_flavors))
|
|
for i in range(len(all_flavors)):
|
|
f = all_flavors[i]
|
|
self.assertIsInstance(f, objects.Flavor)
|
|
self.assertEqual(expected[i]['flavorid'], f.flavorid)
|
|
|
|
def test_get_all_flavors_sorted_list_limit(self):
|
|
limited_flavors = flavors.get_all_flavors_sorted_list(limit=2)
|
|
self.assertEqual(2, len(limited_flavors))
|
|
|
|
def test_get_all_flavors_sorted_list_marker(self):
|
|
all_flavors = flavors.get_all_flavors_sorted_list()
|
|
|
|
# Set the 3rd result as the marker
|
|
marker_flavorid = all_flavors[2].flavorid
|
|
marked_flavors = flavors.get_all_flavors_sorted_list(
|
|
marker=marker_flavorid)
|
|
# We expect everything /after/ the 3rd result
|
|
expected_results = all_flavors[3:]
|
|
self.assertEqual(len(expected_results), len(marked_flavors))
|
|
for i in range(len(marked_flavors)):
|
|
f = marked_flavors[i]
|
|
self.assertIsInstance(f, objects.Flavor)
|
|
self.assertEqual(expected_results[i].flavorid,
|
|
f.flavorid)
|
|
|
|
|
|
class InstanceTypeToolsTest(test.TestCase):
|
|
def _dict_to_metadata(self, data):
|
|
return [{'key': key, 'value': value} for key, value in data.items()]
|
|
|
|
def _test_extract_flavor(self, prefix):
|
|
instance_type = flavors.get_default_flavor()
|
|
instance_type_p = obj_base.obj_to_primitive(instance_type)
|
|
|
|
metadata = {}
|
|
flavors.save_flavor_info(metadata, instance_type, prefix)
|
|
instance = {'system_metadata': self._dict_to_metadata(metadata)}
|
|
_instance_type = flavors.extract_flavor(instance, prefix)
|
|
_instance_type_p = obj_base.obj_to_primitive(_instance_type)
|
|
|
|
props = flavors.system_metadata_flavor_props.keys()
|
|
for key in list(instance_type_p.keys()):
|
|
if key not in props:
|
|
del instance_type_p[key]
|
|
|
|
self.assertEqual(instance_type_p, _instance_type_p)
|
|
|
|
def test_extract_flavor(self):
|
|
self._test_extract_flavor('')
|
|
|
|
def test_extract_flavor_no_sysmeta(self):
|
|
instance = {}
|
|
prefix = ''
|
|
result = flavors.extract_flavor(instance, prefix)
|
|
|
|
self.assertIsNone(result)
|
|
|
|
def test_extract_flavor_prefix(self):
|
|
self._test_extract_flavor('foo_')
|
|
|
|
def test_save_flavor_info(self):
|
|
instance_type = flavors.get_default_flavor()
|
|
|
|
example = {}
|
|
example_prefix = {}
|
|
|
|
for key in flavors.system_metadata_flavor_props.keys():
|
|
example['instance_type_%s' % key] = instance_type[key]
|
|
example_prefix['fooinstance_type_%s' % key] = instance_type[key]
|
|
|
|
metadata = {}
|
|
flavors.save_flavor_info(metadata, instance_type)
|
|
self.assertEqual(example, metadata)
|
|
|
|
metadata = {}
|
|
flavors.save_flavor_info(metadata, instance_type, 'foo')
|
|
self.assertEqual(example_prefix, metadata)
|
|
|
|
def test_delete_flavor_info(self):
|
|
instance_type = flavors.get_default_flavor()
|
|
metadata = {}
|
|
flavors.save_flavor_info(metadata, instance_type)
|
|
flavors.save_flavor_info(metadata, instance_type, '_')
|
|
flavors.delete_flavor_info(metadata, '', '_')
|
|
self.assertEqual(metadata, {})
|
|
|
|
def test_flavor_numa_extras_are_saved(self):
|
|
instance_type = flavors.get_default_flavor()
|
|
instance_type['extra_specs'] = {
|
|
'hw:numa_mem.0': '123',
|
|
'hw:numa_cpus.0': '456',
|
|
'hw:numa_mem.1': '789',
|
|
'hw:numa_cpus.1': 'ABC',
|
|
'foo': 'bar',
|
|
}
|
|
sysmeta = flavors.save_flavor_info({}, instance_type)
|
|
_instance_type = flavors.extract_flavor({'system_metadata': sysmeta})
|
|
expected_extra_specs = {
|
|
'hw:numa_mem.0': '123',
|
|
'hw:numa_cpus.0': '456',
|
|
'hw:numa_mem.1': '789',
|
|
'hw:numa_cpus.1': 'ABC',
|
|
}
|
|
self.assertEqual(expected_extra_specs, _instance_type['extra_specs'])
|
|
flavors.delete_flavor_info(sysmeta, '')
|
|
self.assertEqual({}, sysmeta)
|
|
|
|
|
|
class InstanceTypeFilteringTest(test.TestCase):
|
|
"""Test cases for the filter option available for instance_type_get_all."""
|
|
def setUp(self):
|
|
super(InstanceTypeFilteringTest, self).setUp()
|
|
self.context = context.get_admin_context()
|
|
|
|
def assertFilterResults(self, filters, expected):
|
|
inst_types = db.flavor_get_all(
|
|
self.context, filters=filters)
|
|
inst_names = [i['name'] for i in inst_types]
|
|
self.assertEqual(inst_names, expected)
|
|
|
|
def test_no_filters(self):
|
|
filters = None
|
|
expected = ['m1.tiny', 'm1.small', 'm1.medium', 'm1.large',
|
|
'm1.xlarge']
|
|
self.assertFilterResults(filters, expected)
|
|
|
|
def test_min_memory_mb_filter(self):
|
|
# Exclude tiny instance which is 512 MB.
|
|
filters = dict(min_memory_mb=513)
|
|
expected = ['m1.small', 'm1.medium', 'm1.large', 'm1.xlarge']
|
|
self.assertFilterResults(filters, expected)
|
|
|
|
def test_min_root_gb_filter(self):
|
|
# Exclude everything but large and xlarge which have >= 80 GB.
|
|
filters = dict(min_root_gb=80)
|
|
expected = ['m1.large', 'm1.xlarge']
|
|
self.assertFilterResults(filters, expected)
|
|
|
|
def test_min_memory_mb_AND_root_gb_filter(self):
|
|
# Exclude everything but large and xlarge which have >= 80 GB.
|
|
filters = dict(min_memory_mb=16384, min_root_gb=80)
|
|
expected = ['m1.xlarge']
|
|
self.assertFilterResults(filters, expected)
|
|
|
|
|
|
class CreateInstanceTypeTest(test.TestCase):
|
|
|
|
def assertInvalidInput(self, *create_args, **create_kwargs):
|
|
self.assertRaises(exception.InvalidInput, flavors.create,
|
|
*create_args, **create_kwargs)
|
|
|
|
def test_create_with_valid_name(self):
|
|
# Names can contain alphanumeric and [_.- ]
|
|
flavors.create('azAZ09. -_', 64, 1, 120)
|
|
# And they are not limited to ascii characters
|
|
# E.g.: m1.huge in simplified Chinese
|
|
flavors.create(u'm1.\u5DE8\u5927', 6400, 100, 12000)
|
|
|
|
def test_name_with_special_characters(self):
|
|
# Names can contain all printable characters
|
|
flavors.create('_foo.bar-123', 64, 1, 120)
|
|
|
|
# Ensure instance types raises InvalidInput for invalid characters.
|
|
self.assertInvalidInput('foobar\x00', 64, 1, 120)
|
|
|
|
def test_name_with_non_printable_characters(self):
|
|
# Names cannot contain printable characters
|
|
self.assertInvalidInput(u'm1.\u0868 #', 64, 1, 120)
|
|
|
|
def test_name_length_checks(self):
|
|
MAX_LEN = 255
|
|
|
|
# Flavor name with 255 characters or less is valid.
|
|
flavors.create('a' * MAX_LEN, 64, 1, 120)
|
|
|
|
# Flavor name which is more than 255 characters will cause error.
|
|
self.assertInvalidInput('a' * (MAX_LEN + 1), 64, 1, 120)
|
|
|
|
# Flavor name which is empty should cause an error
|
|
self.assertInvalidInput('', 64, 1, 120)
|
|
|
|
def test_all_whitespace_flavor_names_rejected(self):
|
|
self.assertInvalidInput(' ', 64, 1, 120)
|
|
|
|
def test_flavorid_with_invalid_characters(self):
|
|
# Ensure Flavor ID can only contain [a-zA-Z0-9_.- ]
|
|
self.assertInvalidInput('a', 64, 1, 120, flavorid=u'\u2605')
|
|
self.assertInvalidInput('a', 64, 1, 120, flavorid='%%$%$@#$#@$@#$^%')
|
|
|
|
def test_flavorid_length_checks(self):
|
|
MAX_LEN = 255
|
|
# Flavor ID which is more than 255 characters will cause error.
|
|
self.assertInvalidInput('a', 64, 1, 120, flavorid='a' * (MAX_LEN + 1))
|
|
|
|
def test_memory_must_be_positive_db_integer(self):
|
|
self.assertInvalidInput('flavor1', 'foo', 1, 120)
|
|
self.assertInvalidInput('flavor1', -1, 1, 120)
|
|
self.assertInvalidInput('flavor1', 0, 1, 120)
|
|
self.assertInvalidInput('flavor1', db.MAX_INT + 1, 1, 120)
|
|
flavors.create('flavor1', 1, 1, 120)
|
|
|
|
def test_vcpus_must_be_positive_db_integer(self):
|
|
self.assertInvalidInput('flavor`', 64, 'foo', 120)
|
|
self.assertInvalidInput('flavor1', 64, -1, 120)
|
|
self.assertInvalidInput('flavor1', 64, 0, 120)
|
|
self.assertInvalidInput('flavor1', 64, db.MAX_INT + 1, 120)
|
|
flavors.create('flavor1', 64, 1, 120)
|
|
|
|
def test_root_gb_must_be_nonnegative_db_integer(self):
|
|
self.assertInvalidInput('flavor1', 64, 1, 'foo')
|
|
self.assertInvalidInput('flavor1', 64, 1, -1)
|
|
self.assertInvalidInput('flavor1', 64, 1, db.MAX_INT + 1)
|
|
flavors.create('flavor1', 64, 1, 0)
|
|
flavors.create('flavor2', 64, 1, 120)
|
|
|
|
def test_ephemeral_gb_must_be_nonnegative_db_integer(self):
|
|
self.assertInvalidInput('flavor1', 64, 1, 120, ephemeral_gb='foo')
|
|
self.assertInvalidInput('flavor1', 64, 1, 120, ephemeral_gb=-1)
|
|
self.assertInvalidInput('flavor1', 64, 1, 120,
|
|
ephemeral_gb=db.MAX_INT + 1)
|
|
flavors.create('flavor1', 64, 1, 120, ephemeral_gb=0)
|
|
flavors.create('flavor2', 64, 1, 120, ephemeral_gb=120)
|
|
|
|
def test_swap_must_be_nonnegative_db_integer(self):
|
|
self.assertInvalidInput('flavor1', 64, 1, 120, swap='foo')
|
|
self.assertInvalidInput('flavor1', 64, 1, 120, swap=-1)
|
|
self.assertInvalidInput('flavor1', 64, 1, 120,
|
|
swap=db.MAX_INT + 1)
|
|
flavors.create('flavor1', 64, 1, 120, swap=0)
|
|
flavors.create('flavor2', 64, 1, 120, swap=1)
|
|
|
|
def test_rxtx_factor_must_be_positive_float(self):
|
|
self.assertInvalidInput('flavor1', 64, 1, 120, rxtx_factor='foo')
|
|
self.assertInvalidInput('flavor1', 64, 1, 120, rxtx_factor=-1.0)
|
|
self.assertInvalidInput('flavor1', 64, 1, 120, rxtx_factor=0.0)
|
|
|
|
flavor = flavors.create('flavor1', 64, 1, 120, rxtx_factor=1.0)
|
|
self.assertEqual(1.0, flavor.rxtx_factor)
|
|
|
|
flavor = flavors.create('flavor2', 64, 1, 120, rxtx_factor=1.1)
|
|
self.assertEqual(1.1, flavor.rxtx_factor)
|
|
|
|
def test_rxtx_factor_must_be_within_sql_float_range(self):
|
|
_context = context.get_admin_context()
|
|
db.flavor_get_all(_context)
|
|
# We do * 10 since this is an approximation and we need to make sure
|
|
# the difference is noticeble.
|
|
over_rxtx_factor = flavors.SQL_SP_FLOAT_MAX * 10
|
|
|
|
self.assertInvalidInput('flavor1', 64, 1, 120,
|
|
rxtx_factor=over_rxtx_factor)
|
|
|
|
flavor = flavors.create('flavor2', 64, 1, 120,
|
|
rxtx_factor=flavors.SQL_SP_FLOAT_MAX)
|
|
self.assertEqual(flavors.SQL_SP_FLOAT_MAX, flavor.rxtx_factor)
|
|
|
|
def test_is_public_must_be_valid_bool_string(self):
|
|
self.assertInvalidInput('flavor1', 64, 1, 120, is_public='foo')
|
|
|
|
flavors.create('flavor1', 64, 1, 120, is_public='TRUE')
|
|
flavors.create('flavor2', 64, 1, 120, is_public='False')
|
|
flavors.create('flavor3', 64, 1, 120, is_public='Yes')
|
|
flavors.create('flavor4', 64, 1, 120, is_public='No')
|
|
flavors.create('flavor5', 64, 1, 120, is_public='Y')
|
|
flavors.create('flavor6', 64, 1, 120, is_public='N')
|
|
flavors.create('flavor7', 64, 1, 120, is_public='1')
|
|
flavors.create('flavor8', 64, 1, 120, is_public='0')
|
|
flavors.create('flavor9', 64, 1, 120, is_public='true')
|
|
|
|
def test_flavorid_populated(self):
|
|
flavor1 = flavors.create('flavor1', 64, 1, 120)
|
|
self.assertIsNot(None, flavor1.flavorid)
|
|
|
|
flavor2 = flavors.create('flavor2', 64, 1, 120, flavorid='')
|
|
self.assertIsNot(None, flavor2.flavorid)
|
|
|
|
flavor3 = flavors.create('flavor3', 64, 1, 120, flavorid='foo')
|
|
self.assertEqual('foo', flavor3.flavorid)
|
|
|
|
def test_default_values(self):
|
|
flavor1 = flavors.create('flavor1', 64, 1, 120)
|
|
|
|
self.assertIsNot(None, flavor1.flavorid)
|
|
self.assertEqual(flavor1.ephemeral_gb, 0)
|
|
self.assertEqual(flavor1.swap, 0)
|
|
self.assertEqual(flavor1.rxtx_factor, 1.0)
|
|
|
|
def test_basic_create(self):
|
|
# Ensure instance types can be created.
|
|
ctxt = context.get_admin_context()
|
|
original_list = objects.FlavorList.get_all(ctxt)
|
|
|
|
# Create new type and make sure values stick
|
|
flavor = flavors.create('flavor', 64, 1, 120)
|
|
self.assertEqual(flavor.name, 'flavor')
|
|
self.assertEqual(flavor.memory_mb, 64)
|
|
self.assertEqual(flavor.vcpus, 1)
|
|
self.assertEqual(flavor.root_gb, 120)
|
|
|
|
# Ensure new type shows up in list
|
|
new_list = objects.FlavorList.get_all(ctxt)
|
|
self.assertNotEqual(len(original_list), len(new_list),
|
|
'flavor was not created')
|
|
|
|
def test_create_then_delete(self):
|
|
ctxt = context.get_admin_context()
|
|
original_list = objects.FlavorList.get_all(ctxt)
|
|
|
|
flavor = flavors.create('flavor', 64, 1, 120)
|
|
|
|
# Ensure new type shows up in list
|
|
new_list = objects.FlavorList.get_all(ctxt)
|
|
self.assertNotEqual(len(original_list), len(new_list),
|
|
'instance type was not created')
|
|
|
|
flavors.destroy('flavor')
|
|
self.assertRaises(exception.FlavorNotFound,
|
|
objects.Flavor.get_by_id, ctxt, flavor.id)
|
|
|
|
# Deleted instance should not be in list anymore
|
|
new_list = objects.FlavorList.get_all(ctxt)
|
|
self.assertEqual(len(original_list), len(new_list))
|
|
for i, f in enumerate(original_list):
|
|
self.assertIsInstance(f, objects.Flavor)
|
|
self.assertEqual(f.flavorid, new_list[i].flavorid)
|
|
|
|
def test_duplicate_names_fail(self):
|
|
# Ensures that name duplicates raise FlavorCreateFailed.
|
|
flavors.create('flavor', 256, 1, 120, 200, 'flavor1')
|
|
self.assertRaises(exception.FlavorExists,
|
|
flavors.create,
|
|
'flavor', 64, 1, 120)
|
|
|
|
def test_duplicate_flavorids_fail(self):
|
|
# Ensures that flavorid duplicates raise FlavorCreateFailed.
|
|
flavors.create('flavor1', 64, 1, 120, flavorid='flavorid')
|
|
self.assertRaises(exception.FlavorIdExists,
|
|
flavors.create,
|
|
'flavor2', 64, 1, 120, flavorid='flavorid')
|