commit 907efa24fe59643a275d1fc56ebb5689c7888b95
author Dan Talayco <dan.talayco@bigswitch.com> Wed Sep 19 11:30:06 2012 -0700
committer Dan Talayco <dan.talayco@bigswitch.com> Wed Sep 19 11:30:06 2012 -0700
tree d06fdd9c5e044cab5e5f803e6c9a4b0864092fac
parent 5d0f7cc5afdc3f1a83ba30b8968522060dd1484f

Mainly: complete handshake for ctrl cxn

tests/serial_failover.py

diff --git a/tests/serial_failover.py b/tests/serial_failover.py
index 90a4236..8ae5a46 100644
--- a/tests/serial_failover.py
+++ b/tests/serial_failover.py
@@ -80,11 +80,15 @@
         self.controller.start()
         #@todo Add an option to wait for a pkt transaction to ensure version
         # compatibilty?
-        self.controller.connect()
+        self.controller.connect(timeout=10)
         self.assertTrue(self.controller.active,
                         "Controller startup failed, not active")
         self.assertTrue(self.controller.switch_addr is not None,
                         "Controller startup failed, no switch addr")
+        request = message.features_request()
+        reply, pkt = self.controller.transact(request, timeout=10)
+        self.assertTrue(reply is not None,
+                        "Did not complete features_request for handshake")
         serial_failover_logger.info("Connected " + 
                                     str(self.controller.switch_addr))
 
@@ -117,7 +121,7 @@
         # controller_list is list of ip/port tuples
         partial_list = test_param_get(serial_failover_config,
                                       'controller_list')
-        serial_failover_logger.debug(str(partial_list))
+        serial_failover_logger.debug("ctrl list: " + str(partial_list))
         self.controller_list = [(serial_failover_config["controller_host"],
                                  serial_failover_config["controller_port"])]
         if partial_list is not None: