diff options
author | Rob Crittenden <rcritten@redhat.com> | 2011-01-07 11:17:55 -0500 |
---|---|---|
committer | Rob Crittenden <rcritten@redhat.com> | 2011-01-10 10:32:10 -0500 |
commit | 6f58f38748085e6a104de6f9e992469d3b685d5a (patch) | |
tree | ebc3a92f95feae19afc8ababcf306782e62988f8 /tests/test_xmlrpc/test_user_plugin.py | |
parent | 0a79836082b897ebf734d9073583769cb276937c (diff) | |
download | freeipa-6f58f38748085e6a104de6f9e992469d3b685d5a.tar.gz freeipa-6f58f38748085e6a104de6f9e992469d3b685d5a.tar.xz freeipa-6f58f38748085e6a104de6f9e992469d3b685d5a.zip |
Display the entries that failed when deleting with --continue.
We collected the failures but didn't report it back. This changes the
API of most delete commands so rather than returning a boolean it returns
a dict with the only current key as failed.
This also adds a new parameter flag, suppress_empty. This will try to
not print values that are empty if included. This makes the output of
the delete commands a bit prettier.
ticket 687
Diffstat (limited to 'tests/test_xmlrpc/test_user_plugin.py')
-rw-r--r-- | tests/test_xmlrpc/test_user_plugin.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/tests/test_xmlrpc/test_user_plugin.py b/tests/test_xmlrpc/test_user_plugin.py index 4754e763f..73d170af7 100644 --- a/tests/test_xmlrpc/test_user_plugin.py +++ b/tests/test_xmlrpc/test_user_plugin.py @@ -370,7 +370,7 @@ class test_user(Declarative): desc='Delete %r' % user1, command=('user_del', [user1], {}), expected=dict( - result=True, + result=dict(failed=u''), summary=u'Deleted user "tuser1"', value=user1, ), @@ -411,6 +411,7 @@ class test_user(Declarative): ), ), + dict( desc='Create %r' % user2, command=( @@ -438,11 +439,12 @@ class test_user(Declarative): ), ), + dict( desc='Delete %r and %r at the same time' % (user1, user2), command=('user_del', [user1, user2], {}), expected=dict( - result=True, + result=dict(failed=u''), summary=u'Deleted user "tuser1,tuser2"', value=u','.join((user1, user2)), ), |