From b99d7623f30208137d9f676a003ca493a1b74df9 Mon Sep 17 00:00:00 2001
From: Christian Stenger <christian.stenger@qt.io>
Date: Thu, 2 Sep 2021 16:57:45 +0200
Subject: [PATCH] AutoTest: Fix synchronizing test tools

It is wrong to query the global active state when
adding test tools known to be active.

Change-Id: I89ab292e1a71b43757e6308fa4b4c584d62fe28b
Reviewed-by: David Schulz <david.schulz@qt.io>
---
 src/plugins/autotest/testtreemodel.cpp | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/plugins/autotest/testtreemodel.cpp b/src/plugins/autotest/testtreemodel.cpp
index b6f979fe335..8992fdeabeb 100644
--- a/src/plugins/autotest/testtreemodel.cpp
+++ b/src/plugins/autotest/testtreemodel.cpp
@@ -391,11 +391,9 @@ void TestTreeModel::synchronizeTestTools()
 
     for (ITestTool *testTool : qAsConst(tools)) {
         ITestTreeItem *testToolRootNode = testTool->rootNode();
-        if (testTool->active()) {
-            invisibleRoot->appendChild(testToolRootNode);
-            if (!oldFrameworkRoots.removeOne(testToolRootNode))
-                newlyAdded.insert(testTool);
-        }
+        invisibleRoot->appendChild(testToolRootNode);
+        if (!oldFrameworkRoots.removeOne(testToolRootNode))
+            newlyAdded.insert(testTool);
     }
 
     if (project) {