Revert "[clangd][test] Disable a particular testcase in FindExplicitReferencesTest...
authorKadir Cetinkaya <kadircet@google.com>
Fri, 17 Jan 2020 07:51:13 +0000 (08:51 +0100)
committerKadir Cetinkaya <kadircet@google.com>
Fri, 17 Jan 2020 07:51:13 +0000 (08:51 +0100)
This reverts commit 42b3c38903c9c80e129ce678db7f522967e91eeb.

Should've been fixed by d54d71b67e60

clang-tools-extra/clangd/unittests/FindTargetTests.cpp

index c96b3a572322bebf49a8855609fa2d33f27d90c6..408ebe24e773643f179ce796bde1f461ce1399b0 100644 (file)
@@ -770,10 +770,6 @@ TEST_F(FindExplicitReferencesTest, All) {
         "1: targets = {vector}\n"
         "2: targets = {x}\n"},
        // Handle UnresolvedLookupExpr.
-       // FIXME
-       // This case fails when expensive checks are enabled.
-       // Seems like the order of ns1::func and ns2::func isn't defined.
-       #ifndef EXPENSIVE_CHECKS
        {R"cpp(
             namespace ns1 { void func(char*); }
             namespace ns2 { void func(int*); }
@@ -787,7 +783,6 @@ TEST_F(FindExplicitReferencesTest, All) {
         )cpp",
         "0: targets = {ns1::func, ns2::func}\n"
         "1: targets = {t}\n"},
-        #endif
        // Handle UnresolvedMemberExpr.
        {R"cpp(
             struct X {