From 41fee90d8eded9f6ac7c172e9bdc05cf8e9af9d0 Mon Sep 17 00:00:00 2001 From: Christian Kandeler Date: Mon, 31 Jan 2022 16:00:39 +0100 Subject: [PATCH] CppEditor: Fix testing Now that clangd is enabled by default, we must explicitly turn it off to test the built-in follow symbol functionality. Change-Id: Ib275893a920145868b314ce02e9dd80f656bade4 Reviewed-by: Reviewed-by: Christian Stenger --- .../cppeditor/followsymbol_switchmethoddecldef_test.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp b/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp index 81b410a3cda..63f16e98096 100644 --- a/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp +++ b/src/plugins/cppeditor/followsymbol_switchmethoddecldef_test.cpp @@ -251,7 +251,7 @@ F2TestCase::F2TestCase(CppEditorAction action, const QString curTestName = QLatin1String(QTest::currentTestFunction()); const QString tag = QLatin1String(QTest::currentDataTag()); - const bool useClangd = ClangdSettings::instance().useClangd(); + const bool useClangd = m_testKit; if (useClangd) { if (curTestName == "testFollowSymbolQObjectConnect" || curTestName == "testFollowSymbolQObjectOldStyleConnect") { @@ -442,7 +442,7 @@ F2TestCase::F2TestCase(CppEditorAction action, // qDebug() << "Expected line:" << expectedLine; // qDebug() << "Expected column:" << expectedColumn; - if (!ClangdSettings::instance().useClangd()) { + if (!useClangd) { QEXPECT_FAIL("globalVarFromEnum", "Contributor works on a fix.", Abort); QEXPECT_FAIL("matchFunctionSignature_Follow_5", "foo(int) resolved as CallAST", Abort); }