[Freeipa-devel] [PATCH] 0005 Add list of domains associated to our realm to cn=etc

[Jan Cholasta]

Hi,

On 12.2.2013 17:27, Ana Krivokapic wrote:
> Add new LDAP container to store the list of domains associated with IPA
> realm.
> Add two new ipa commands (ipa realmdomains-show and ipa
> realmdomains-mod) to allow manipulation of the list of realm domains.
> Unit test file covering these new commands was added.
>
> https://fedorahosted.org/freeipa/ticket/2945
>

I have just noticed a failure in the test for this:

Traceback (most recent call last):
   File "/usr/lib/python2.7/site-packages/nose/loader.py", line 390, in 
loadTestsFromName
     addr.filename, addr.module)
   File "/usr/lib/python2.7/site-packages/nose/importer.py", line 39, in 
importFromPath
     return self.importFromDir(dir_path, fqname)
   File "/usr/lib/python2.7/site-packages/nose/importer.py", line 86, in 
importFromDir
     mod = load_module(part_fqname, fh, filename, desc)
   File 
"/home/jcholast/freeipa/tests/test_xmlrpc/test_realmdomains_plugin.py", 
line 35, in <module>
     bad_domain = u'this-domain-does-not-exist-%s.com' % 
''.join(random.choice(string.lowercase) for x in range(10))
UnicodeDecodeError: 'utf8' codec can't decode byte 0xed in position 0: 
invalid continuation byte

This is happening because string.lowercase is locale-dependent, so it 
might contain "bad" values, such as '\xed'. Please use 
string.ascii_lowercase instead to prevent this.

Honza

-- 
Jan Cholasta