From: Jelmer Vernooij <jelmer@samba.org>
Date: Sun, 4 Mar 2012 01:29:34 +0000 (+0100)
Subject: selftest.target: Add tests for check.
X-Git-Tag: tdb-1.2.10~409
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=e7633d86c03c57b6fc8a4e5b4a26f7e9159c6ec7;p=thirdparty%2Fsamba.git

selftest.target: Add tests for check.
---

diff --git a/selftest/target/__init__.py b/selftest/target/__init__.py
index ac2cc19a480..72e0770d92b 100644
--- a/selftest/target/__init__.py
+++ b/selftest/target/__init__.py
@@ -128,11 +128,10 @@ class EnvironmentManager(object):
                 raise EnvironmentDown(running_env.get_log())
             return running_env
 
-
         env = self.target.setup_env(envname, prefix)
         if env is None:
             return None
 
-        self.running_envs[env] = env
+        self.running_envs[envname] = env
 
         return env
diff --git a/selftest/tests/test_target.py b/selftest/tests/test_target.py
index 343de930cbf..0abc79b6bb5 100644
--- a/selftest/tests/test_target.py
+++ b/selftest/tests/test_target.py
@@ -63,3 +63,11 @@ class EnvironmentManagerTests(unittest.TestCase):
         env = self.mgr.setup_env("something", "prefix")
         self.assertEquals(env.name, "something")
         self.assertEquals(env.prefix, "prefix")
+
+    def test_check(self):
+        env = self.mgr.setup_env("something", "prefix")
+        self.assertTrue(env.check())
+        self.assertTrue(self.mgr.check_env("something"))
+        env.check_ret = False
+        self.assertFalse(env.check())
+        self.assertFalse(self.mgr.check_env("something"))