openstack
/
nova
Code Issues Proposed changes

Use o.vo FlexibleBooleanField 

As of o.vo 0.4.0, FlexibleBooleanField has existed in o.vo, so change
Nova over to use o.vo's version of it.

Change-Id: If016161ea213f2226499eda792265a7a092f1385
This commit is contained in:
Ryan Rossiter 2015-11-13 21:09:52 +00:00
parent cb9028b9dd
commit de9b0a811a
2 changed files with 1 additions and 35 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
nova/objects/fields.py
View File

@ -15,7 +15,6 @@
from collections import OrderedDict
import netaddr
from oslo_utils import strutils
from oslo_versionedobjects import fields
import six
@ -57,6 +56,7 @@ DictProxyField = fields.DictProxyField
ObjectField = fields.ObjectField
ListOfObjectsField = fields.ListOfObjectsField
VersionPredicateField = fields.VersionPredicateField
FlexibleBooleanField = fields.FlexibleBooleanField
# NOTE(danms): These are things we need to import for some of our
@ -413,13 +413,6 @@ class PciDeviceType(Enum):
valid_values=PciDeviceType.ALL)
# NOTE(danms): Remove this on next release of oslo.versionedobjects
class FlexibleBoolean(fields.Boolean):
@staticmethod
def coerce(obj, attr, value):
return strutils.bool_from_string(value)
class IPAddress(FieldType):
@staticmethod
def coerce(obj, attr, value):
@ -662,16 +655,6 @@ class PciDeviceTypeField(BaseEnumField):
AUTO_TYPE = PciDeviceType()
# FIXME(danms): Remove this after oslo.versionedobjects gets it
# This is a flexible interpretation of boolean
# values using common user friendly semantics for
# truth/falsehood. ie strings like 'yes', 'no',
# 'on', 'off', 't', 'f' get mapped to values you
# would expect.
class FlexibleBooleanField(AutoTypedField):
AUTO_TYPE = FlexibleBoolean()
class IPAddressField(AutoTypedField):
AUTO_TYPE = IPAddress()

17
nova/tests/unit/objects/test_fields.py
View File

@ -627,23 +627,6 @@ class TestBoolean(TestField):
self.from_primitive_values = self.coerce_good_values[0:2]
class TestFlexibleBoolean(TestField):
def setUp(self):
super(TestFlexibleBoolean, self).setUp()
self.field = fields.FlexibleBooleanField()
self.coerce_good_values = [(True, True), (False, False),
("true", True), ("false", False),
("t", True), ("f", False),
("yes", True), ("no", False),
("y", True), ("n", False),
("on", True), ("off", False),
(1, True), (0, False),
('frog', False), ('', False)]
self.coerce_bad_values = []
self.to_primitive_values = self.coerce_good_values[0:2]
self.from_primitive_values = self.coerce_good_values[0:2]
class TestDateTime(TestField):
def setUp(self):
super(TestDateTime, self).setUp()