diff options
author | Jan Cholasta <jcholast@redhat.com> | 2016-05-18 09:42:56 +0200 |
---|---|---|
committer | Jan Cholasta <jcholast@redhat.com> | 2016-05-25 16:06:26 +0200 |
commit | e0275abe6f5192e68b7f57acf37b01aaa89003ea (patch) | |
tree | 3023bbb3db7e3369e865eabfe6b878e09cf7d38c /ipatests/test_xmlrpc/test_realmdomains_plugin.py | |
parent | 861294c0d62891dc9977e7203cc07496e35ec939 (diff) | |
download | freeipa-e0275abe6f5192e68b7f57acf37b01aaa89003ea.tar.gz freeipa-e0275abe6f5192e68b7f57acf37b01aaa89003ea.tar.xz freeipa-e0275abe6f5192e68b7f57acf37b01aaa89003ea.zip |
rpc: include structured error information in responses
Include keyword arguments of exceptions in RPC responses. This is limited
to JSON-RPC, as XML-RPC does not support additional data in error
responses.
Include keyword arguments of messages in RPC responses.
Include keyword arguments of exceptions in batch command result.
https://fedorahosted.org/freeipa/ticket/4739
Reviewed-By: David Kupka <dkupka@redhat.com>
Diffstat (limited to 'ipatests/test_xmlrpc/test_realmdomains_plugin.py')
-rw-r--r-- | ipatests/test_xmlrpc/test_realmdomains_plugin.py | 46 |
1 files changed, 40 insertions, 6 deletions
diff --git a/ipatests/test_xmlrpc/test_realmdomains_plugin.py b/ipatests/test_xmlrpc/test_realmdomains_plugin.py index cd35729ed..dfc33514b 100644 --- a/ipatests/test_xmlrpc/test_realmdomains_plugin.py +++ b/ipatests/test_xmlrpc/test_realmdomains_plugin.py @@ -106,7 +106,13 @@ class test_realmdomains(Declarative): "'%s'" % (new_domain_1, api.env.realm), u'code': 13011, u'type': u'warning', - u'name': u'KerberosTXTRecordCreationFailure'}, + u'name': u'KerberosTXTRecordCreationFailure', + u'data': { + u'domain': new_domain_1, + u'realm': api.env.realm, + u'error': (u"%s.: DNS zone not found" % + new_domain_1), + }}, ), result=dict( associateddomain=[our_domain, new_domain_1], @@ -131,7 +137,13 @@ class test_realmdomains(Declarative): realm=api.env.realm), u'code': 13011, u'type': u'warning', - u'name': u'KerberosTXTRecordCreationFailure'}, + u'name': u'KerberosTXTRecordCreationFailure', + u'data': { + u'domain': new_domain_2, + u'realm': api.env.realm, + u'error': (u"%s.: DNS zone not found" % + new_domain_2), + }}, ), ), ), @@ -151,7 +163,12 @@ class test_realmdomains(Declarative): "manually." % dict(domain=new_domain_2), u'code': 13012, u'type': u'warning', - u'name': u'KerberosTXTRecordDeletionFailure'}, + u'name': u'KerberosTXTRecordDeletionFailure', + u'data': { + u'domain': new_domain_2, + u'error': (u"%s.: DNS zone not found" % + new_domain_2), + }}, ), ), ), @@ -173,7 +190,13 @@ class test_realmdomains(Declarative): realm=api.env.realm), u'code': 13011, u'type': u'warning', - u'name': u'KerberosTXTRecordCreationFailure'}, + u'name': u'KerberosTXTRecordCreationFailure', + u'data': { + u'domain': new_domain_2, + u'realm': api.env.realm, + u'error': (u"%s.: DNS zone not found" % + new_domain_2), + }}, {u'message': u"The _kerberos TXT record from domain " "%(domain)s could not be removed (%(domain)s.: " "DNS zone not found).\nThis can happen if the zone " @@ -181,7 +204,12 @@ class test_realmdomains(Declarative): "manually." % dict(domain=new_domain_1), u'code': 13012, u'type': u'warning', - u'name': u'KerberosTXTRecordDeletionFailure'}, + u'name': u'KerberosTXTRecordDeletionFailure', + u'data': { + u'domain': new_domain_1, + u'error': (u"%s.: DNS zone not found" % + new_domain_1), + }}, ), ), ), @@ -242,7 +270,13 @@ class test_realmdomains(Declarative): realm=api.env.realm), u'code': 13011, u'type': u'warning', - u'name': u'KerberosTXTRecordCreationFailure'}, + u'name': u'KerberosTXTRecordCreationFailure', + u'data': { + u'domain': bad_domain, + u'realm': api.env.realm, + u'error': (u"%s.: DNS zone not found" % + bad_domain), + }}, ), ), ), |