Commit 0f271e25 authored by Milian Wolff's avatar Milian Wolff

Remove expected failure for test passing with clang 9

The upstream bug isn't resolved, and its behavior is still
broken, yet the test case works properly for us now. Potentially
this is related to some of the other options we are setting.
KeepGoing alone doesn't seem to do the trick though.
parent 84952f49
......@@ -51,6 +51,7 @@
#include "duchain/clangparsingenvironmentfile.h"
#include "duchain/clangparsingenvironment.h"
#include "duchain/parsesession.h"
#include "duchain/clanghelpers.h"
#include "testprovider.h"
......@@ -60,6 +61,7 @@
#include <QSignalSpy>
#include <QLoggingCategory>
#include <QThread>
#include <QVersionNumber>
QTEST_MAIN(TestDUChain)
......@@ -301,7 +303,8 @@ class B : public A<int>
// clang stops processing when it encounters the second missing
// header, or similar.
// XFAIL this check until https://bugs.llvm.org/show_bug.cgi?id=38155 is fixed
QEXPECT_FAIL("", "Base class isn't assigned correctly", Continue);
if (QVersionNumber::fromString(ClangHelpers::clangVersion()) < QVersionNumber(9, 0, 0))
QEXPECT_FAIL("", "Base class isn't assigned correctly", Continue);
QCOMPARE(b->baseClassesSize(), 1u);
#if CINDEX_VERSION_MINOR < 34
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment