BIND 10 trac2353, updated. d5c0c97ef8bf7d4b9ae2709b8a92eb1532f4c9b9 [2353] Test verbose case of BoB.start_simple()

BIND 10 source code commits bind10-changes at lists.isc.org
Fri Nov 9 08:04:58 UTC 2012


The branch, trac2353 has been updated
       via  d5c0c97ef8bf7d4b9ae2709b8a92eb1532f4c9b9 (commit)
       via  a7b31b7b074db8359b5c93c3b0d37c807a78ad7e (commit)
       via  7503458897ccad85875ef3bc929ec803750179e8 (commit)
      from  37da25f2eca6b2af80cc5bc2a7e65c5644acf2fb (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit d5c0c97ef8bf7d4b9ae2709b8a92eb1532f4c9b9
Author: Mukund Sivaraman <muks at isc.org>
Date:   Fri Nov 9 13:30:51 2012 +0530

    [2353] Test verbose case of BoB.start_simple()

commit a7b31b7b074db8359b5c93c3b0d37c807a78ad7e
Author: Mukund Sivaraman <muks at isc.org>
Date:   Fri Nov 9 13:26:39 2012 +0530

    [2353] runnable doesn't have to be set here

commit 7503458897ccad85875ef3bc929ec803750179e8
Author: Mukund Sivaraman <muks at isc.org>
Date:   Fri Nov 9 13:26:00 2012 +0530

    [2353] Directly use bob.components instead of bob.get_processes()

-----------------------------------------------------------------------

Summary of changes:
 src/bin/bind10/tests/bind10_test.py.in |   17 ++++++++++++++---
 1 file changed, 14 insertions(+), 3 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/bin/bind10/tests/bind10_test.py.in b/src/bin/bind10/tests/bind10_test.py.in
index 9b54371..27f7982 100644
--- a/src/bin/bind10/tests/bind10_test.py.in
+++ b/src/bin/bind10/tests/bind10_test.py.in
@@ -1472,22 +1472,33 @@ class TestBossComponents(unittest.TestCase):
     def test_register_process(self):
         '''Test that processes can be registered with BoB.'''
         bob = MockBob()
-        bob.runnable = True
         component = MockComponent('test', 53, 'Test')
+
+        self.assertFalse(53 in bob.components)
         bob.register_process(53, component)
-        self.assertEqual([[53, 'test', 'Test']], bob.get_processes())
+        self.assertTrue(53 in bob.components)
 
     def test_start_simple(self):
         '''Test simple process startup.'''
         bob = BoB()
-        bob.verbose = False
         bob.c_channel_env = {}
+
+        # non-verbose case
+        bob.verbose = False
         pi = bob.start_simple('/bin/true')
         self.assertEqual('/bin/true', pi.name)
         self.assertEqual(['/bin/true'], pi.args)
         self.assertEqual({}, pi.env)
         self.assertNotEqual(0, pi.pid)
 
+        # verbose case
+        bob.verbose = True
+        pi = bob.start_simple('/bin/true')
+        self.assertEqual('/bin/true', pi.name)
+        self.assertEqual(['/bin/true', '-v'], pi.args)
+        self.assertEqual({}, pi.env)
+        self.assertNotEqual(0, pi.pid)
+
     # there is another test with this name (minus the digits), but both
     # do different things.
     def test_start_auth2(self):



More information about the bind10-changes mailing list