CppTools/Editor: Make sure test code is free of diagnostic messages

...where applicable.

Change-Id: Ie42e1012598adf124e5f66b6a53eda01724dfb71
Reviewed-by: Przemyslaw Gorszkowski <pgorszkowski@gmail.com>
Reviewed-by: Orgad Shaneh <orgads@gmail.com>

1 files changed, 5 insertions, 4 deletions

diff --git a/src/plugins/cpptools/cpppointerdeclarationformatter_test.cpp b/src/plugins/cpptools/cpppointerdeclarationformatter_test.cpp
index 1212ea0060..4fd9eae00d 100644
--- a/src/plugins/cpptools/cpppointerdeclarationformatter_test.cpp
+++ b/src/plugins/cpptools/cpppointerdeclarationformatter_test.cpp
@@ -99,6 +99,7 @@ public:
         document->setUtf8Source(preprocessedSource);
         document->parse(parseMode);
         document->check();
+        QVERIFY(document->diagnosticMessages().isEmpty());
         AST *ast = document->translationUnit()->ast();
         QVERIFY(ast);
 
@@ -614,25 +615,25 @@ void CppToolsPlugin::test_format_pointerdeclaration_macros_data()
 
     source = QLatin1String(
         "#define FOO int*\n"
-        "FOO @f() {};\n");
+        "FOO @f() {}\n");
     QTest::newRow("macro-in-function-definition-returntype")
         << source << stripCursor(source);
 
     source = QLatin1String(
         "#define FOO int*\n"
-        "int f(FOO @a) {};\n");
+        "int f(FOO @a) {}\n");
     QTest::newRow("macro-in-function-definition-param")
         << source << stripCursor(source);
 
     source = QLatin1String(
         "#define FOO int*\n"
-        "while (FOO @s = 0);\n");
+        "void f() { while (FOO @s = 0) {} }\n");
     QTest::newRow("macro-in-if-while-for")
         << source << stripCursor(source);
 
     source = QLatin1String(
         "#define FOO int*\n"
-        "foreach (FOO @s, list);\n");
+        "void f() { foreach (FOO @s, list) {} }\n");
     QTest::newRow("macro-in-foreach")
         << source << stripCursor(source);