From b750a3786f42395f0cddd8ed8f92a2bcfda1b270 Mon Sep 17 00:00:00 2001 From: Allan Sandfeld Jensen Date: Fri, 9 Dec 2016 11:23:04 +0100 Subject: [PATCH] Lower timeout on tst_QSemaphore::tryAcquireWithTimeout The test was so slow it was blacklisted on all platforms for timing out. This patch lowers the timeout to a 5th and tries removing the blacklist. Change-Id: Ib28b21de572517c548a14300f26815598efe91e2 Reviewed-by: Thiago Macieira --- tests/auto/corelib/thread/qsemaphore/BLACKLIST | 3 --- tests/auto/corelib/thread/qsemaphore/tst_qsemaphore.cpp | 6 +++--- 2 files changed, 3 insertions(+), 6 deletions(-) delete mode 100644 tests/auto/corelib/thread/qsemaphore/BLACKLIST diff --git a/tests/auto/corelib/thread/qsemaphore/BLACKLIST b/tests/auto/corelib/thread/qsemaphore/BLACKLIST deleted file mode 100644 index 9f6f6e3ba6..0000000000 --- a/tests/auto/corelib/thread/qsemaphore/BLACKLIST +++ /dev/null @@ -1,3 +0,0 @@ -# Times out randomly on linux, windows, osx -[tryAcquireWithTimeout] -* diff --git a/tests/auto/corelib/thread/qsemaphore/tst_qsemaphore.cpp b/tests/auto/corelib/thread/qsemaphore/tst_qsemaphore.cpp index 8597bf1a6d..f1eb32a282 100644 --- a/tests/auto/corelib/thread/qsemaphore/tst_qsemaphore.cpp +++ b/tests/auto/corelib/thread/qsemaphore/tst_qsemaphore.cpp @@ -202,8 +202,8 @@ void tst_QSemaphore::tryAcquireWithTimeout_data() { QTest::addColumn("timeout"); - QTest::newRow("1s") << 1000; - QTest::newRow("10s") << 10000; + QTest::newRow("0.2s") << 200; + QTest::newRow("2s") << 2000; } void tst_QSemaphore::tryAcquireWithTimeout() @@ -212,7 +212,7 @@ void tst_QSemaphore::tryAcquireWithTimeout() // timers are not guaranteed to be accurate down to the last millisecond, // so we permit the elapsed times to be up to this far from the expected value. - int fuzz = 50; + int fuzz = 50 + (timeout / 20); QSemaphore semaphore; QElapsedTimer time;