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