BIND 10 trac2353, updated. 37da25f2eca6b2af80cc5bc2a7e65c5644acf2fb [2353] Just check the output of get_processes() is empty (as a boolean)

[BIND 10 source code commits]

The branch, trac2353 has been updated
       via  37da25f2eca6b2af80cc5bc2a7e65c5644acf2fb (commit)
       via  0f228f2d53a626d647aa728585342c889ff8f846 (commit)
       via  8b31b93c4c62b76b3ec70d0e9c1a18c0f51bd459 (commit)
       via  f9445ffa2d128ab0730abb3592e75475023ba755 (commit)
      from  494bbca16cea584ba6ad5d2c5f9a273d809f902d (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 37da25f2eca6b2af80cc5bc2a7e65c5644acf2fb
Author: Mukund Sivaraman <muks at isc.org>
Date:   Fri Nov 9 11:46:34 2012 +0530

    [2353] Just check the output of get_processes() is empty (as a boolean)

commit 0f228f2d53a626d647aa728585342c889ff8f846
Author: Mukund Sivaraman <muks at isc.org>
Date:   Fri Nov 9 11:44:19 2012 +0530

    [2353] Check that kill_started_components() forcefully kills components

commit 8b31b93c4c62b76b3ec70d0e9c1a18c0f51bd459
Author: Mukund Sivaraman <muks at isc.org>
Date:   Fri Nov 9 11:36:16 2012 +0530

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

commit f9445ffa2d128ab0730abb3592e75475023ba755
Author: Mukund Sivaraman <muks at isc.org>
Date:   Fri Nov 9 11:32:55 2012 +0530

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

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

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

-----------------------------------------------------------------------
diff --git a/src/bin/bind10/tests/bind10_test.py.in b/src/bin/bind10/tests/bind10_test.py.in
index 4e3bd13..9b54371 100644
--- a/src/bin/bind10/tests/bind10_test.py.in
+++ b/src/bin/bind10/tests/bind10_test.py.in
@@ -998,6 +998,7 @@ class MockComponent:
         self.pid = lambda: pid
         self.address = lambda: address
         self.restarted = False
+        self.forceful = False
 
     def get_restart_time(self):
         return 0                # arbitrary dummy value
@@ -1012,8 +1013,8 @@ class MockComponent:
     def failed(self, status):
         return False
 
-    def kill(self, status):
-        return
+    def kill(self, forceful):
+        self.forceful = forceful
 
 class TestBossCmd(unittest.TestCase):
     def test_ping(self):
@@ -1413,7 +1414,6 @@ class TestBossComponents(unittest.TestCase):
     def test_get_processes(self):
         '''Test that procsses are returned correctly, sorted by pid.'''
         bob = MockBob()
-        bob.runnable = True
 
         pids = []
         pids.extend(range(0, 20))
@@ -1451,13 +1451,14 @@ class TestBossComponents(unittest.TestCase):
     def test_kill_started_components(self):
         '''Test that started components are killed.'''
         bob = MockBob()
-        bob.runnable = True
+
         component = MockComponent('test', 53, 'Test')
         bob.components[53] = component
 
         self.assertEqual([[53, 'test', 'Test']], bob.get_processes())
         bob.kill_started_components()
-        self.assertEqual([], bob.get_processes())
+        self.assertFalse(bob.get_processes())
+        self.assertTrue(component.forceful)
 
     def test_start_process(self):
         '''Test that processes can be started.'''