diff options
author | Jason McDonald <jason.mcdonald@nokia.com> | 2011-04-12 14:53:36 +1000 |
---|---|---|
committer | Jason McDonald <jason.mcdonald@nokia.com> | 2011-04-12 15:38:20 +1000 |
commit | 3475168550c1a804f04f2a4edfeb30c04cd36551 (patch) | |
tree | ec1587bdc87e6f554c31e09cb1a3a2e26f16b181 /tests/auto/qprocess | |
parent | 78722d6b6e9ba0f6de1a4eba232aad2f96a339f2 (diff) |
Reduce usage of Q_ASSERT in autotests.
Using Q_ASSERT does nothing in release-mode builds, and in debug builds
it causes tests to terminate prematurely. It is much better to use
QVERIFY or QCOMPARE.
Task-number: QTBUG-17582
Reviewed-by: Rohan McGovern
Diffstat (limited to 'tests/auto/qprocess')
-rw-r--r-- | tests/auto/qprocess/tst_qprocess.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/auto/qprocess/tst_qprocess.cpp b/tests/auto/qprocess/tst_qprocess.cpp index 2a8874eab3..3e0c3ff412 100644 --- a/tests/auto/qprocess/tst_qprocess.cpp +++ b/tests/auto/qprocess/tst_qprocess.cpp @@ -668,7 +668,7 @@ void tst_QProcess::exitStatus() QSKIP("This test opens a crash dialog on Windows", SkipSingle); #endif - Q_ASSERT(processList.count() == exitStatus.count()); + QCOMPARE(exitStatus.count(), processList.count()); for (int i = 0; i < processList.count(); ++i) { process->start(processList.at(i)); QVERIFY(process->waitForStarted(5000)); |