summaryrefslogtreecommitdiffstats
path: root/tests/auto/qbuffer
diff options
context:
space:
mode:
authorRohan McGovern <rohan.mcgovern@nokia.com>2011-05-18 10:44:47 +1000
committerRohan McGovern <rohan.mcgovern@nokia.com>2011-05-18 10:57:56 +1000
commit6e50394c4eac9b3e91f6df7e944181cf7f54b0fe (patch)
tree5d59d83c15f75dacad3036065603e0dd0a8e7727 /tests/auto/qbuffer
parentb8af05d3e2e7914c4edac5ae1e61f55ab76100bc (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 (cherry picked from commit 3475168550c1a804f04f2a4edfeb30c04cd36551) Change-Id: Ic39972b685ca35a9a71d9c8d03e8dae31481fb19
Diffstat (limited to 'tests/auto/qbuffer')
-rw-r--r--tests/auto/qbuffer/tst_qbuffer.cpp3
1 files changed, 1 insertions, 2 deletions
diff --git a/tests/auto/qbuffer/tst_qbuffer.cpp b/tests/auto/qbuffer/tst_qbuffer.cpp
index 3e3cc739bf..5d6b84f827 100644
--- a/tests/auto/qbuffer/tst_qbuffer.cpp
+++ b/tests/auto/qbuffer/tst_qbuffer.cpp
@@ -309,8 +309,7 @@ void tst_QBuffer::seekTest()
// (see Task 184730)
{
char c;
- const int offset = 1;
- Q_ASSERT(offset > 0); // any positive integer will do
+ const int offset = 1; // any positive integer will do
const qint64 pos = buf.size() + offset;
QVERIFY(buf.seek(pos));
QCOMPARE(buf.pos(), pos);