IntenseWebs/freeipa
3
0
Fork 0
mirror of https://salsa.debian.org/freeipa-team/freeipa.git synced 2025-02-25 18:55:28 -06:00
Code Issues Packages Projects Releases Wiki Activity

259: Option.__normalize_scalar() now raises a TypeError if not isinstance(value, basestring); updated corresponding unit tests

Browse Source
This commit is contained in:
Jason Gerard DeRose 2008-09-04 07:47:07 +00:00
parent 71d36aa6a0
commit c7cd694d4f
2 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
ipalib/public.py
View File

@ -125,8 +125,8 @@ class Option(plugable.ReadOnly):
return self.__convert_scalar(value)
def __normalize_scalar(self, value):
if type(value) is not self.type.type:
raise_TypeError(value, self.type.type, 'value')
if not isinstance(value, basestring):
raise_TypeError(value, basestring, 'value')
return self.__normalize(value)
def normalize(self, value):

12
ipalib/tests/test_public.py
View File

@ -202,10 +202,10 @@ class test_Option(ClassChecker):
# Scenario 2: multivalue=False, normalize=callback
o = self.cls(name, doc, t, normalize=callback)
for v in (u'Hello', u'hello'): # Okay
assert o.normalize(v) == u'hello'
for v in [None, 'hello', (u'Hello',)]: # Not unicode
check_TypeError(v, unicode, 'value', o.normalize, v)
for v in (u'Hello', u'hello', 'Hello'): # Okay
assert o.normalize(v) == 'hello'
for v in [None, 42, (u'Hello',)]: # Not basestring
check_TypeError(v, basestring, 'value', o.normalize, v)
# Scenario 3: multivalue=True, normalize=None
o = self.cls(name, doc, t, multivalue=True)
@ -219,9 +219,9 @@ class test_Option(ClassChecker):
assert o.normalize(value) == (u'hello',)
for v in (None, u'Hello', [u'hello']): # Not tuple
check_TypeError(v, tuple, 'value', o.normalize, v)
fail = 'Hello' # Not unicode
fail = 42 # Not basestring
for v in [(fail,), (u'Hello', fail)]: # Non unicode member
check_TypeError(fail, unicode, 'value', o.normalize, v)
check_TypeError(fail, basestring, 'value', o.normalize, v)
def test_validate(self):
"""