From ad3cf68ac222bb288945e7e3d1c549ce8b49c02c Mon Sep 17 00:00:00 2001 From: John Dennis Date: Wed, 20 Jul 2011 19:39:05 -0400 Subject: Ticket 1485 - DN pairwise grouping The pairwise grouping used to form RDN's and AVA's proved to be confusing in practice, this patch removes that functionality thus requiring programmers to explicitly pair attr,value using a tuple or list. In addition it was discovered additional functionality was needed to support some DN operations in freeipa. DN objects now support startswith(), endswith() and the "in" membership test. These functions and operators will accept either a DN or RDN. The unittest was modified to remove the pairwise tests and add new explicit tests. The unittest was augmented to test the new functionality. In addition the unittest was cleaned up a bit to use common utilty functions for improved readabilty and robustness. The documentation was updated. fix test_role_plugin use of DN to avoid pairwise grouping --- tests/test_xmlrpc/test_role_plugin.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tests/test_xmlrpc/test_role_plugin.py') diff --git a/tests/test_xmlrpc/test_role_plugin.py b/tests/test_xmlrpc/test_role_plugin.py index 82342c340..e1485cf42 100644 --- a/tests/test_xmlrpc/test_role_plugin.py +++ b/tests/test_xmlrpc/test_role_plugin.py @@ -44,7 +44,7 @@ group1 = u'testgroup1' group1_dn = u'cn=%s,%s,%s' % (group1, api.env.container_group, api.env.basedn) privilege1 = u'r,w privilege 1' -privilege1_dn = DN('cn', privilege1, DN(api.env.container_privilege), DN(api.env.basedn)) +privilege1_dn = DN(('cn', privilege1), DN(api.env.container_privilege), DN(api.env.basedn)) def escape_comma(value): return value.replace(',', '\\,') -- cgit