};
return false;
};
+ auto InClassScope = [](const NamedDecl &ND) {
+ return ND.getDeclContext()->getDeclKind() == Decl::CXXRecord;
+ };
// We only complete symbol's name, which is the same as the name of the
// *primary* template in case of template specializations.
if (isExplicitTemplateSpecialization(&ND))
if (InTopLevelScope(ND))
return true;
+ // Always index enum constants, even if they're not in the top level scope:
+ // when
+ // --all-scopes-completion is set, we'll want to complete those as well.
if (const auto *EnumDecl = dyn_cast<clang::EnumDecl>(ND.getDeclContext()))
- return InTopLevelScope(*EnumDecl) && !EnumDecl->isScoped();
+ return (InTopLevelScope(*EnumDecl) || InClassScope(*EnumDecl)) && !EnumDecl->isScoped();
return false;
}
}
)cpp",
{cls("nx::Clangd1"), cls("ny::Clangd2"), cls("Clangd3"),
- cls("na::nb::Clangd4")},
+ cls("na::nb::Clangd4"), enmConstant("na::C::Clangd5")},
Opts);
EXPECT_THAT(
Results.Completions,
- UnorderedElementsAre(AllOf(qualifier("nx::"), named("Clangd1")),
- AllOf(qualifier("ny::"), named("Clangd2")),
- AllOf(qualifier(""), scope(""), named("Clangd3")),
- AllOf(qualifier("nb::"), named("Clangd4"))));
+ UnorderedElementsAre(AllOf(qualifier("nx::"), named("Clangd1"),
+ kind(CompletionItemKind::Class)),
+ AllOf(qualifier("ny::"), named("Clangd2"),
+ kind(CompletionItemKind::Class)),
+ AllOf(qualifier(""), scope(""), named("Clangd3"),
+ kind(CompletionItemKind::Class)),
+ AllOf(qualifier("nb::"), named("Clangd4"),
+ kind(CompletionItemKind::Class)),
+ AllOf(qualifier("C::"), named("Clangd5"),
+ kind(CompletionItemKind::EnumMember))));
}
TEST(CompletionTest, NoQualifierIfShadowed) {
kind(CompletionItemKind::Reference))));
}
+TEST(CompletionTest, Enums) {
+ const char *Header(R"cpp(
+ namespace ns {
+ enum Unscoped { Clangd1 };
+ class C {
+ enum Unscoped { Clangd2 };
+ };
+ enum class Scoped { Clangd3 };
+ })cpp");
+ const char *Source(R"cpp(
+ void bar() {
+ Clangd^
+ })cpp");
+ auto Index = TestTU::withHeaderCode(Header).index();
+ clangd::CodeCompleteOptions Opts;
+ Opts.Index = Index.get();
+ Opts.AllScopes = true;
+ auto R = completions(Source, {}, Opts);
+ EXPECT_THAT(R.Completions,
+ ElementsAre(AllOf(scope("ns::"), named("Clangd1"),
+ kind(CompletionItemKind::EnumMember)),
+ AllOf(scope("ns::C::"), named("Clangd2"),
+ kind(CompletionItemKind::EnumMember))));
+}
+
TEST(CompletionTest, ScopeIsUnresolved) {
clangd::CodeCompleteOptions Opts = {};
Opts.AllScopes = true;
Black
};
}
+ class Color3 {
+ enum {
+ Blue
+ };
+ };
)";
runSymbolCollector(Header, /*Main=*/"");
EXPECT_THAT(Symbols,
AllOf(qName("Color2"), forCodeCompletion(true)),
AllOf(qName("Color2::Yellow"), forCodeCompletion(false)),
AllOf(qName("ns"), forCodeCompletion(true)),
- AllOf(qName("ns::Black"), forCodeCompletion(true))));
+ AllOf(qName("ns::Black"), forCodeCompletion(true)),
+ AllOf(qName("Color3"), forCodeCompletion(true)),
+ AllOf(qName("Color3::Blue"), forCodeCompletion(true))));
}
TEST_F(SymbolCollectorTest, NamelessSymbols) {