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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to