From 77a8005dcb12a274193c8009df03e84ff6b51582 Mon Sep 17 00:00:00 2001
From: Giuseppe D'Angelo <giuseppe.dangelo@kdab.com>
Date: Thu, 4 Jun 2020 02:24:19 +0200
Subject: QStringBuilder: code tidies in the tests

Including other headers, while defining very questionable macro
names, can and will result in trouble. Stop doing that.

While at it, move from including <QtTest> to include-what-you-use,
and clean up the code a bit.

Change-Id: Idb02ef2b612c0805baecac3ce6edd435609aca4c
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
---
 .../text/qstringbuilder/qstringbuilder1/stringbuilder.cpp      |  4 ++--
 .../qstringbuilder/qstringbuilder1/tst_qstringbuilder1.cpp     | 10 +++++++---
 2 files changed, 9 insertions(+), 5 deletions(-)

(limited to 'tests/auto/corelib/text/qstringbuilder/qstringbuilder1')

diff --git a/tests/auto/corelib/text/qstringbuilder/qstringbuilder1/stringbuilder.cpp b/tests/auto/corelib/text/qstringbuilder/qstringbuilder1/stringbuilder.cpp
index ac7f439248..e31708ad72 100644
--- a/tests/auto/corelib/text/qstringbuilder/qstringbuilder1/stringbuilder.cpp
+++ b/tests/auto/corelib/text/qstringbuilder/qstringbuilder1/stringbuilder.cpp
@@ -26,7 +26,8 @@
 **
 ****************************************************************************/
 
-#include <QtTest/QtTest>
+// Do not include anything in this file. We are being #included with
+// a bunch of defines that may break other legitimate code.
 
 #define LITERAL "some literal"
 #define LITERAL_LEN (sizeof(LITERAL)-1)
@@ -46,7 +47,6 @@
 
 #ifndef P
 # error You need to define P
-# define P +
 #endif
 
 //fix for gcc4.0: if the operator+ does not exist without QT_USE_FAST_OPERATOR_PLUS
diff --git a/tests/auto/corelib/text/qstringbuilder/qstringbuilder1/tst_qstringbuilder1.cpp b/tests/auto/corelib/text/qstringbuilder/qstringbuilder1/tst_qstringbuilder1.cpp
index 2d320748f2..fcf004b4fc 100644
--- a/tests/auto/corelib/text/qstringbuilder/qstringbuilder1/tst_qstringbuilder1.cpp
+++ b/tests/auto/corelib/text/qstringbuilder/qstringbuilder1/tst_qstringbuilder1.cpp
@@ -30,13 +30,14 @@
 // SCENARIO 1
 // this is the "no harm done" version. Only operator% is active,
 // with NO_CAST * defined
-#define P %
 #undef QT_USE_QSTRINGBUILDER
 #define QT_NO_CAST_FROM_ASCII
 #define QT_NO_CAST_TO_ASCII
 
-
-#include <QtTest/QtTest>
+#include <QtCore/QObject>
+#include <QtCore/QString>
+#include <QtCore/QStringBuilder>
+#include <QtTest/QTest>
 
 #define LITERAL "some literal"
 
@@ -50,7 +51,10 @@ private slots:
     void scenario() { runScenario(); }
 };
 
+#define P %
 #include "stringbuilder.cpp"
+#undef P
+
 #include "tst_qstringbuilder1.moc"
 
 QTEST_APPLESS_MAIN(tst_QStringBuilder1)
-- 
cgit v1.2.3