You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "Ken Giusti (Jira)" <ji...@apache.org> on 2019/11/14 14:45:00 UTC
[jira] [Created] (DISPATCH-1484) Remove hand-coded per-test
timeouts, replace with TIMEOUT global value
Ken Giusti created DISPATCH-1484:
------------------------------------
Summary: Remove hand-coded per-test timeouts, replace with TIMEOUT global value
Key: DISPATCH-1484
URL: https://issues.apache.org/jira/browse/DISPATCH-1484
Project: Qpid Dispatch
Issue Type: Test
Components: Tests
Affects Versions: 1.9.0
Reporter: Ken Giusti
Fix For: Backlog
Many tests use short test timeouts hardcoded into each individual test. For example:
self.timer = event.reactor.schedule(10.0, Timeout(self))
where the Timeout() class forces the individual test to stop with a timeout error.
The problem with this is that when a test is run with run-time checking turned on (ASAN, TSAN, Valgrind...) these short timeouts are too short and we often get test failures simply because the test was running a bit slower.
This work item is to update all such timeouts to use the TIMEOUT global value as provided by the system_test.py module. This TIMEOUT is currently 60 seconds and can be overridden by the env var "QPID_SYSTEM_TEST_TIMEOUT".
Note well that TIMEOUT is NOT the overall ctest timeout (currently 360 seconds, but can be overridden via the --timeout parameter to the ctest command). TIMEOUT only applies to each individual test with a TestCase class, where the overall ctest timeout applies to the full TestCase group of tests.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org