From 6b59f96ae6feb32c890643bc289a010ab0fe08ce Mon Sep 17 00:00:00 2001 From: "kotler.linden" Date: Wed, 30 Sep 2009 00:31:53 +0000 Subject: [PATCH] minor bug fix found through the power of unittests :) --- pyogp/lib/base/message_manager.py | 4 ++-- pyogp/lib/base/tests/test_message_manager.py | 10 +++++++--- 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/pyogp/lib/base/message_manager.py b/pyogp/lib/base/message_manager.py index 67149a1..98e975d 100644 --- a/pyogp/lib/base/message_manager.py +++ b/pyogp/lib/base/message_manager.py @@ -57,7 +57,7 @@ class MessageManager(object): self.message_handler = MessageHandler() logger.debug("Initializing the Message Manager ") - + self.host = Host((region.sim_ip, region.sim_port)) # initialize the manager's base attributes #self.builder = MessageBuilder() # @@ -104,7 +104,7 @@ class MessageManager(object): def stop_monitors(self): """ stops monitoring coroutines """ #stops udp_dispatcher - self._running = False + self._is_running = False #stops event_queue if self.event_queue._running: self.event_queue.stop() diff --git a/pyogp/lib/base/tests/test_message_manager.py b/pyogp/lib/base/tests/test_message_manager.py index 369d12d..19537db 100644 --- a/pyogp/lib/base/tests/test_message_manager.py +++ b/pyogp/lib/base/tests/test_message_manager.py @@ -40,12 +40,16 @@ class TestMessageManager(unittest.TestCase): def tearDown(self): pass - def test_start_monitors(self): + def test_start_stop_monitors(self): self.message_manager.start_monitors() + api.sleep(0) self.assertTrue(self.message_manager._is_running) - api.sleep(.1) self.assertTrue(self.message_manager.event_queue._running) - + self.message_manager.stop_monitors() + api.sleep(0) + self.assertFalse(self.message_manager._is_running) + #self.assertFalse(self.message_manager.event_queue._running) + def test_enqueue_message(self): pass