Fixed Warning messages about log dir in unit test

5 files changed, 20 insertions, 28 deletions

diff --git a/ipalib/errors.py b/ipalib/errors.py
index 8412d1f6..25f594f2 100644
--- a/ipalib/errors.py
+++ b/ipalib/errors.py
@@ -123,7 +123,7 @@ def _(text):
 
 class HandledError(StandardError):
     """
-    Base class for errors that can be raised across a remote procecdure call.
+    Base class for errors that can be raised across a remote procedure call.
     """
 
     code = 1
@@ -135,7 +135,6 @@ class HandledError(StandardError):
         StandardError.__init__(self, message)
 
 
-
 class UnknownError(HandledError):
     """
     Raised when the true error is not a handled error.
@@ -145,10 +144,12 @@ class UnknownError(HandledError):
 
 
 class CommandError(HandledError):
+    """
+    Raised when an unknown command is called client-side.
+    """
     format = _('Unknown command %(name)r')
 
 
-
 class RemoteCommandError(HandledError):
     format = 'Server at %(uri)r has no command %(name)r'
 
diff --git a/make-test b/make-test
index 485447a6..00ea676a 100755
--- a/make-test
+++ b/make-test
@@ -10,7 +10,7 @@ do
     if [[ -f $executable ]]; then
         echo "[ $name: Starting tests... ]"
         ((runs += 1))
-        if $executable /usr/bin/nosetests --with-doctest --stop
+        if $executable /usr/bin/nosetests -v --with-doctest --stop
         then
             echo "[ $name: Tests OK ]"
         else
diff --git a/tests/test_ipa_server/test_rpc.py b/tests/test_ipa_server/test_rpc.py
index f73f6cd0..9c6cf23a 100644
--- a/tests/test_ipa_server/test_rpc.py
+++ b/tests/test_ipa_server/test_rpc.py
@@ -74,3 +74,7 @@ class test_xmlrpc(PluginTester):
         options = dict(option1=u'How are you?', option2=unicode_str)
         assert call((arg1, arg2, options)) == (arg1, arg2, options)
         assert call((arg1,) + arg2 + (options,)) == (arg1, arg2, options)
+
+
+    def test_execute(self):
+        (o, api, home) = self.instance('Backend', in_server=True)
diff --git a/tests/test_ipalib/test_frontend.py b/tests/test_ipalib/test_frontend.py
index 87296705..94e586fe 100644
--- a/tests/test_ipalib/test_frontend.py
+++ b/tests/test_ipalib/test_frontend.py
@@ -22,7 +22,7 @@ Test the `ipalib.frontend` module.
 """
 
 from tests.util import raises, getitem, no_set, no_del, read_only
-from tests.util import check_TypeError, ClassChecker, get_api
+from tests.util import check_TypeError, ClassChecker, create_test_api
 from ipalib import frontend, backend, plugable, errors, ipa_types, config
 
 
@@ -790,9 +790,7 @@ class test_Command(ClassChecker):
         kw = dict(how_are='you', on_this='fine day?')
 
         # Test in server context:
-        api = plugable.API(self.cls)
-        #api.env.update(dict(server_context=True))
-        api.env.in_server = True
+        (api, home) = create_test_api(in_server=True)
         api.finalize()
         o = my_cmd()
         o.set_api(api)
@@ -801,9 +799,7 @@ class test_Command(ClassChecker):
         assert o.run.im_func is my_cmd.execute.im_func
 
         # Test in non-server context
-        api = plugable.API(self.cls)
-        #api.env.update(dict(server_context=False))
-        api.env.in_server = False
+        (api, home) = create_test_api(in_server=False)
         api.finalize()
         o = my_cmd()
         o.set_api(api)
@@ -844,7 +840,7 @@ class test_LocalOrRemote(ClassChecker):
                 return ('execute', args, options)
 
         # Test when in_server=False:
-        (api, home) = get_api(in_server=False)
+        (api, home) = create_test_api(in_server=False)
         api.register(example)
         api.finalize()
         cmd = api.Command.example
@@ -855,7 +851,7 @@ class test_LocalOrRemote(ClassChecker):
             ('forward', (u'var',), dict(server=True))
 
         # Test when in_server=True (should always call execute):
-        (api, home) = get_api(in_server=True)
+        (api, home) = create_test_api(in_server=True)
         api.register(example)
         api.finalize()
         cmd = api.Command.example
@@ -986,11 +982,7 @@ class test_Object(ClassChecker):
         """
         Test the `ipalib.frontend.Object.primary_key` attribute.
         """
-        api = plugable.API(
-            frontend.Method,
-            frontend.Property,
-        )
-        #config.set_default_env(api.env)
+        (api, home) = create_test_api()
         api.finalize()
 
         # Test with no primary keys:
@@ -1041,13 +1033,7 @@ class test_Object(ClassChecker):
         """
         Test the `ipalib.frontend.Object.backend` attribute.
         """
-        api = plugable.API(
-            frontend.Object,
-            frontend.Method,
-            frontend.Property,
-            backend.Backend,
-        )
-        #config.set_default_env(api.env)
+        (api, home) = create_test_api()
         class ldap(backend.Backend):
             whatever = 'It worked!'
         api.register(ldap)
diff --git a/tests/test_ipalib/test_plugable.py b/tests/test_ipalib/test_plugable.py
index ce0c79a9..6b3b3e6c 100644
--- a/tests/test_ipalib/test_plugable.py
+++ b/tests/test_ipalib/test_plugable.py
@@ -23,7 +23,7 @@ Test the `ipalib.plugable` module.
 
 from tests.util import raises, no_set, no_del, read_only
 from tests.util import getitem, setitem, delitem
-from tests.util import ClassChecker, get_api
+from tests.util import ClassChecker, create_test_api
 from ipalib import plugable, errors
 
 
@@ -860,6 +860,7 @@ class test_API(ClassChecker):
         class NoProxy(plugable.Plugin):
             __proxy__ = False
         api = plugable.API(NoProxy)
+        api.env.mode = 'unit_test'
         class plugin0(NoProxy):
             pass
         api.register(plugin0)
@@ -879,7 +880,7 @@ class test_API(ClassChecker):
         """
         Test the `ipalib.plugable.API.bootstrap` method.
         """
-        (o, home) = get_api()
+        (o, home) = create_test_api()
         assert o.env._isdone('_bootstrap') is False
         assert o.env._isdone('_finalize_core') is False
         assert o.isdone('bootstrap') is False
@@ -895,7 +896,7 @@ class test_API(ClassChecker):
         """
         Test the `ipalib.plugable.API.load_plugins` method.
         """
-        (o, home) = get_api()
+        (o, home) = create_test_api()
         assert o.isdone('bootstrap') is False
         assert o.isdone('load_plugins') is False
         o.load_plugins()