summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * Remove Q_ASSERT from QVariant autotestJason McDonald2011-05-181-6/+9
| | | | | | | | | | | | | | | | | | | | | | I missed one Q_ASSERT in the previous commit. Also changing to use Q_FUNC_INFO instead of hard-coded function names in the warning messages. Change-Id: I0ff5b2b2cda02597836beb5d2811fa8dd2a344ab Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 74a4aad171b39018d596b99684286b9426a091a5)
| * Remove Q_ASSERT's from QTreeView autotestJason McDonald2011-05-181-2/+4
| | | | | | | | | | | | | | | | | | Report fatal errors instead of failing silently in non-debug builds. Change-Id: Ieaff30b71dba2a385a5fffc93d2a8c0f5864aa18 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit e12b912de89088a307c3519a01198a5314b8dd1a)
| * Remove Q_ASSERT from qtesselator autotestJason McDonald2011-05-181-1/+2
| | | | | | | | | | | | | | | | | | Report a fatal error rather than failing silently in non-debug builds. Change-Id: I625c5aa6f86a5764cd8f078baa074d6475a67736 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit c0c7e04c9248ce38278aceefeb527f29149cfb4e)
| * Remove unused function from qtessellator autotestJason McDonald2011-05-181-13/+0
| | | | | | | | | | | | | | Change-Id: I24c1db8392370d19970421a014f7bb6c4bc7c6e8 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 224050aae75fd5b1d6c57909d894a3eba7ac43fc)
| * Remove Q_ASSERT's from qreadwritelock autotestJason McDonald2011-05-181-8/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | The tryWriteLock testfunction didn't do anything useful in non-debug builds, due to the thread having the important code inside Q_ASSERT's, which are no-ops in non-debug builds. This commit removes the Q_ASSERT's, counts the number of failures in the thread and fails the test if there are any failures recorded. Change-Id: I4750f66eeba22ab51ba348ebc06704052421f1ae Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 00f724c943b83f10f9ca9475570708536947538e)
| * Remove Q_ASSERT's from QObject autotestJason McDonald2011-05-181-3/+6
| | | | | | | | | | | | | | | | | | | | | | Rather than aborting in debug builds and failing silently in release builds, report a fatal error in all builds if the helper class methods are called more than once. Change-Id: Id1fda8dc7caf008bbc8f6ae6384abee1aa8f50eb Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 0c157af3e5aa7c56c53b9eabe1510cc639610969)
| * Remove Q_ASSERT's from QObject autotestJason McDonald2011-05-181-3/+6
| | | | | | | | | | | | | | | | | | | | Rather than aborting in debug builds and failing silently in release builds, report a meaningful fatal error in all builds. Change-Id: I3b874f187d482e9785a839cab1a3855b631404c3 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 844906474dd6905d36003143a03a2fbccf0ad0dd)
| * Remove Q_ASSERT's from QNetworkReply autotestJason McDonald2011-05-181-2/+7
| | | | | | | | | | | | | | | | | | | | Rather than aborting in debug builds and ignoring the failure in release builds, report a fatal error in all builds. Change-Id: I1fb8e692c65cf23cf5d2453173db742260ddca48 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 136f39bbb63706819eb18d000f305edbe1cc6a81)
| * Remove Q_ASSERT from qitemmodel autotestJason McDonald2011-05-181-7/+22
| | | | | | | | | | | | | | | | | | | | Don't perform actions needed for the test inside Q_ASSERT, because these actions won't happen in a non-debug build. Change-Id: I0758e9f29f38c1a83a1501357286877f05beb6c8 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit b7d421da6d5e945b74ffbc3a7aa635a6ce63ebda)
| * Remove Q_ASSERT's from QMutex autotestJason McDonald2011-05-181-4/+7
| | | | | | | | | | | | | | | | | | | | | | Don't perform actions needed for the test inside Q_ASSERT, because these actions won't happen in a non-debug build. Instead count the number of failures and verify that the count is zero at the end of the test. Change-Id: Ibe0d194111e5247118d59a7760f0946d2c44faf9 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 8d2127d44e6c8af6b041688376de0e5747a1810a)
| * Remove Q_ASSERT from QMetaType autotestJason McDonald2011-05-181-1/+10
| | | | | | | | | | | | | | | | | | | | | | Replace Q_ASSERT in helper class with code to count failures and report meaningful warnings. The test function then fails the test if any failures were recorded. Change-Id: I0d6650e6036c8e45729c16d1dbb7543b4fb42553 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 8512dc16553ecf511c40a2ba7a815f777d7d5c59)
| * Remove Q_ASSERT's in qitemview autotestJason McDonald2011-05-181-12/+53
| | | | | | | | | | | | | | | | | | | | | | Rather than aborting in debug builds and failing silently in release builds, report a meaningful warning message and return an appropriate value to indicate the error. Change-Id: I0ceb0a0bfaef34cc6127d768cc75ecfc5a24e3c9 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 7cb0a3baeb361c9909e8ab2caf436e8e44b6a3b6)
| * Remove Q_ASSERT from qitemmodel autotestJason McDonald2011-05-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | If populateTestData() is passed an unknown type of model, report a meaningful fatal error rather than failing silently in a release build and aborting with an uninformative error message in a debug build. Change-Id: I7ee8d1262bbced304f9f24c2ec9b40a38704d982 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 6b2e89843a573cd7c143bca17cec8e69209d2fbd)
| * Remove Q_ASSERT from algorithms autotestJason McDonald2011-05-181-1/+2
| | | | | | | | | | | | | | | | | | | | Report a meaningful warning message rather than aborting in a debug build and failing silently in a release build. Change-Id: Ic77589143ff0d22c95c7b10f2e511f68b3da4d86 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 75767c69fbba3149a4505287ed441886188a9fd3)
| * Remove Q_ASSERT's from qabstractfileengine testJason McDonald2011-05-181-4/+4
| | | | | | | | | | | | | | | | | | | | | | Rather than aborting in debug builds and failing silently in release builds, report a meaningful warning message and skip setting the filename, which should cause a subsequent test failure. Change-Id: I3ae4f4de7b02bf2194019047fa87d8ae06d95634 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit e6f7509f18ae88069373d59073cbdcda88fd19c8)
| * Remove Q_ASSERT's from modeltestJason McDonald2011-05-181-10/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rather than aborting in debug builds and ignoring the failures in release builds, report specific warnings on each failure, count the failures, and fail the test if the failure counts are non-zero at the end of the test. The same change is also made for a QCOMPARE that appeared inappropriately inside a helper class. QCOMPARE may only appear directly in a test function. Change-Id: I81f0ce80512fa72c67f5aa72c0511a4b650d5d20 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 11254ff409395b67cb28453b3c069860bc28c14d)
| * Remove Q_ASSERT from modeltestJason McDonald2011-05-181-1/+2
| | | | | | | | | | | | | | | | | | | | | | Report an informative fatal error if passed a null model, rather than aborting in a debug build and giving a bunch of signal connection errors in a release build. Change-Id: Ia240e741b9d6ec03fd5ed3a14cf4fa44b55af911 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 841cc610df61a266ebcca2bfef2542a7d4fea68d)
| * Remove Q_ASSERT's from modeltestJason McDonald2011-05-181-3/+7
| | | | | | | | | | | | | | | | | | Report a fatal error in all builds, not just in debug builds. Change-Id: Ia9e265b2082b55fbac18ca046e586de863ac0623 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 22e99d92a71645d643ebd54c6209bced80f7c5b8)
| * Remove Q_ASSERT's from QAbstractFileEngine autotestJason McDonald2011-05-181-10/+39
| | | | | | | | | | | | | | | | | | | | | | | | Rather than aborting in debug builds and ignoring failures in release builds, report meaningful warnings into the test output and return sentinel values that will cause QVERIFY/QCOMPARE of the returned values to fail the test. Change-Id: I2c5a820637337d0762c71db10a4f270d36b31662 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 524bd6b7120f70a178b03bc6d337e08abd327076)
| * Remove Q_ASSERT's from QTextEdit autotestJason McDonald2011-05-181-3/+3
| | | | | | | | | | | | | | | | | | | | Convert Q_ASSERT to QCOMPARE and remove mention of Q_ASSERT in comments to make it easier to check for unwanted Q_ASSERT's in the future. Change-Id: I7ae250428d4d5682806e724a7207523e65cd9851 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 3290dd0d3563310f6bdbabf74105367bd8c611d3)
| * Remove Q_ASSERT's from large file autotest.Jason McDonald2011-05-181-5/+0
| | | | | | | | | | | | | | | | | | These assertions were made obsolete by the previous change to this test. Change-Id: I3156b273a727f753a6c093c8fc7453b694c777ee Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 57f6ed6664f55ccee62b81750a2b3520249503fd)
| * Remove Q_ASSERT from QMutex autotestJason McDonald2011-05-181-4/+14
| | | | | | | | | | | | | | | | | | | | | | Rather than aborting in debug mode and doing nothing in release mode when the invariant is violated, count the failures and fail the test gracefully. Change-Id: Ie193460c478ddde540b6b15aafdce32f471b4b2b Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit f18e0e01468899731bc3777649d69fd6d0041012)
| * Remove Q_ASSERT from qtesselator autotestJason McDonald2011-05-181-1/+3
| | | | | | | | | | | | | | | | | | | | If the test data is incorrect, print a meaningful warning into the test output. Change-Id: Ibfe64ef41a0ff3eb47c2385d1c31191334a675a8 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 28ccfa472991c0480e67b5f204a567c4023ba6d4)
| * Remove Q_ASSERT's from exceptionsafety_objects testJason McDonald2011-05-181-3/+6
| | | | | | | | | | | | | | | | | | Report a fatal error in all builds, not just in debug builds. Change-Id: Iad990e2d3b29a55e995254d466e967a4a722bb23 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit a212b402da4b4af9d453fa96dbfa2d32271e857d)
| * Remove Q_ASSERT in gestures autotestJason McDonald2011-05-181-4/+12
| | | | | | | | | | | | | | | | | | | | | | Rather than aborting on a bad gesture event in debug builds and ignoring the error in release builds, record a count of bad events and fail the test if the count is non-zero at the end of the test function. Change-Id: I6ddd46a5a656185c13eae4bbbb496b986a0c92f6 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 5953d930bc07fa6734a11d053d26a3f80e9c1e89)
| * Remove Q_ASSERT's from QTreeView autotestJason McDonald2011-05-181-5/+9
| | | | | | | | | | | | | | | | | | | | Issue a meaningful fatal error in preference to aborting in debug mode builds and crashing in release mode builds. Change-Id: I7bb04e1e222fd6167be19b5d88caac27b43d88df Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit cb126ff7ad08e9801e2911511aa9aeb728faa8f3)
| * Remove Q_ASSERT from qtesselator autotestJason McDonald2011-05-181-1/+2
| | | | | | | | | | | | | | | | | | | | If the test data is incorrect, print a meaningful error message into the test output. Change-Id: I505a86b4411619d3c3be9541c96c082f92589ff7 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 8e12d9e322ec7d8f7eece09b6f15cba640615f9e)
| * Remove mention of Q_ASSERT in QTableView test.Jason McDonald2011-05-181-2/+2
| | | | | | | | | | | | | | Change-Id: If27cb7031cdde222d1dd988188569dca4b1fb6d9 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 324b9db856f00a82d7fd3426d9b698211290efaa)
| * Remove Q_ASSERT's from QXmlStream autotest.Jason McDonald2011-05-181-3/+2
| | | | | | | | | | | | | | | | | | | | Report a fatal error in all builds rather than only in debug mode builds. Change-Id: I6b3a999dd3ccf00ae66a86b43c125d7082cf0ae5 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit c183540c6c244f327ebd834167799072dcc3b72d)
| * Remove Q_ASSERT's from qtestlib selftestsJason McDonald2011-05-181-3/+4
| | | | | | | | | | | | | | | | | | | | Report a fatal error in all builds rather than just in debug mode builds. Change-Id: Ie54e55ea2dd13bbb12e4265278db832ac1297c28 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 8a1279b0ae180fae94d3da1a4eeb20df5a748d33)
| * Remove unused code from QXmlStream autotestJason McDonald2011-05-181-15/+2
| | | | | | | | | | | | Change-Id: I719e502ef14848a22d41de71245fe6b6758c8d6b Reviewed-by: Rohan McGovern (cherry picked from commit 614474fb3d440eabd3ba7e381d246064ce39f4fa)
| * Remove Q_ASSERT from qxmlsimplereader autotestJason McDonald2011-05-181-10/+14
| | | | | | | | | | | | | | | | | | | | Verify the success of setting the document content in the test function rather than using Q_ASSERT. Change-Id: I25e017e7f92803867a998b4f09ad9f717212a9bc Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit d7453aef14b80a5972909ed99f1b4c765b7436a5)
| * Remove Q_ASSERT from QXmlInputSource autotestJason McDonald2011-05-181-3/+1
| | | | | | | | | | | | | | | | | | | | Report a fatal error in all builds rather than aborting in debug mode builds only. Change-Id: I53b348f077c6533433e32bbebd159066a2bc2c37 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 4f818b8bcf51cf9c654edbf319d90f6c77a09b3e)
| * Remove Q_ASSERT from QLibrary autotestJason McDonald2011-05-181-1/+1
| | | | | | | | | | | | | | | | | | | | Report a regular test failure rather than aborting when an unknown operation is requested. Change-Id: I3678f42f9e9f895d8d0d49a768d4c8b575d1863f Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 22aa68ecec96da4d14a4c9fe2f6e4a337b13b4a3)
| * Remove Q_ASSERT from QSharedPointer autotestJason McDonald2011-05-181-1/+2
| | | | | | | | | | | | | | | | | | | | Report a meaningful fatal error instead of aborting in debug mode builds and failing silently in release mode builds. Change-Id: I0f939dc40810f072a3e24aeba085191793c49ef3 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit b988b2f9f04ac0ebce7a779cd9bd5d394bfa1c95)
| * Remove Q_ASSERT from QThread autotestJason McDonald2011-05-181-1/+1
| | | | | | | | | | | | | | | | | | | | Report a meaningful fatal error instead of aborting in debug mode builds and failing silently in release mode builds. Change-Id: If9a6c0655e021b3bb6b7d894aefd9bbcc58e4605 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit fed42f25e129af1f4f2c03a075530ad7111d2fdd)
| * Remove Q_ASSERT's from QWizard autotestJason McDonald2011-05-181-10/+11
| | | | | | | | | | | | | | | | | | | | | | Combine the two checking functions (check and checkInvariant) and have the resulting check function return a bool instead of asserting so that the test function can QVERIFY and fail gracefully. Change-Id: Ib069b5424483ba6ffb9caf75036c4f325e9dba51 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit a54900d2227bfb2021c5d1255817824aff11cf32)
| * Remove mention of Q_ASSERT in commentsJason McDonald2011-05-181-1/+1
| | | | | | | | | | | | | | | | | | This will make it easier to check for stray Q_ASSERT's in the future. Change-Id: I5ddafe1bf26819566460920deae8ff448008c93f Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit b2db13f245f09f1d14af64d2199284c78cbb5b6e)
| * Remove Q_ASSERT's from QVariant autotest.Jason McDonald2011-05-181-5/+8
| | | | | | | | | | | | | | | | | | | | Print a meaningful warning message instead of aborting in debug mode builds and failing silently in release mode builds. Change-Id: I44143c5fbe6b6af87bdf5bd231cfaf9a3c9c33f8 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit a8dc321b6669dd263a25a0cb5766e5b7150c6e20)
| * Remove Q_ASSERT from QTextOdfWriter autotestJason McDonald2011-05-181-4/+7
| | | | | | | | | | | | | | | | | | | | If the <dummy> tag can't be found in the data, return an empty string rather than asserting, so that the test fails gracefully. Change-Id: I536f08c9c3e942817680849d96d035999d4994db Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit a88dd1c5f62b5ded8ea0d76d185127ef28549c72)
| * Remove Q_ASSERT from qtesselator autotestJason McDonald2011-05-181-2/+6
| | | | | | | | | | | | | | | | | | | | | | If the test data is incorrect, print a meaningful warning into the test output instead of deferencing an iterator beyond the last element in the list. Change-Id: I7be4f282639453de6d8240a2f17253025b415337 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit f01e2a5024db69913aed016e2854b2589ca85080)
| * Remove Q_ASSERT from qsslsocket autotest.Rohan McGovern2011-05-181-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the server thread exit early if the socket is invalid. This will make the test fail gracefully instead of crashing. Change-Id: Ia9564c94dd32d65e6e9bdb4a2410f1512409546c Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit b9b3ced2eafe13e491d5af5ad7432733941bfd8d) Conflicts: tests/auto/qsslsocket/tst_qsslsocket.cpp
| * Remove Q_ASSERT's from qsharedpointer autotestJason McDonald2011-05-181-4/+10
| | | | | | | | | | | | | | | | | | | | Report a meaningful warning message into the test output rather than aborting in debug mode and doing nothing in release mode. Change-Id: I6883fccbce0139c763f36f6839bb3452d8f69c1c Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 5b86c4abc676b36d60e6625e89535e7f4907d2b3)
| * Remove Q_ASSERT's from QObject autotest.Jason McDonald2011-05-181-2/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The Receiver class has two slots that aren't meant to get called during the test (they're there to catch broken parsing of slot names). Rather than asserting when one of them gets called, which does nothing in a release mode build, this commit makes the slots record the number of times they were called (as for the other slots in the test) and fails the test gracefully if either of those slots was called. Change-Id: Ia0393026cb96ffdc6190b5e7bd951f75d231b11e Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 7bd6ca895e5fa4de197d9d7bf2e7b578c01c3c2a)
| * Remove Q_ASSERT's from QMetaType autotest.Jason McDonald2011-05-181-5/+27
| | | | | | | | | | | | | | | | | | | | Instead of asserting in debug mode and doing nothing in release mode, put specific warnings in the test output and fail the test gracefully. Change-Id: I453a0ab7ddef5b2acf55f77f71a59a940d93ae54 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit a3b2fa3f1beffa7709c11522d4e2db9ec8f952e0)
| * Remove Q_ASSERT from QMetaObject autotest.Jason McDonald2011-05-181-2/+12
| | | | | | | | | | | | | | | | | | | | Issue a helpful warning rather than ignoring the error in release mode builds. Change-Id: Iea7bf4d63f8227abaf7dcf1700c5fef62763afad Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 6f082a3fdd368cd7a969b7f324354583d081da82)
| * Change Q_ASSERT to QVERIFY in QMenu test.Jason McDonald2011-05-181-1/+1
| | | | | | | | | | | | | | Change-Id: Ifc08d4eabd8565cfd242ec2e70cf597e92f28fe3 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 291f04a4a7be4a975c6b8eaeeed4e0c196e23128)
| * Remove Q_ASSERT from qitemview autotestJason McDonald2011-05-182-7/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | Allow createView() to return a null view on failure (which it already did in release mode builds), but fail the test gracefully instead of crashing when a null view is returned. Also reordered the code slightly to put all the QFETCH's together. Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern Change-Id: I3a41c738cb0e4e860db2342b30817a804b20f30d (cherry picked from commit 51b1d8dc20a2557d2002550b03e5b9f37f425fb4)
| * Remove Q_ASSERT from qitemmodel autotestJason McDonald2011-05-182-1/+12
| | | | | | | | | | | | | | | | | | | | | | Allow createModel() to return a null model on failure (which it already did in release mode builds), but fail the test gracefully instead of crashing when a null model is returned. Change-Id: Iaa0ec31183e0c90fcea512223d01e07ed188a380 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit 0d773d839e39f4f4bfd0bf9d652e4f2650adfcb0)
| * Remove mention of Q_ASSERT from qheaderview autotestJason McDonald2011-05-181-1/+1
| | | | | | | | | | | | | | Change-Id: I3a97411c0b53345bffbadb60a3e7caabfb0441d2 Task-number: QTBUG-17582 Reviewed-by: Rohan McGovern (cherry picked from commit a388e104febb131cd35438bbb1397eeab41d3ab8)