summaryrefslogtreecommitdiffstats
path: root/tests
diff options
context:
space:
mode:
Diffstat (limited to 'tests')
-rw-r--r--tests/test_ipaserver/test_rpcserver.py3
-rw-r--r--tests/test_xmlrpc/test_hbac_plugin.py6
-rw-r--r--tests/util.py4
3 files changed, 6 insertions, 7 deletions
diff --git a/tests/test_ipaserver/test_rpcserver.py b/tests/test_ipaserver/test_rpcserver.py
index 97632b05d..96d4614a1 100644
--- a/tests/test_ipaserver/test_rpcserver.py
+++ b/tests/test_ipaserver/test_rpcserver.py
@@ -96,7 +96,7 @@ def test_internal_error():
assert s.headers == [('Content-Type', 'text/html; charset=utf-8')]
-def test_internal_error():
+def test_unauthorized_error():
f = rpcserver.HTTP_Status()
t = rpcserver._unauthorized_template
s = StartResponse()
@@ -109,7 +109,6 @@ def test_internal_error():
assert s.headers == [('Content-Type', 'text/html; charset=utf-8')]
-
def test_params_2_args_options():
"""
Test the `ipaserver.rpcserver.params_2_args_options` function.
diff --git a/tests/test_xmlrpc/test_hbac_plugin.py b/tests/test_xmlrpc/test_hbac_plugin.py
index bd50a585e..78c4973c9 100644
--- a/tests/test_xmlrpc/test_hbac_plugin.py
+++ b/tests/test_xmlrpc/test_hbac_plugin.py
@@ -265,8 +265,8 @@ class test_hbac(XMLRPC_test):
assert 'hostgroup' in failed['memberhost']
assert not failed['memberhost']['hostgroup']
entry = ret['result']
- assert 'memberhost_host' not in res[1]
- assert 'memberhost_hostgroup' not in res[1]
+ assert 'memberhost_host' not in entry
+ assert 'memberhost_hostgroup' not in entry
def test_a_hbacrule_add_sourcehost(self):
"""
@@ -327,7 +327,7 @@ class test_hbac(XMLRPC_test):
entry = ret['result']
assert 'memberservice service' not in entry
- def test_b_hbacrule_remove_host(self):
+ def test_b_hbacrule_remove_sourcehost(self):
"""
Test removing source host and hostgroup from HBAC rule using `xmlrpc.hbacrule_remove_host`.
"""
diff --git a/tests/util.py b/tests/util.py
index 5a365fbcd..9bce7c08c 100644
--- a/tests/util.py
+++ b/tests/util.py
@@ -446,7 +446,7 @@ def check_TypeError(value, type_, name, callback, *args, **kw):
assert e.type is type_
assert e.name == name
assert type(e.name) is str
- assert str(e) == ipalib.errors.TYPE_FORMAT % (name, type_, value)
+ assert str(e) == ipalib.errors.TYPE_ERROR % (name, type_, value)
return e
@@ -592,7 +592,7 @@ class DummyClass(object):
def __process(self, name_, args_, kw_):
if self.__i >= len(self.__calls):
raise AssertionError(
- 'extra call: %s, %r, %r' % (name, args, kw)
+ 'extra call: %s, %r, %r' % (name_, args_, kw_)
)
(name, args, kw, result) = self.__calls[self.__i]
self.__i += 1