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>
---
 .../qstringbuilder/qstringbuilder1/tst_qstringbuilder1.cpp     | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

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

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