diff options
author | Jason McDonald <jason.mcdonald@nokia.com> | 2011-10-18 12:50:36 +1000 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2011-10-19 07:24:13 +0200 |
commit | 8f02339e76b9b21df028e29735475313760f82c6 (patch) | |
tree | da655389e77d590196152b3ec230bc67348da880 /tests/auto/corelib/thread | |
parent | ae1810658b75dd0eb0247b0e951ce315c8ce4c86 (diff) |
Remove DEPENDS_ON from qtestlib API.
The DEPENDS_ON macro didn't do anything and has misled many users to
think that they can write test functions that depend on other test
functions.
Task-number: QTBUG-21851
Change-Id: Ibe65b2d5d88bb81b6a0ebbe0b220f7d409a1446c
Reviewed-by: Rohan McGovern <rohan.mcgovern@nokia.com>
Diffstat (limited to 'tests/auto/corelib/thread')
-rw-r--r-- | tests/auto/corelib/thread/qsemaphore/tst_qsemaphore.cpp | 8 | ||||
-rw-r--r-- | tests/auto/corelib/thread/qthread/tst_qthread.cpp | 28 |
2 files changed, 0 insertions, 36 deletions
diff --git a/tests/auto/corelib/thread/qsemaphore/tst_qsemaphore.cpp b/tests/auto/corelib/thread/qsemaphore/tst_qsemaphore.cpp index a2c6bf16b7..7c410db148 100644 --- a/tests/auto/corelib/thread/qsemaphore/tst_qsemaphore.cpp +++ b/tests/auto/corelib/thread/qsemaphore/tst_qsemaphore.cpp @@ -64,8 +64,6 @@ private slots: void tryAcquireWithTimeout_data(); void tryAcquireWithTimeout(); void tryAcquireWithTimeoutStarvation(); - void release(); - void available(); void producerConsumer(); }; @@ -363,12 +361,6 @@ void tst_QSemaphore::tryAcquireWithTimeoutStarvation() QVERIFY(consumer.wait()); } -void tst_QSemaphore::release() -{ DEPENDS_ON("acquire"); } - -void tst_QSemaphore::available() -{ DEPENDS_ON("acquire"); } - const char alphabet[] = "ACGTH"; const int AlphabetSize = sizeof(alphabet) - 1; diff --git a/tests/auto/corelib/thread/qthread/tst_qthread.cpp b/tests/auto/corelib/thread/qthread/tst_qthread.cpp index b2d09f2baf..f832995e75 100644 --- a/tests/auto/corelib/thread/qthread/tst_qthread.cpp +++ b/tests/auto/corelib/thread/qthread/tst_qthread.cpp @@ -79,20 +79,15 @@ private slots: void isFinished(); void isRunning(); void setPriority(); - void priority(); void setStackSize(); - void stackSize(); void exit(); void start(); void terminate(); void quit(); - void wait(); void started(); void finished(); void terminated(); - void run(); void exec(); - void setTerminationEnabled(); void sleep(); void msleep(); void usleep(); @@ -415,9 +410,6 @@ void tst_QThread::setPriority() QCOMPARE(thread.priority(), QThread::InheritPriority); } -void tst_QThread::priority() -{ DEPENDS_ON("setPriority"); } - void tst_QThread::setStackSize() { Simple_Thread thread; @@ -428,11 +420,6 @@ void tst_QThread::setStackSize() QCOMPARE(thread.stackSize(), 0u); } -void tst_QThread::stackSize() -{ - DEPENDS_ON("setStackSize"); -} - void tst_QThread::exit() { Exit_Thread thread; @@ -537,12 +524,6 @@ void tst_QThread::quit() QCOMPARE(thread2.result, 0); } -void tst_QThread::wait() -{ - DEPENDS_ON("isRunning"); - DEPENDS_ON("isFinished"); -} - void tst_QThread::started() { SignalRecorder recorder; @@ -579,14 +560,8 @@ void tst_QThread::terminated() QVERIFY(recorder.wasActivated()); } -void tst_QThread::run() -{ DEPENDS_ON("wait()"); } - void tst_QThread::exec() { - DEPENDS_ON("exit()"); - DEPENDS_ON("quit()"); - class MultipleExecThread : public QThread { public: @@ -621,9 +596,6 @@ void tst_QThread::exec() QCOMPARE(thread.res2, 2); } -void tst_QThread::setTerminationEnabled() -{ DEPENDS_ON("terminate"); } - void tst_QThread::sleep() { Sleep_Thread thread; |