diff options
author | Christian Kandeler <christian.kandeler@qt.io> | 2021-08-30 10:58:08 +0200 |
---|---|---|
committer | Christian Kandeler <christian.kandeler@qt.io> | 2021-09-01 14:53:58 +0000 |
commit | 284817fae6514701902ccdb834c2faa46462f2e8 (patch) | |
tree | 44a8c7d9813dc110b61c4639036366c7696bd7e9 /src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp | |
parent | 3e1fa0f170d523971d2c3c12da15a6e291f56511 (diff) |
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>
Diffstat (limited to 'src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp')
-rw-r--r-- | src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp | 36 |
1 files changed, 17 insertions, 19 deletions
diff --git a/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp b/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp index 10c78edc8a..f005848b83 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); } |