This is an automated email from the ASF dual-hosted git repository.

gmurthy pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git


The following commit(s) were added to refs/heads/main by this push:
     new 9671cd4  DISPATCH-2113: Removed all relative path references in system 
tests. This closes #1210
9671cd4 is described below

commit 9671cd4e8dd3a7a974e75694c80364e1932380ad
Author: Ganesh Murthy <gmur...@apache.org>
AuthorDate: Tue May 11 12:39:59 2021 -0400

    DISPATCH-2113: Removed all relative path references in system tests. This 
closes #1210
---
 tests/system_tests_bad_configuration.py        |  8 +++--
 tests/system_tests_policy.py                   |  2 +-
 tests/system_tests_policy_oversize_compound.py | 38 ++++++++++++------------
 tests/system_tests_protocol_settings.py        | 27 +++++++----------
 tests/system_tests_tcp_adaptor.py              | 24 +++++++++------
 tests/system_tests_topology_disposition.py.in  | 41 +++++++++++++-------------
 6 files changed, 71 insertions(+), 69 deletions(-)

diff --git a/tests/system_tests_bad_configuration.py 
b/tests/system_tests_bad_configuration.py
index 85bdfdd..ba06922 100644
--- a/tests/system_tests_bad_configuration.py
+++ b/tests/system_tests_bad_configuration.py
@@ -109,7 +109,7 @@ class RouterTestBadConfiguration(TestCase):
         then it stops scheduling new attempts.
         :return:
         """
-        with open('../setUpClass/test-router.log', 'r') as router_log:
+        with open(self.router.logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
             expected_log_snip = "Connection to %s" % 
self.unresolvable_host_name
             errors_caught = [line for line in log_lines if expected_log_snip 
in line and "failed" in line]
@@ -180,7 +180,8 @@ class RouterTestIdFailCtrlChar(TestCase):
         """
         Writes illegal config, runs router, examines console output
         """
-        conf_path = "../setUpClass/test-router-ctrl-char.conf"
+        parent_path = os.path.dirname(os.getcwd())
+        conf_path = os.path.join(parent_path, 
"setUpClass/test-router-ctrl-char.conf")
         with open(conf_path, 'w') as router_conf:
             router_conf.write("router { \n")
             router_conf.write("    id: abc\\bdef \n")
@@ -221,7 +222,8 @@ class RouterTestIdFailWhiteSpace(TestCase):
         """
         Writes illegal config, runs router, examines console output
         """
-        conf_path = "../setUpClass/test-router-whitespace.conf"
+        parent_path = os.path.dirname(os.getcwd())
+        conf_path = os.path.join(parent_path, 
"setUpClass/test-router-whitespace.conf")
         with open(conf_path, 'w') as router_conf:
             router_conf.write("router { \n")
             router_conf.write("    id: abc def \n")
diff --git a/tests/system_tests_policy.py b/tests/system_tests_policy.py
index d4e2afa..0db1ba5 100644
--- a/tests/system_tests_policy.py
+++ b/tests/system_tests_policy.py
@@ -341,7 +341,7 @@ class SenderReceiverLimits(TestCase):
         # To accomodate the file lag this test may retry reading the log file.
         verified = False
         for tries in range(5):
-            with open('../setUpClass/SenderReceiverLimits.log', 'r') as 
router_log:
+            with open(self.router.logfile_path, 'r') as router_log:
                 log_lines = router_log.read().split("\n")
                 close_lines = [s for s in log_lines if "senders_denied=1, 
receivers_denied=1" in s]
                 verified = len(close_lines) == 1
diff --git a/tests/system_tests_policy_oversize_compound.py 
b/tests/system_tests_policy_oversize_compound.py
index 1ed68f1..9ef74c8 100644
--- a/tests/system_tests_policy_oversize_compound.py
+++ b/tests/system_tests_policy_oversize_compound.py
@@ -762,13 +762,13 @@ class MaxMessageSizeBlockOversize(TestCase):
             raise Exception("%s\n%s" % (e, out))
         return out
 
-    def sense_n_closed_lines(self, routername, 
pattern=OVERSIZE_CONDITION_NAME):
+    def sense_n_closed_lines(self, router, pattern=OVERSIZE_CONDITION_NAME):
         """
         Read a router's log file and count how many size-exceeded lines are in 
it.
         :param routername:
         :return: (int, int) tuple with counts of lines in and lines out
         """
-        with open("../setUpClass/%s.log" % routername, 'r') as router_log:
+        with open(router.logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
         i_closed_lines = [s for s in log_lines if pattern in s and "<-" in s]
         o_closed_lines = [s for s in log_lines if pattern in s and "->" in s]
@@ -789,7 +789,7 @@ class MaxMessageSizeBlockOversize(TestCase):
     #
 
     def test_60_block_oversize_EB1_INTB_at_INTB(self):
-        ibefore, obefore = self.sense_n_closed_lines("EB1")
+        ibefore, obefore = self.sense_n_closed_lines(self.EB1)
         test = OversizeMessageTransferTest(MaxMessageSizeBlockOversize,
                                            MaxMessageSizeBlockOversize.EB1,
                                            MaxMessageSizeBlockOversize.INT_B,
@@ -803,7 +803,7 @@ class MaxMessageSizeBlockOversize(TestCase):
         self.assertTrue(test.error is None)
 
         # Verify that interrouter link was shut down
-        iafter, oafter = self.sense_n_closed_lines("EB1")
+        iafter, oafter = self.sense_n_closed_lines(self.EB1)
         idelta = iafter - ibefore
         odelta = oafter - obefore
         success = odelta == 0 and idelta == 1
@@ -814,7 +814,7 @@ class MaxMessageSizeBlockOversize(TestCase):
             self.assertTrue(success, "Expected router to generate close with 
condition: message size exceeded")
 
         # Verfiy that a link was closed with the expected pattern(s)
-        ilink1, olink1 = self.sense_n_closed_lines("EB1", 
pattern=OVERSIZE_LINK_CONDITION_NAME)
+        ilink1, olink1 = self.sense_n_closed_lines(self.EB1, 
pattern=OVERSIZE_LINK_CONDITION_NAME)
         success = olink1 > 0
         if (not success):
             test.logger.log("FAIL: Did not see link close in log file. 
oBefore: %d, oAfter: %d, iBefore:%d, iAfter:%d" %
@@ -837,7 +837,7 @@ class MaxMessageSizeBlockOversize(TestCase):
     #
 
     def test_61_block_oversize_EB1_EA1_at_INTB(self):
-        ibefore, obefore = self.sense_n_closed_lines("EB1")
+        ibefore, obefore = self.sense_n_closed_lines(self.EB1)
         test = OversizeMessageTransferTest(MaxMessageSizeBlockOversize,
                                            MaxMessageSizeBlockOversize.EB1,
                                            MaxMessageSizeBlockOversize.EA1,
@@ -851,7 +851,7 @@ class MaxMessageSizeBlockOversize(TestCase):
         self.assertTrue(test.error is None)
 
         # Verify that interrouter link was shut down
-        iafter, oafter = self.sense_n_closed_lines("EB1")
+        iafter, oafter = self.sense_n_closed_lines(self.EB1)
         idelta = iafter - ibefore
         odelta = oafter - obefore
         success = odelta == 0 and idelta == 1
@@ -876,7 +876,7 @@ class MaxMessageSizeBlockOversize(TestCase):
     #
 
     def test_70_block_oversize_EB1_INTB_at_both(self):
-        ibefore, obefore = self.sense_n_closed_lines("EB1")
+        ibefore, obefore = self.sense_n_closed_lines(self.EB1)
         test = OversizeMessageTransferTest(MaxMessageSizeBlockOversize,
                                            MaxMessageSizeBlockOversize.EB1,
                                            MaxMessageSizeBlockOversize.INT_B,
@@ -896,7 +896,7 @@ class MaxMessageSizeBlockOversize(TestCase):
         # oversize condition before it has forwarded too many bytes of the 
first message
         # to INT.B. Then EB1 aborts the first message to INT.B and INT.B never
         # detects an oversize condition.
-        iafter, oafter = self.sense_n_closed_lines("EB1")
+        iafter, oafter = self.sense_n_closed_lines(self.EB1)
         idelta = iafter - ibefore
         odelta = oafter - obefore
         success = odelta == 1 and (idelta == 0 or idelta == 1)
@@ -927,7 +927,7 @@ class MaxMessageSizeBlockOversize(TestCase):
     #
 
     def test_80_block_multicast_EB1_INTB_at_INTB(self):
-        ibefore, obefore = self.sense_n_closed_lines("EB1")
+        ibefore, obefore = self.sense_n_closed_lines(self.EB1)
         count = 10
         test = OversizeMulticastTransferTest(MaxMessageSizeBlockOversize,
                                              MaxMessageSizeBlockOversize.EB1,
@@ -946,7 +946,7 @@ class MaxMessageSizeBlockOversize(TestCase):
         self.assertTrue(test.error is None)
 
         # Verify that interrouter link was shut down
-        iafter, oafter = self.sense_n_closed_lines("EB1")
+        iafter, oafter = self.sense_n_closed_lines(self.EB1)
         idelta = iafter - ibefore
         odelta = oafter - obefore
         success = odelta == 0 and idelta == 1
@@ -977,7 +977,7 @@ class MaxMessageSizeBlockOversize(TestCase):
     #
 
     def test_81_block_multicast_EA1(self):
-        ibefore, obefore = self.sense_n_closed_lines("EA1")
+        ibefore, obefore = self.sense_n_closed_lines(self.EA1)
         count = 10
         test = OversizeMulticastTransferTest(MaxMessageSizeBlockOversize,
                                              MaxMessageSizeBlockOversize.EA1,
@@ -996,7 +996,7 @@ class MaxMessageSizeBlockOversize(TestCase):
         self.assertTrue(test.error is None)
 
         # Verify that interrouter link was shut down
-        iafter, oafter = self.sense_n_closed_lines("EA1")
+        iafter, oafter = self.sense_n_closed_lines(self.EA1)
         idelta = iafter - ibefore
         odelta = oafter - obefore
         success = odelta == 1 and idelta == 0
@@ -1027,7 +1027,7 @@ class MaxMessageSizeBlockOversize(TestCase):
     #
 
     def test_82_block_multicast_INTA(self):
-        ibefore, obefore = self.sense_n_closed_lines("INT.A")
+        ibefore, obefore = self.sense_n_closed_lines(self.INT_A)
         count = 10
         test = OversizeMulticastTransferTest(MaxMessageSizeBlockOversize,
                                              MaxMessageSizeBlockOversize.INT_A,
@@ -1046,7 +1046,7 @@ class MaxMessageSizeBlockOversize(TestCase):
         self.assertTrue(test.error is None)
 
         # Verify that interrouter link was shut down
-        iafter, oafter = self.sense_n_closed_lines("INT.A")
+        iafter, oafter = self.sense_n_closed_lines(self.INT_A)
         idelta = iafter - ibefore
         odelta = oafter - obefore
         success = odelta == 1 and idelta == 0
@@ -1213,13 +1213,13 @@ class MaxMessageSizeLinkRouteOversize(TestCase):
         cls.EA1.wait_connectors()
         cls.EB1.wait_connectors()
 
-    def sense_n_closed_lines(self, routername):
+    def sense_n_closed_lines(self, router):
         """
         Read a router's log file and count how many size-exceeded lines are in 
it.
         :param routername:
         :return: (int, int) tuple with counts of lines in and lines out
         """
-        with open("../setUpClass/%s.log" % routername, 'r') as router_log:
+        with open(router.logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
         i_closed_lines = [s for s in log_lines if OVERSIZE_CONDITION_NAME in s 
and "<-" in s]
         o_closed_lines = [s for s in log_lines if OVERSIZE_CONDITION_NAME in s 
and "->" in s]
@@ -1240,7 +1240,7 @@ class MaxMessageSizeLinkRouteOversize(TestCase):
     #
 
     def test_90_block_link_route_EB1_INTB(self):
-        ibefore, obefore = self.sense_n_closed_lines("EB1")
+        ibefore, obefore = self.sense_n_closed_lines(self.EB1)
         test = OversizeMessageTransferTest(MaxMessageSizeLinkRouteOversize,
                                            MaxMessageSizeLinkRouteOversize.EB1,
                                            None,
@@ -1255,7 +1255,7 @@ class MaxMessageSizeLinkRouteOversize(TestCase):
         self.assertTrue(test.error is None)
 
         # Verify that interrouter link was shut down
-        iafter, oafter = self.sense_n_closed_lines("EB1")
+        iafter, oafter = self.sense_n_closed_lines(self.EB1)
         idelta = iafter - ibefore
         odelta = oafter - obefore
         success = odelta == 1 and (idelta == 0 or idelta == 1)
diff --git a/tests/system_tests_protocol_settings.py 
b/tests/system_tests_protocol_settings.py
index 8466d52..c3f1010 100644
--- a/tests/system_tests_protocol_settings.py
+++ b/tests/system_tests_protocol_settings.py
@@ -23,6 +23,7 @@ from __future__ import absolute_import
 from __future__ import print_function
 
 import sys
+import os
 from system_test import TestCase, Qdrouterd, main_module
 from system_test import unittest
 from proton.utils import BlockingConnection
@@ -50,7 +51,7 @@ class MaxFrameMaxSessionFramesTest(TestCase):
         bc.create_receiver("xxx")
         bc.close()
 
-        with open('../setUpClass/MaxFrameMaxSessionFrames.log', 'r') as 
router_log:
+        with open(self.router.logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
             open_lines = [s for s in log_lines if "-> @open" in s]
             # max-frame is from the config
@@ -83,8 +84,7 @@ class MaxSessionsTest(TestCase):
         bc = BlockingConnection(self.router.addresses[0])
         bc.create_receiver("xxx")
         bc.close()
-
-        with open('../setUpClass/MaxSessions.log', 'r') as router_log:
+        with open(self.router.logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
             open_lines = [s for s in log_lines if "-> @open" in s]
             # channel-max is 9
@@ -112,8 +112,7 @@ class MaxSessionsZeroTest(TestCase):
         bc = BlockingConnection(self.router.addresses[0])
         bc.create_receiver("xxx")
         bc.close()
-
-        with open('../setUpClass/MaxSessionsZero.log', 'r') as router_log:
+        with open(self.router.logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
             open_lines = [s for s in log_lines if "-> @open" in s]
             # channel-max is 0. Should get proton default 32767
@@ -141,8 +140,7 @@ class MaxSessionsLargeTest(TestCase):
         bc = BlockingConnection(self.router.addresses[0])
         bc.create_receiver("xxx")
         bc.close()
-
-        with open('../setUpClass/MaxSessionsLarge.log', 'r') as router_log:
+        with open(self.router.logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
             open_lines = [s for s in log_lines if "-> @open" in s]
             # channel-max is 0. Should get proton default 32767
@@ -170,8 +168,7 @@ class MaxFrameSmallTest(TestCase):
         bc = BlockingConnection(self.router.addresses[0])
         bc.create_receiver("xxx")
         bc.close()
-
-        with open('../setUpClass/MaxFrameSmall.log', 'r') as router_log:
+        with open(self.router.logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
             open_lines = [s for s in log_lines if "-> @open" in s]
             # if frame size <= 512 proton set min of 512
@@ -199,8 +196,7 @@ class MaxFrameDefaultTest(TestCase):
         bc = BlockingConnection(self.router.addresses[0])
         bc.create_receiver("xxx")
         bc.close()
-
-        with open('../setUpClass/MaxFrameDefault.log', 'r') as router_log:
+        with open(self.router.logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
             open_lines = [s for s in log_lines if "-> @open" in s]
             # if frame size not set then a default is used
@@ -228,8 +224,7 @@ class MaxSessionFramesDefaultTest(TestCase):
         bc = BlockingConnection(self.router.addresses[0])
         bc.create_receiver("xxx")
         bc.close()
-
-        with open('../setUpClass/MaxSessionFramesDefault.log', 'r') as 
router_log:
+        with open(self.router.logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
             open_lines = [s for s in log_lines if "-> @open" in s]
             # if frame size not set then a default is used
@@ -270,7 +265,7 @@ class MaxFrameMaxSessionFramesZeroTest(TestCase):
         bc.create_receiver("xxx")
         bc.close()
 
-        with open('../setUpClass/MaxFrameMaxSessionFramesZero.log', 'r') as 
router_log:
+        with open(self.router.logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
             open_lines = [s for s in log_lines if "-> @open" in s]
             # max-frame gets set to protocol min
@@ -326,7 +321,7 @@ class ConnectorSettingsDefaultTest(TestCase):
         cls.routers[1].wait_router_connected('QDR.A')
 
     def test_connector_default(self):
-        with open('../setUpClass/A.log', 'r') as router_log:
+        with open(self.routers[0].logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
             open_lines = [s for s in log_lines if "<- @open" in s]
             # defaults
@@ -384,7 +379,7 @@ class ConnectorSettingsNondefaultTest(TestCase):
         cls.routers[1].wait_router_connected('QDR.A')
 
     def test_connector_default(self):
-        with open('../setUpClass/A.log', 'r') as router_log:
+        with open(self.routers[0].logfile_path, 'r') as router_log:
             log_lines = router_log.read().split("\n")
             open_lines = [s for s in log_lines if "<- @open" in s]
             # nondefaults
diff --git a/tests/system_tests_tcp_adaptor.py 
b/tests/system_tests_tcp_adaptor.py
index b2f1aad..f60de91 100644
--- a/tests/system_tests_tcp_adaptor.py
+++ b/tests/system_tests_tcp_adaptor.py
@@ -137,10 +137,12 @@ class EchoClientRunner():
         self.name = "%s_%s_%s_%s" % \
                     (self.test_name, self.client_n, self.size, self.count)
         self.client_prefix = "ECHO_CLIENT %s" % self.name
+
+        parent_path = os.path.dirname(os.getcwd())
         self.client_logger = Logger(title=self.client_prefix,
                                     print_to_console=self.print_client_logs,
                                     save_for_dump=False,
-                                    
ofilename="../setUpClass/TcpAdaptor_echo_client_%s.log" % self.name)
+                                    ofilename=os.path.join(parent_path, 
"setUpClass/TcpAdaptor_echo_client_%s.log" % self.name))
 
         try:
             self.e_client = TcpEchoClient(prefix=self.client_prefix,
@@ -311,10 +313,11 @@ class TcpAdaptor(TestCase):
         # define logging levels
         cls.print_logs_server = False
         cls.print_logs_client = True
+        parent_path = os.path.dirname(os.getcwd())
         cls.logger = Logger(title="TcpAdaptor-testClass",
                             print_to_console=True,
                             save_for_dump=False,
-                            ofilename='../setUpClass/TcpAdaptor.log')
+                            ofilename=os.path.join(parent_path, 
"setUpClass/TcpAdaptor.log"))
         # Write a dummy log line for scraper.
         cls.logger.log("SERVER (info) Container Name: TCP_TEST")
 
@@ -324,13 +327,14 @@ class TcpAdaptor(TestCase):
 
         # start echo servers immediately after the echo server
         # ports are assigned.
+        parent_path = os.path.dirname(os.getcwd())
         for rtr in cls.router_order:
             test_name = "TcpAdaptor"
             server_prefix = "ECHO_SERVER %s ES_%s" % (test_name, rtr)
             server_logger = Logger(title=test_name,
                                    print_to_console=cls.print_logs_server,
                                    save_for_dump=False,
-                                   
ofilename="../setUpClass/TcpAdaptor_echo_server_%s.log" % rtr)
+                                   ofilename=os.path.join(parent_path, 
"setUpClass/TcpAdaptor_echo_server_%s.log" % rtr))
             cls.logger.log("TCP_TEST Launching echo server '%s'" % 
server_prefix)
             server = TcpEchoServer(prefix=server_prefix,
                                    port=cls.tcp_server_listener_ports[rtr],
@@ -344,7 +348,7 @@ class TcpAdaptor(TestCase):
         server_logger = Logger(title="TcpAdaptor",
                                print_to_console=cls.print_logs_server,
                                save_for_dump=False,
-                               
ofilename="../setUpClass/TcpAdaptor_echo_server_NS_CONN_STALL.log")
+                               ofilename=os.path.join(parent_path, 
"setUpClass/TcpAdaptor_echo_server_NS_CONN_STALL.log"))
         cls.logger.log("TCP_TEST Launching echo server '%s'" % server_prefix)
         server = TcpEchoServer(prefix=server_prefix,
                                port=cls.EC2_conn_stall_connector_port,
@@ -460,13 +464,15 @@ class TcpAdaptor(TestCase):
             cls.logger.log("TCP_TEST %s" % line)
 
         # write to shell script
-        with open("../setUpClass/TcpAdaptor-ports.sh", 'w') as o_file:
+        parent_path = os.path.dirname(os.getcwd())
+        file_name = os.path.join(parent_path, "setUpClass/TcpAdaptor-ports.sh")
+        with open(file_name, 'w') as o_file:
             for line in p_out:
                 o_file.write("set %s\n" % line)
 
         # Write a script to run scraper on this test's log files
         scraper_abspath = os.path.join(os.environ.get('BUILD_DIR'), 'tests', 
'scraper', 'scraper.py')
-        logs_dir     = os.path.abspath("../setUpClass")
+        logs_dir     = os.path.join(parent_path, "setUpClass")
         main_log     = "TcpAdaptor.log"
         echo_logs    = "TcpAdaptor_echo*"
         big_test_log = "TcpAdaptor_all.log"
@@ -474,8 +480,7 @@ class TcpAdaptor(TestCase):
         edge_logs    = "E*.log"
         log_modules_spec = "--log-modules 
TCP_ADAPTOR,TCP_TEST,ECHO_SERVER,ECHO_CLIENT"
         html_output  = "TcpAdaptor.html"
-
-        with open("../setUpClass/TcpAdaptor-run-scraper.sh", 'w') as o_file:
+        with open(os.path.join(parent_path, 
"setUpClass/TcpAdaptor-run-scraper.sh"), 'w') as o_file:
             o_file.write("#!/bin/bash\n\n")
             o_file.write("# Script to run scraper on test class TcpAdaptor 
test result\n")
             o_file.write("# cd into logs directory\n")
@@ -905,10 +910,11 @@ class TcpAdaptorManagementTest(TestCase):
         # Start the echo server. This is the server that the tcpConnector
         # will be connecting to.
         server_prefix = "ECHO_SERVER ES_%s" % cls.test_name
+        parent_path = os.path.dirname(os.getcwd())
         cls.logger = Logger(title="TcpAdaptor",
                             print_to_console=True,
                             save_for_dump=False,
-                            
ofilename="../setUpClass/TcpAdaptor_echo_server.log")
+                            ofilename=os.path.join(parent_path, 
"setUpClass/TcpAdaptor_echo_server.log"))
         cls.echo_server = TcpEchoServer(prefix=server_prefix,
                                         port=cls.tcp_server_port,
                                         logger=cls.logger)
diff --git a/tests/system_tests_topology_disposition.py.in 
b/tests/system_tests_topology_disposition.py.in
index d3b2a09..1e9798d 100644
--- a/tests/system_tests_topology_disposition.py.in
+++ b/tests/system_tests_topology_disposition.py.in
@@ -332,20 +332,20 @@ class TopologyDispositionTests (TestCase):
                ]
                )
 
-        router_A = cls.routers[0]
-        router_B = cls.routers[1]
-        router_C = cls.routers[2]
-        router_D = cls.routers[3]
+        cls.router_A = cls.routers[0]
+        cls.router_B = cls.routers[1]
+        cls.router_C = cls.routers[2]
+        cls.router_D = cls.routers[3]
 
-        router_A.wait_router_connected('B')
-        router_A.wait_router_connected('C')
-        router_A.wait_router_connected('D')
+        cls.router_A.wait_router_connected('B')
+        cls.router_A.wait_router_connected('C')
+        cls.router_A.wait_router_connected('D')
 
         cls.client_addrs = dict()
-        cls.client_addrs['A'] = router_A.addresses[0]
-        cls.client_addrs['B'] = router_B.addresses[0]
-        cls.client_addrs['C'] = router_C.addresses[0]
-        cls.client_addrs['D'] = router_D.addresses[0]
+        cls.client_addrs['A'] = cls.router_A.addresses[0]
+        cls.client_addrs['B'] = cls.router_B.addresses[0]
+        cls.client_addrs['C'] = cls.router_C.addresses[0]
+        cls.client_addrs['D'] = cls.router_D.addresses[0]
 
         # 1 means skip that test.
         cls.skip = {'test_01' : 0,
@@ -384,30 +384,29 @@ class TopologyDispositionTests (TestCase):
             self.skipTest("Test skipped during development.")
         st_key = "PROTOCOL (trace) [C"
         qc_key = "qd.conn-id\"="
-        for letter in ['A', 'B', 'C', 'D']:
-            log_name = ('../setUpClass/%s.log' % letter)
-            with open(log_name, 'r') as router_log:
+        for router in [self.router_A, self.router_B, self.router_C, 
self.router_D]:
+            with open(router.logfile_path, 'r') as router_log:
                 log_lines = router_log.read().split("\n")
                 outbound_opens = [s for s in log_lines if "-> @open" in s]
                 for oopen in outbound_opens:
                     sti = oopen.find(st_key)
                     if sti < 0:
-                        error = "Log %s, line '%s' has no SERVER key" % 
(log_name, oopen)
+                        error = "Log %s, line '%s' has no SERVER key" % 
(router.logfile_path, oopen)
                         break
                     qci = oopen.find(qc_key)
                     if qci < 0:
-                        error = "Log %s, line '%s' has no qd.conn-id key" % 
(log_name, oopen)
+                        error = "Log %s, line '%s' has no qd.conn-id key" % 
(router.logfile_path, oopen)
                         break
                     sti += len(st_key)
                     qci += len(qc_key)
                     while not oopen[sti] == "]":
                         if not oopen[sti] == oopen[qci]:
-                            error = "log %s, line '%s' server conn_id != 
published conn-id" % (log_name, oopen)
+                            error = "log %s, line '%s' server conn_id != 
published conn-id" % (router.logfile_path, oopen)
                             break
                         sti += 1
                         qci += 1
                     if error is None and oopen[qci].isdigit():
-                        error = "log %s, line '%s' published conn-id is too 
big" % (log_name, oopen)
+                        error = "log %s, line '%s' published conn-id is too 
big" % (router.logfile_path, oopen)
                 self.assertIsNone(error)
             self.assertIsNone(error)
 
@@ -421,8 +420,8 @@ class TopologyDispositionTests (TestCase):
 
         # aggregate all the log files
         files = []
-        for letter in ['A', 'B', 'C', 'D']:
-            files.append('../setUpClass/%s.log' % letter)
+        for router in [self.router_A, self.router_B, self.router_C, 
self.router_D]:
+            files.append(router.logfile_path)
         p = self.popen(['/usr/bin/env', '${PY_STRING}', scraper_path, '-f'] + 
files,
                        stdin=PIPE, stdout=PIPE, stderr=STDOUT,
                        universal_newlines=True)
@@ -439,7 +438,7 @@ class TopologyDispositionTests (TestCase):
         self.assertIn('</body>', out)
 
         # split A.log
-        p = self.popen(['/usr/bin/env', '${PY_STRING}', scraper_path, 
'--split', '-f', '../setUpClass/A.log'],
+        p = self.popen(['/usr/bin/env', '${PY_STRING}', scraper_path, 
'--split', '-f', self.router_A.logfile_path],
                        stdin=PIPE, stdout=PIPE, stderr=STDOUT,
                        universal_newlines=True)
         out = p.communicate(None)[0]

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@qpid.apache.org
For additional commands, e-mail: commits-h...@qpid.apache.org

Reply via email to