path: root/tests/auto
diff options
authorJason McDonald <>2011-11-17 13:29:36 +1000
committerQt by Nokia <>2011-11-17 07:21:03 +0100
commit2b883dc9bb6cfd186a23a945597f67a9f9fc270e (patch)
tree831226ed5343e6afffc509b1191e67b658213bad /tests/auto
parent577feb71ca98021804bd0111858a48f8421e08f0 (diff)
Improve Q_FUNC_INFO autotest.
The availableWithoutDebug() test was trying to force QT_NO_DEBUG to be defined, but was actually doing the opposite. While I'm not satisfied with the way this test works (it would be better to test with QT_NO_DEBUG already defined when Q_FUNC_INFO is included). at least this test now does what it says it does. Additionally, the test has been improved by checking that Q_FUNC_INFO returns a non-empty string, as opposed to just compiling. Change-Id: I9175e3a7406152b6250b507a61b5062e3cfabeb8 Reviewed-by: Rohan McGovern <>
Diffstat (limited to 'tests/auto')
1 files changed, 6 insertions, 7 deletions
diff --git a/tests/auto/corelib/global/q_func_info/tst_q_func_info.cpp b/tests/auto/corelib/global/q_func_info/tst_q_func_info.cpp
index 0a2a259462..e4949348f4 100644
--- a/tests/auto/corelib/global/q_func_info/tst_q_func_info.cpp
+++ b/tests/auto/corelib/global/q_func_info/tst_q_func_info.cpp
@@ -123,20 +123,19 @@ void tst_q_func_info::isOfTypeConstChar() const
/* \internal
Ensure that the macro is available even though QT_NO_DEBUG
- is defined. We do this by undefining it, and turning it on again
- backwards(just so we don't break stuff), if it was in fact defined.
+ is defined. If QT_NO_DEBUG is not defined, we define it
+ just for this function then undef it again afterwards.
void tst_q_func_info::availableWithoutDebug() const
#ifndef QT_NO_DEBUG
-# define USE_DEBUG
+# define UNDEF_NO_DEBUG
# define QT_NO_DEBUG
-#undef QT_NO_DEBUG
- QString::fromLatin1(Q_FUNC_INFO);
-#ifdef USE_DEBUG
+ QVERIFY(!QString::fromLatin1(Q_FUNC_INFO).isEmpty());
# undef QT_NO_DEBUG
-# undef USE_DEBUG