gui: count test failures in test runner summary

src/qt/test/test_main.cpp

@@ -71,8 +71,6 @@ int main(int argc, char* argv[])
     gArgs.ForceSetArg("-upnp", "0");
     gArgs.ForceSetArg("-natpmp", "0");
 
-    bool fInvalid = false;
-
     // Prefer the "minimal" platform for the test instead of the normal default
     // platform ("xcb", "windows", or "cocoa") so tests can't unintentionally
     // interfere with any background GUIs and don't require extra resources.
@@ -88,35 +86,30 @@ int main(int argc, char* argv[])
     app.setApplicationName("Bitcoin-Qt-test");
     app.createNode(*init);
 
+    int fInvalid{0};
+
     AppTests app_tests(app);
-    if (QTest::qExec(&app_tests) != 0) {
+    fInvalid += QTest::qExec(&app_tests);
-        fInvalid = true;
+
-    }
     OptionTests options_tests(app.node());
-    if (QTest::qExec(&options_tests) != 0) {
+    fInvalid += QTest::qExec(&options_tests);
-        fInvalid = true;
+
-    }
     URITests test1;
-    if (QTest::qExec(&test1) != 0) {
+    fInvalid += QTest::qExec(&test1);
-        fInvalid = true;
+
-    }
     RPCNestedTests test3(app.node());
-    if (QTest::qExec(&test3) != 0) {
+    fInvalid += QTest::qExec(&test3);
-        fInvalid = true;
+
-    }
 #ifdef ENABLE_WALLET
     WalletTests test5(app.node());
-    if (QTest::qExec(&test5) != 0) {
+    fInvalid += QTest::qExec(&test5);
-        fInvalid = true;
+
-    }
     AddressBookTests test6(app.node());
-    if (QTest::qExec(&test6) != 0) {
+    fInvalid += QTest::qExec(&test6);
-        fInvalid = true;
-    }
 #endif
 
     if (fInvalid) {
-        qWarning("\nThere were errors in some of the tests above.\n");
+        qWarning("\nFailed tests: %d\n", fInvalid);
     } else {
         qDebug("\nAll tests passed.\n");
     }