CMakeBuildSystem: Run ctest process asynchronously in main thread

Instead of running it in other thread using blocking interface.

Amends 675a72e296

Change-Id: Ia4c7701fa4cd9efea4aeccd979dfb8cde28a889e
Reviewed-by: Christian Stenger <christian.stenger@qt.io>
Reviewed-by: <github-actions-qt-creator@cristianadam.eu>

src/plugins/cmakeprojectmanager/cmakebuildsystem.cpp

@@ -10,7 +10,6 @@
 #include "cmakekitinformation.h"
 #include "cmakeproject.h"
 #include "cmakeprojectconstants.h"
-#include "cmakeprojectnodes.h"
 #include "cmakeprojectmanagertr.h"
 #include "cmakeprojectplugin.h"
 #include "cmakespecificsettings.h"
@@ -24,14 +23,11 @@
 
 #include <cppeditor/cppeditorconstants.h>
 #include <cppeditor/cppprojectupdater.h>
-#include <cppeditor/generatedcodemodelsupport.h>
 
-#include <projectexplorer/buildsteplist.h>
 #include <projectexplorer/extracompiler.h>
 #include <projectexplorer/kitinformation.h>
 #include <projectexplorer/projectexplorer.h>
 #include <projectexplorer/projectexplorerconstants.h>
-#include <projectexplorer/session.h>
 #include <projectexplorer/target.h>
 #include <projectexplorer/taskhub.h>
 
@@ -45,15 +41,11 @@
 #include <utils/checkablemessagebox.h>
 #include <utils/fileutils.h>
 #include <utils/macroexpander.h>
-#include <utils/mimeutils.h>
 #include <utils/qtcassert.h>
 #include <utils/qtcprocess.h>
-#include <utils/runextensions.h>
 
 #include <QClipboard>
-#include <QDir>
 #include <QGuiApplication>
-#include <QHash>
 #include <QJsonArray>
 #include <QJsonDocument>
 #include <QJsonObject>
@@ -191,7 +183,6 @@ CMakeBuildSystem::CMakeBuildSystem(CMakeBuildConfiguration *bc)
 
 CMakeBuildSystem::~CMakeBuildSystem()
 {
-    m_futureSynchronizer.waitForFinished();
     if (!m_treeScanner.isFinished()) {
         auto future = m_treeScanner.future();
         future.cancel();
@@ -914,26 +905,15 @@ void CMakeBuildSystem::runCTest()
     const BuildDirParameters parameters(this);
     QTC_ASSERT(parameters.isValid(), return);
 
-    const CommandLine cmd { m_ctestPath, { "-N", "--show-only=json-v1" } };
     ensureBuildDirectory(parameters);
-    const FilePath workingDirectory = parameters.buildDirectory;
-    const Environment environment = buildConfiguration()->environment();
-
-    auto future = Utils::runAsync([cmd, workingDirectory, environment]
-                                  (QFutureInterface<QByteArray> &futureInterface) {
-        QtcProcess process;
-        process.setEnvironment(environment);
-        process.setWorkingDirectory(workingDirectory);
-        process.setCommand(cmd);
-        process.start();
-        if (!process.waitForFinished() || process.result() != ProcessResult::FinishedWithSuccess)
-            return;
-        futureInterface.reportResult(process.readAllRawStandardOutput());
-    });
-
-    Utils::onFinished(future, this, [this](const QFuture<QByteArray> &future) {
-        if (future.resultCount()) {
-            const QJsonDocument json = QJsonDocument::fromJson(future.result());
+    m_ctestProcess.reset(new QtcProcess);
+    m_ctestProcess->setEnvironment(buildConfiguration()->environment());
+    m_ctestProcess->setWorkingDirectory(parameters.buildDirectory);
+    m_ctestProcess->setCommand({m_ctestPath, { "-N", "--show-only=json-v1"}});
+    connect(m_ctestProcess.get(), &QtcProcess::done, this, [this] {
+        if (m_ctestProcess->result() == ProcessResult::FinishedWithSuccess) {
+            const QJsonDocument json
+                = QJsonDocument::fromJson(m_ctestProcess->readAllRawStandardOutput());
             if (!json.isEmpty() && json.isObject()) {
                 const QJsonObject jsonObj = json.object();
                 const QJsonObject btGraph = jsonObj.value("backtraceGraph").toObject();
@@ -972,8 +952,7 @@ void CMakeBuildSystem::runCTest()
         }
         emit testInformationUpdated();
     });
-
-    m_futureSynchronizer.addFuture(future);
+    m_ctestProcess->start();
 }
 
 CMakeBuildConfiguration *CMakeBuildSystem::cmakeBuildConfiguration() const
@@ -981,7 +960,7 @@ CMakeBuildConfiguration *CMakeBuildSystem::cmakeBuildConfiguration() const
     return static_cast<CMakeBuildConfiguration *>(BuildSystem::buildConfiguration());
 }
 
-static Utils::FilePaths librarySearchPaths(const CMakeBuildSystem *bs, const QString &buildKey)
+static FilePaths librarySearchPaths(const CMakeBuildSystem *bs, const QString &buildKey)
 {
     const CMakeBuildTarget cmakeBuildTarget
         = Utils::findOrDefault(bs->buildTargets(), Utils::equal(&CMakeBuildTarget::title, buildKey));
@@ -1359,7 +1338,7 @@ QList<QPair<Id, QString>> CMakeBuildSystem::generators() const
     return result;
 }
 
-void CMakeBuildSystem::runGenerator(Utils::Id id)
+void CMakeBuildSystem::runGenerator(Id id)
 {
     QTC_ASSERT(cmakeBuildConfiguration(), return);
     const auto showError = [](const QString &detail) {

src/plugins/cmakeprojectmanager/cmakebuildsystem.h

@@ -11,7 +11,6 @@
 #include <projectexplorer/buildsystem.h>
 
 #include <utils/fileutils.h>
-#include <utils/futuresynchronizer.h>
 #include <utils/temporarydirectory.h>
 
 namespace CppEditor { class CppProjectUpdater; }
@@ -19,6 +18,7 @@ namespace ProjectExplorer {
     class ExtraCompiler;
     class FolderNode;
 }
+namespace Utils { class QtcProcess; }
 
 namespace CMakeProjectManager {
 
@@ -206,8 +206,8 @@ private:
 
     // CTest integration
     Utils::FilePath m_ctestPath;
+    std::unique_ptr<Utils::QtcProcess> m_ctestProcess;
     QList<ProjectExplorer::TestCaseInfo> m_testNames;
-    Utils::FutureSynchronizer m_futureSynchronizer;
 
     CMakeConfig m_configurationFromCMake;
     CMakeConfig m_configurationChanges;