summaryrefslogtreecommitdiffstats
path: root/tests/test_xmlrpc
diff options
context:
space:
mode:
authorPetr Viktorin <pviktori@redhat.com>2012-03-14 11:33:41 -0400
committerRob Crittenden <rcritten@redhat.com>2012-03-25 18:05:33 -0400
commit0b01751c1bd7c579ab8f7fb64d9182c6f107ab3b (patch)
tree904d4cb862f600f0249d6db99bbfc4423bc944e0 /tests/test_xmlrpc
parenta735420a9ba3d507855a75a1a48f79a2358c7081 (diff)
downloadfreeipa-0b01751c1bd7c579ab8f7fb64d9182c6f107ab3b.tar.gz
freeipa-0b01751c1bd7c579ab8f7fb64d9182c6f107ab3b.tar.xz
freeipa-0b01751c1bd7c579ab8f7fb64d9182c6f107ab3b.zip
Use valid argument names in tests
Some of our tests used unintended extra options, or options with misspelled, wrongly copy-pasted or otherwise bad names. These are ignored, so the intended argument was treated as missing. The test itself can still pass but may be rendered ineffective or fragile. This only fixes those of such errors that appear in the test suite. Fixing code in the framework and actual rejecting of unknown arguments is deferred for later (ticket #2509).
Diffstat (limited to 'tests/test_xmlrpc')
-rw-r--r--tests/test_xmlrpc/test_automember_plugin.py5
-rw-r--r--tests/test_xmlrpc/test_delegation_plugin.py2
-rw-r--r--tests/test_xmlrpc/test_hbac_plugin.py6
-rw-r--r--tests/test_xmlrpc/test_hbactest_plugin.py2
-rw-r--r--tests/test_xmlrpc/test_host_plugin.py2
-rw-r--r--tests/test_xmlrpc/test_pwpolicy.py2
-rw-r--r--tests/test_xmlrpc/test_role_plugin.py2
-rw-r--r--tests/test_xmlrpc/test_sudorule_plugin.py6
8 files changed, 12 insertions, 15 deletions
diff --git a/tests/test_xmlrpc/test_automember_plugin.py b/tests/test_xmlrpc/test_automember_plugin.py
index 1241bd669..265c3af86 100644
--- a/tests/test_xmlrpc/test_automember_plugin.py
+++ b/tests/test_xmlrpc/test_automember_plugin.py
@@ -718,10 +718,7 @@ class test_automember(Declarative):
dict(
desc='Retrieve default automember group for groups',
command=(
- 'automember_default_group_show', [], dict(
- type=u'group',
- automemberdefaultgroup=defaultgroup1,
- )
+ 'automember_default_group_show', [], dict(type=u'group')
),
expected=dict(
result=dict(
diff --git a/tests/test_xmlrpc/test_delegation_plugin.py b/tests/test_xmlrpc/test_delegation_plugin.py
index 5030f8bc2..c4473b9d4 100644
--- a/tests/test_xmlrpc/test_delegation_plugin.py
+++ b/tests/test_xmlrpc/test_delegation_plugin.py
@@ -45,7 +45,7 @@ class test_delegation(Declarative):
dict(
desc='Try to update non-existent %r' % delegation1,
- command=('delegation_mod', [delegation1], dict(description=u'Foo')),
+ command=('delegation_mod', [delegation1], dict(group=u'admins')),
expected=errors.NotFound(reason='no such entry'),
),
diff --git a/tests/test_xmlrpc/test_hbac_plugin.py b/tests/test_xmlrpc/test_hbac_plugin.py
index bbaf9d25b..58265dc07 100644
--- a/tests/test_xmlrpc/test_hbac_plugin.py
+++ b/tests/test_xmlrpc/test_hbac_plugin.py
@@ -123,7 +123,7 @@ class test_hbac(XMLRPC_test):
Test searching for HBAC rules using `xmlrpc.hbacrule_find`.
"""
ret = api.Command['hbacrule_find'](
- name=self.rule_name, accessruletype=self.rule_type,
+ cn=self.rule_name, accessruletype=self.rule_type,
description=self.rule_desc_mod
)
assert ret['truncated'] is False
@@ -155,7 +155,7 @@ class test_hbac(XMLRPC_test):
self.test_sourcehostgroup, description=u'desc'
)
self.failsafe_add(api.Object.hbacsvc,
- self.test_service, description=u'desc', force=True
+ self.test_service, description=u'desc',
)
def test_8_hbacrule_add_user(self):
@@ -424,7 +424,7 @@ class test_hbac(XMLRPC_test):
"""
api.Command['hbacrule_mod'](self.rule_name, usercategory=u'all')
try:
- api.Command['hbacrule_add_user'](self.rule_name, users=u'admin')
+ api.Command['hbacrule_add_user'](self.rule_name, user=u'admin')
finally:
api.Command['hbacrule_mod'](self.rule_name, usercategory=u'')
diff --git a/tests/test_xmlrpc/test_hbactest_plugin.py b/tests/test_xmlrpc/test_hbactest_plugin.py
index 5d829b14c..bc12e8974 100644
--- a/tests/test_xmlrpc/test_hbactest_plugin.py
+++ b/tests/test_xmlrpc/test_hbactest_plugin.py
@@ -79,7 +79,7 @@ class test_hbactest(XMLRPC_test):
self.test_sourcehostgroup, description=u'desc'
)
self.failsafe_add(api.Object.hbacsvc,
- self.test_service, description=u'desc', force=True
+ self.test_service, description=u'desc'
)
for i in [0,1,2,3]:
diff --git a/tests/test_xmlrpc/test_host_plugin.py b/tests/test_xmlrpc/test_host_plugin.py
index 5e49e2bad..2ef99c4a4 100644
--- a/tests/test_xmlrpc/test_host_plugin.py
+++ b/tests/test_xmlrpc/test_host_plugin.py
@@ -123,7 +123,7 @@ class test_host(Declarative):
command=('host_add', [fqdn1],
dict(
description=u'Test host 1',
- localityname=u'Undisclosed location 1',
+ locality=u'Undisclosed location 1',
force=True,
),
),
diff --git a/tests/test_xmlrpc/test_pwpolicy.py b/tests/test_xmlrpc/test_pwpolicy.py
index c0ead9f78..6b5d64cf3 100644
--- a/tests/test_xmlrpc/test_pwpolicy.py
+++ b/tests/test_xmlrpc/test_pwpolicy.py
@@ -49,7 +49,7 @@ class test_pwpolicy(XMLRPC_test):
self.failsafe_add(
api.Object.user, self.user, givenname=u'Test', sn=u'User'
)
- api.Command.group_add_member(self.group, users=self.user)
+ api.Command.group_add_member(self.group, user=self.user)
entry = api.Command['pwpolicy_add'](self.group, **self.kw)['result']
assert_attr_equal(entry, 'krbminpwdlife', '30')
diff --git a/tests/test_xmlrpc/test_role_plugin.py b/tests/test_xmlrpc/test_role_plugin.py
index 6f6de3214..8a5dede23 100644
--- a/tests/test_xmlrpc/test_role_plugin.py
+++ b/tests/test_xmlrpc/test_role_plugin.py
@@ -81,7 +81,7 @@ class test_role(Declarative):
dict(
desc='Try to rename non-existent %r' % role1,
- command=('role_del', [role1], dict(setattr=u'cn=%s' % renamedrole1)),
+ command=('role_mod', [role1], dict(setattr=u'cn=%s' % renamedrole1)),
expected=errors.NotFound(reason='no such entry'),
),
diff --git a/tests/test_xmlrpc/test_sudorule_plugin.py b/tests/test_xmlrpc/test_sudorule_plugin.py
index 2a597c37b..7c45256ec 100644
--- a/tests/test_xmlrpc/test_sudorule_plugin.py
+++ b/tests/test_xmlrpc/test_sudorule_plugin.py
@@ -97,7 +97,7 @@ class test_sudorule(XMLRPC_test):
Test searching for Sudo rules using `xmlrpc.sudorule_find`.
"""
ret = api.Command['sudorule_find'](
- name=self.rule_name,
+ cn=self.rule_name,
description=self.rule_desc_mod
)
assert ret['truncated'] is False
@@ -131,7 +131,7 @@ class test_sudorule(XMLRPC_test):
self.test_sudodenycmdgroup, description=u'desc'
)
self.failsafe_add(api.Object.sudocmd,
- self.test_command, description=u'desc', force=True
+ self.test_command, description=u'desc'
)
def test_8_sudorule_add_user(self):
@@ -592,7 +592,7 @@ class test_sudorule(XMLRPC_test):
"""
api.Command['sudorule_mod'](self.rule_name, ipasudorunasusercategory=u'all')
try:
- api.Command['sudorule_add_runasuser'](self.rule_name, sudocmd=self.test_user)
+ api.Command['sudorule_add_runasuser'](self.rule_name, user=self.test_user)
finally:
api.Command['sudorule_mod'](self.rule_name, ipasudorunasusercategory=u'')