Move the remaining blacklist setting into TestMethods::invokeTest()

Move the remaining QTestResult::setBlacklistCurrentTest()
call from checkBlackLists() into TestMethods::invokeTest()

Change-Id: I6f0bb26ef612e707ad013d1e42e47b06f40482a0
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>
Reviewed-by: Jason McDonald <macadder1@gmail.com>
Reviewed-by: Dimitrios Apostolou <jimis@qt.io>

2 files changed, 4 insertions, 3 deletions

diff --git a/src/testlib/qtestblacklist.cpp b/src/testlib/qtestblacklist.cpp
index d3f01883e2..ccd8ba73c4 100644
--- a/src/testlib/qtestblacklist.cpp
+++ b/src/testlib/qtestblacklist.cpp
@@ -302,7 +302,6 @@ bool checkBlackLists(const char *slot, const char *data, const char *global)
         }
     }
 
-    QTestResult::setBlacklistCurrentTest(ignore);
     return ignore;
 }
 
diff --git a/src/testlib/qtestcase.cpp b/src/testlib/qtestcase.cpp
index 7540c9a247..d315312363 100644
--- a/src/testlib/qtestcase.cpp
+++ b/src/testlib/qtestcase.cpp
@@ -1443,8 +1443,10 @@ bool TestMethods::invokeTest(int index, QLatin1StringView tag, WatchDog *watchDo
             if (dataTagMatches(tag, QLatin1StringView(dataTag(curDataIndex)),
                                QLatin1StringView(globalDataTag(curGlobalDataIndex)))) {
                 foundFunction = true;
-                QTestPrivate::checkBlackLists(name.constData(), dataTag(curDataIndex),
-                                              globalDataTag(curGlobalDataIndex));
+                if (QTestPrivate::checkBlackLists(name.constData(), dataTag(curDataIndex),
+                                                  globalDataTag(curGlobalDataIndex))) {
+                    QTestResult::setBlacklistCurrentTest(true);
+                 }
 
                 QTestDataSetter s(curDataIndex >= dataCount ? nullptr : table.testData(curDataIndex));