Petr Viktorin wrote:
On 03/26/2012 10:44 PM, Rob Crittenden wrote:
Petr Viktorin wrote:
Patch 25 fixes errors I found by running pylint on the testsuite. They
were in code that was unused, either by error or because it only runs on

Patch 26 adds a test for the batch plugin.

In patch 25 the second test_internal_error should really be
test_unauthorized_error. I think that is a clearer name. Otherwise looks

Patch 26 needs a very minor rebase to fix an error caused by improved
error code handling:

expected = Fuzzy(u"invalid 'gidnumber'.*", <type 'unicode'>, None)
got = u"invalid 'gid': Gettext('must be an integer', domain='ipa',

I tested this:

diff --git a/tests/test_xmlrpc/
index e4280ed..d69bfd9 100644
--- a/tests/test_xmlrpc/
+++ b/tests/test_xmlrpc/
@@ -186,7 +186,7 @@ class test_batch(Declarative):
dict(error=u"'params' is required"),
dict(error=u"'givenname' is required"),
dict(error=u"'description' is required"),
- dict(error=Fuzzy(u"invalid 'gidnumber'.*")),
+ dict(error=Fuzzy(u"invalid 'gid'.*")),


Thank you! Fixed, attaching updated patches.

These look ok but it is baffling to me why tuple needs to be added to the Output format in batch. Do you know when it is being converted into a tuple?


Freeipa-devel mailing list

Reply via email to