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_role_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_role_plugin.py')
| -rw-r--r-- | tests/test_xmlrpc/test_role_plugin.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/test_xmlrpc/test_role_plugin.py b/tests/test_xmlrpc/test_role_plugin.py index 283da2a0e..28d1c6b43 100644 --- a/tests/test_xmlrpc/test_role_plugin.py +++ b/tests/test_xmlrpc/test_role_plugin.py @@ -411,7 +411,7 @@ class test_role(Declarative): desc='Delete %r' % group1, command=('group_del', [group1], {}), expected=dict( - result=True, + result=dict(failed=u''), value=group1, summary=u'Deleted group "testgroup1"', ) @@ -495,7 +495,7 @@ class test_role(Declarative): desc='Delete %r' % role1, command=('role_del', [role1], {}), expected=dict( - result=True, + result=dict(failed=u''), value=role1, summary=u'Deleted role "%s"' % role1, ) @@ -545,7 +545,7 @@ class test_role(Declarative): desc='Delete %r' % role2, command=('role_del', [role2], {}), expected=dict( - result=True, + result=dict(failed=u''), value=role2, summary=u'Deleted role "%s"' % role2, ) |