diff options
author | Jr Aquino <jr.aquino@citrix.com> | 2011-06-16 11:57:13 -0700 |
---|---|---|
committer | Rob Crittenden <rcritten@redhat.com> | 2011-06-16 19:21:07 -0400 |
commit | 44cdf8ef54ff761a5e38919b8cdce5128928985a (patch) | |
tree | 9faa2078dfcea92034b4d7c480ab67b37415e18e /tests/test_xmlrpc | |
parent | 0377123b191a0c94434ef1eeda653426ddd0a440 (diff) | |
download | freeipa-44cdf8ef54ff761a5e38919b8cdce5128928985a.tar.gz freeipa-44cdf8ef54ff761a5e38919b8cdce5128928985a.tar.xz freeipa-44cdf8ef54ff761a5e38919b8cdce5128928985a.zip |
Raise DuplicateEntry Error when adding a duplicate sudo option
https://fedorahosted.org/freeipa/ticket/1276
https://fedorahosted.org/freeipa/ticket/1277
https://fedorahosted.org/freeipa/ticket/1308
Added new Exception: AttrValueNotFound
Fixed XML Test for Sudorule remove_option
1276 (Raise AttrValueNotFound when trying to remove a non-existent option from Sudo rule)
1277 (Raise DuplicateEntry Error when adding a duplicate sudo option)
1308 (Make sudooption a required option for sudorule_remove_option)
Diffstat (limited to 'tests/test_xmlrpc')
-rw-r--r-- | tests/test_xmlrpc/test_sudorule_plugin.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/tests/test_xmlrpc/test_sudorule_plugin.py b/tests/test_xmlrpc/test_sudorule_plugin.py index 3de1adce0..88c608d61 100644 --- a/tests/test_xmlrpc/test_sudorule_plugin.py +++ b/tests/test_xmlrpc/test_sudorule_plugin.py @@ -316,7 +316,6 @@ class test_sudorule(XMLRPC_test): ret = api.Command['sudorule_add_runasgroup']( self.rule_name, group=self.test_external_group ) - print ret assert ret['completed'] == 1 failed = ret['failed'] entry = ret['result'] @@ -330,7 +329,6 @@ class test_sudorule(XMLRPC_test): ret = api.Command['sudorule_remove_runasgroup']( self.rule_name, group=self.test_external_group ) - print ret assert ret['completed'] == 1 failed = ret['failed'] entry = ret['result'] @@ -355,7 +353,8 @@ class test_sudorule(XMLRPC_test): ret = api.Command['sudorule_remove_option']( self.rule_name, ipasudoopt=self.test_option ) - assert ret['result'] is True + entry = ret['result'] + assert 'ipasudoopt' not in entry def test_a_sudorule_add_host(self): """ |