From 284817fae6514701902ccdb834c2faa46462f2e8 Mon Sep 17 00:00:00 2001
From: Christian Kandeler <christian.kandeler@qt.io>
Date: Mon, 30 Aug 2021 10:58:08 +0200
Subject: Merge CppTools into CppEditor

There was no proper separation of responsibilities between these
plugins. In particular, CppTools had lots of editor-related
functionality, so it's not clear why it was separated out in the first
place.
In fact, for a lot of code, it seemed quite arbitrary where it was put
(just one example: switchHeaderSource() was in CppTools, wheras
switchDeclarationDefinition() was in CppEditor).
Merging the plugins will enable us to get rid of various convoluted
pseudo-abstractions that were only introduced to keep up the artificial
separation.

Change-Id: Iafc3bce625b4794f6d4aa03df6cddc7f2d26716a
Reviewed-by: Christian Stenger <christian.stenger@qt.io>
---
 .../followsymbol_switchmethoddecldef_test.cpp      | 36 ++++++++++------------
 1 file changed, 17 insertions(+), 19 deletions(-)

(limited to 'src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp')

diff --git a/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp b/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp
index 10c78edc8a0..f005848b83b 100644
--- a/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp
+++ b/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp
@@ -25,19 +25,18 @@
 
 #include "followsymbol_switchmethoddecldef_test.h"
 
+#include "cppcodemodelsettings.h"
 #include "cppeditor.h"
 #include "cppeditorplugin.h"
 #include "cppeditortestcase.h"
 #include "cppeditorwidget.h"
-
-#include <cpptools/cppcodemodelsettings.h>
-#include <cpptools/cppelementevaluator.h>
-#include <cpptools/cppfollowsymbolundercursor.h>
-#include <cpptools/cppvirtualfunctionassistprovider.h>
-#include <cpptools/cppvirtualfunctionproposalitem.h>
-#include <cpptools/cpptoolsreuse.h>
-#include <cpptools/cpptoolstestcase.h>
-#include <cpptools/cppmodelmanager.h>
+#include "cppelementevaluator.h"
+#include "cppfollowsymbolundercursor.h"
+#include "cppmodelmanager.h"
+#include "cpptoolsreuse.h"
+#include "cpptoolstestcase.h"
+#include "cppvirtualfunctionassistprovider.h"
+#include "cppvirtualfunctionproposalitem.h"
 
 #include <projectexplorer/kitmanager.h>
 #include <projectexplorer/projectexplorer.h>
@@ -80,7 +79,6 @@
  */
 
 using namespace CPlusPlus;
-using namespace CppTools;
 using namespace TextEditor;
 using namespace Core;
 using namespace ProjectExplorer;
@@ -261,7 +259,7 @@ public:
 
     ~F2TestCase()
     {
-        CppTools::ClangdSettings::setUseClangd(m_prevUseClangd);
+        ClangdSettings::setUseClangd(m_prevUseClangd);
     }
 
     static ProjectExplorer::Kit *m_testKit;
@@ -281,12 +279,12 @@ ProjectExplorer::Kit *F2TestCase::m_testKit = nullptr;
 F2TestCase::F2TestCase(CppEditorAction action,
                        const QList<TestDocumentPtr> &testFiles,
                        OverrideItemList expectedVirtualFunctionProposal)
-    : m_prevUseClangd(CppTools::ClangdSettings::instance().useClangd())
+    : m_prevUseClangd(ClangdSettings::instance().useClangd())
 {
     QVERIFY(succeededSoFar());
 
     if (m_testKit)
-        CppTools::ClangdSettings::setUseClangd(true);
+        ClangdSettings::setUseClangd(true);
 
     // Check if there are initial and target position markers
     TestDocumentPtr initialTestFile = testFileWithInitialCursorMarker(testFiles);
@@ -298,7 +296,7 @@ F2TestCase::F2TestCase(CppEditorAction action,
 
     const QString curTestName = QLatin1String(QTest::currentTestFunction());
     const QString tag = QLatin1String(QTest::currentDataTag());
-    const bool useClangd = CppTools::ClangdSettings::instance().useClangd();
+    const bool useClangd = ClangdSettings::instance().useClangd();
     if (useClangd) {
         if (curTestName == "testFollowSymbolQObjectConnect"
                 || curTestName == "testFollowSymbolQObjectOldStyleConnect") {
@@ -319,7 +317,7 @@ F2TestCase::F2TestCase(CppEditorAction action,
     }
 
     // Write files to disk
-    CppTools::Tests::TemporaryDir temporaryDir;
+    ::CppEditor::Tests::TemporaryDir temporaryDir;
     QVERIFY(temporaryDir.isValid());
     QString projectFileContent = "CppApplication { files: [";
     foreach (TestDocumentPtr testFile, testFiles) {
@@ -351,8 +349,8 @@ F2TestCase::F2TestCase(CppEditorAction action,
         openProjectResult.project()->configureAsExampleProject(m_testKit);
 
         // Wait until project is fully indexed.
-        QVERIFY(CppTools::Tests::waitForSignalOrTimeout(openProjectResult.project(),
-                &Project::indexingFinished, CppTools::Tests::clangdIndexingTimeout()));
+        QVERIFY(::CppEditor::Tests::waitForSignalOrTimeout(openProjectResult.project(),
+                &Project::indexingFinished, ::CppEditor::Tests::clangdIndexingTimeout()));
     }
 
     // Update Code Model
@@ -443,7 +441,7 @@ F2TestCase::F2TestCase(CppEditorAction action,
         QEXPECT_FAIL("infiniteLoopLocalTypedef_QTCREATORBUG-11999",
                      "clangd bug: Go to definition does not return", Abort);
         if (expectedVirtualFunctionProposal.size() <= 1) {
-            QVERIFY(CppTools::Tests::waitForSignalOrTimeout(EditorManager::instance(),
+            QVERIFY(::CppEditor::Tests::waitForSignalOrTimeout(EditorManager::instance(),
                                                             &EditorManager::linkOpened, 10000));
         } else {
             QTimer t;
@@ -489,7 +487,7 @@ F2TestCase::F2TestCase(CppEditorAction action,
 //    qDebug() << "Expected line:" << expectedLine;
 //    qDebug() << "Expected column:" << expectedColumn;
 
-    if (!CppTools::ClangdSettings::instance().useClangd()) {
+    if (!ClangdSettings::instance().useClangd()) {
         QEXPECT_FAIL("globalVarFromEnum", "Contributor works on a fix.", Abort);
         QEXPECT_FAIL("matchFunctionSignature_Follow_5", "foo(int) resolved as CallAST", Abort);
     }
-- 
cgit v1.2.3