From 58f0f3928c1284a687c9ef38f0276473518a1567 Mon Sep 17 00:00:00 2001 From: Christian Kandeler Date: Thu, 1 Jul 2021 11:16:38 +0200 Subject: [PATCH] Clangd: Fix tooltip test - We require the "full AST" feature from clangd 13. - QSKIP does not work like QEXPECT_FAIL. Change-Id: Ie1c417013d668ace600fad76e83276ca961b8327 Reviewed-by: David Schulz --- src/plugins/clangcodemodel/test/clangdtests.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/plugins/clangcodemodel/test/clangdtests.cpp b/src/plugins/clangcodemodel/test/clangdtests.cpp index 2d5fb510d3f..7772fec131e 100644 --- a/src/plugins/clangcodemodel/test/clangdtests.cpp +++ b/src/plugins/clangcodemodel/test/clangdtests.cpp @@ -499,6 +499,7 @@ ClangdTestTooltips::ClangdTestTooltips() { setProjectFileName("tooltips.pro"); setSourceFileNames({"tooltips.cpp"}); + setMinimumVersion(13); } void ClangdTestTooltips::test_data() @@ -600,7 +601,8 @@ void ClangdTestTooltips::test() QCOMPARE(editor->document(), doc); QVERIFY(editor->editorWidget()); - QSKIP("IncludeDirective", "FIXME: clangd sends empty or no hover data for includes", Abort); + if (QLatin1String(QTest::currentDataTag()) == "IncludeDirective") + QSKIP("FIXME: clangd sends empty or no hover data for includes"); QTimer timer; timer.setSingleShot(true);