diff options
author | Ondrej Hamada <ohamada@redhat.com> | 2012-04-11 09:37:15 +0200 |
---|---|---|
committer | Rob Crittenden <rcritten@redhat.com> | 2012-04-11 22:29:04 -0400 |
commit | 2584e9be673985d530447b836b38cb4ddbf8ee21 (patch) | |
tree | a70500c0ba84397987f9e05007235bba970d355a /tests | |
parent | fca43ccd474db550cd68c17a9a03fc9436128b34 (diff) | |
download | freeipa-2584e9be673985d530447b836b38cb4ddbf8ee21.tar.gz freeipa-2584e9be673985d530447b836b38cb4ddbf8ee21.tar.xz freeipa-2584e9be673985d530447b836b38cb4ddbf8ee21.zip |
Unable to rename permission object
The update was failing because of the case insensitivity of permission
object DN. Unit-tests added.
https://fedorahosted.org/freeipa/ticket/2571
Diffstat (limited to 'tests')
-rw-r--r-- | tests/test_xmlrpc/test_permission_plugin.py | 45 |
1 files changed, 41 insertions, 4 deletions
diff --git a/tests/test_xmlrpc/test_permission_plugin.py b/tests/test_xmlrpc/test_permission_plugin.py index d730399c1..8ee682a58 100644 --- a/tests/test_xmlrpc/test_permission_plugin.py +++ b/tests/test_xmlrpc/test_permission_plugin.py @@ -36,6 +36,10 @@ permission1_renamed = u'testperm1_rn' permission1_renamed_dn = DN(('cn',permission1_renamed), api.env.container_permission,api.env.basedn) +permission1_renamed_ucase = u'Testperm_RN' +permission1_renamed_ucase_dn = DN(('cn',permission1_renamed_ucase.lower()), + api.env.container_permission,api.env.basedn) + permission2 = u'testperm2' permission2_dn = DN(('cn',permission2), @@ -466,6 +470,17 @@ class test_permission(Declarative): dict( + desc='Try to rename %r to empty name' % (permission1), + command=( + 'permission_mod', [permission1], dict(rename=u'', + permissions=u'all',) + ), + expected=errors.ValidationError(name=u'rename', + error=u'New name can not be empty'), + ), + + + dict( desc='Check integrity of original permission %r' % permission1, command=('permission_show', [permission1], {}), expected=dict( @@ -506,12 +521,34 @@ class test_permission(Declarative): dict( - desc='Delete %r' % permission1_renamed, - command=('permission_del', [permission1_renamed], {}), + desc='Rename %r to permission %r' % (permission1_renamed, + permission1_renamed_ucase), + command=( + 'permission_mod', [permission1_renamed], dict(rename=permission1_renamed_ucase, + permissions= u'write',) + ), expected=dict( - result=dict(failed=u''), value=permission1_renamed, - summary=u'Deleted permission "%s"' % permission1_renamed, + summary=u'Modified permission "%s"' % permission1_renamed, + result={ + 'dn': lambda x: DN(x) == permission1_renamed_ucase_dn, + 'cn': [permission1_renamed_ucase.lower()], + 'member_privilege': [privilege1], + 'type': u'user', + 'permissions': [u'write'], + 'memberof': u'ipausers', + }, + ), + ), + + + dict( + desc='Delete %r' % permission1_renamed_ucase, + command=('permission_del', [permission1_renamed_ucase], {}), + expected=dict( + result=dict(failed=u''), + value=permission1_renamed_ucase, + summary=u'Deleted permission "%s"' % permission1_renamed_ucase, ) ), |