diff options
| author | Ana Krivokapic <akrivoka@redhat.com> | 2013-02-21 10:56:03 -0500 |
|---|---|---|
| committer | Rob Crittenden <rcritten@redhat.com> | 2013-03-14 11:44:24 -0400 |
| commit | 66356f0daf2a55c7e64dc648e0f8c765e9a56151 (patch) | |
| tree | 8816618e93bd1d681af4a52e11042ba9842b097a /tests/test_xmlrpc/xmlrpc_test.py | |
| parent | c4ab8dae35e952ae74e49c5ad0fbfbc0718f60f8 (diff) | |
| download | freeipa.git-66356f0daf2a55c7e64dc648e0f8c765e9a56151.tar.gz freeipa.git-66356f0daf2a55c7e64dc648e0f8c765e9a56151.tar.xz freeipa.git-66356f0daf2a55c7e64dc648e0f8c765e9a56151.zip | |
Improve error messages for external group members
When adding a duplicate member to a group, an error message is issued,
informing the user that the entry is already a member of the group.
Similarly, when trying to delete an entry which is not a member,
an error message is issued, informing the user that the entry is not
a member of the group. These error messages were missing in case of
external members.
This patch also adds support for using the AD\name or name@ad.domain.com
format in ipa group-remove-member command. This format was supported in
group-add-member, but not in group-remove-member.
Unit test file covering these cases was also added.
https://fedorahosted.org/freeipa/ticket/3254
Diffstat (limited to 'tests/test_xmlrpc/xmlrpc_test.py')
| -rw-r--r-- | tests/test_xmlrpc/xmlrpc_test.py | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/tests/test_xmlrpc/xmlrpc_test.py b/tests/test_xmlrpc/xmlrpc_test.py index 89a65308..cfd7bd33 100644 --- a/tests/test_xmlrpc/xmlrpc_test.py +++ b/tests/test_xmlrpc/xmlrpc_test.py @@ -49,6 +49,9 @@ _sid_identifier_authority = '(0x[0-9a-f]{1,12}|[0-9]{1,10})' fuzzy_domain_sid = Fuzzy( '^S-1-5-21-%(idauth)s-%(idauth)s-%(idauth)s$' % dict(idauth=_sid_identifier_authority) ) +fuzzy_user_or_group_sid = Fuzzy( + '^S-1-5-21-%(idauth)s-%(idauth)s-%(idauth)s-%(idauth)s$' % dict(idauth=_sid_identifier_authority) +) # Matches netgroup dn. Note (?i) at the beginning of the regexp is the ingnore case flag fuzzy_netgroupdn = Fuzzy( |