[
https://issues.apache.org/jira/browse/DISPATCH-2266?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17444408#comment-17444408
]
ASF GitHub Bot commented on DISPATCH-2266:
------------------------------------------
jiridanek commented on a change in pull request #1437:
URL: https://github.com/apache/qpid-dispatch/pull/1437#discussion_r750093572
##########
File path: tests/system_tests_edge_router.py
##########
@@ -1895,6 +1895,10 @@ def test_51_link_route_proxy_configured(self):
if self.skip['test_51'] :
self.skipTest("Test skipped during development.")
+ # The previous test waited for address CfgLinkRoute1 on router INT_B
+ # We will wait for address CfgLinkRoute1 to be unsubscribed
+ self.INT_B.wait_address_unsubscribed("CfgLinkRoute1")
Review comment:
Depending on the specific order in which the tests are run sounds
dangerous. I think that this line should be part of the previous test. It is
probably sufficient to put it at the end. That way it would be skipped if the
test failed before it got to it, but that should be rare,...
There is currently not a good way to have a per-test tear-down in
system-tests. One could be developed... Maybe a decorator on the test method?
Or go all-in with pytest and use fixtures, which do have their teardowns? The
least disruptive thing would be to have a list in the test class,
`self.addresses_requiring_unsubscribe`, which would be cleaned in the shared
tear_down code. Tests would populate it with addresses that require this
treatment.
--
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]
> system_tests_edge_router stalls on Travis CI
> --------------------------------------------
>
> Key: DISPATCH-2266
> URL: https://issues.apache.org/jira/browse/DISPATCH-2266
> Project: Qpid Dispatch
> Issue Type: Test
> Components: Tests
> Affects Versions: 1.17.0
> Reporter: Ganesh Murthy
> Assignee: Ganesh Murthy
> Priority: Major
>
> The system_tests_edge_router stalls on Travis CI intermittently
> {noformat}
> 55: test_50_link_topology (system_tests_edge_router.LinkRouteProxyTest) ... ok
> No output has been received in the last 10m0s, this potentially indicates a
> stalled build or something wrong with the build itself.
> Check the details on how to adjust your build configuration on:
> https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received
> The build has been terminated
> {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
