kgiusti commented on a change in pull request #1437:
URL: https://github.com/apache/qpid-dispatch/pull/1437#discussion_r750544421
##########
File path: tests/system_tests_edge_router.py
##########
@@ -1887,6 +1887,8 @@ def test_50_link_topology(self):
bc_a.close()
bc_b.close()
+ self.INT_B.wait_address_unsubscribed("CfgLinkRoute1")
Review comment:
It would be better to do this check in
test_51_link_route_proxy_configured to keep it all in one place and prevent
accidental breakage if - for whatever reason - tests need to be re-ordered.
Eventually all tests that enable a link route(s) should perform that type of
check at the start of the test.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
