123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362 |
- //===- NamespaceEndCommentsFixerTest.cpp - Formatting unit tests ----------===//
- //
- // The LLVM Compiler Infrastructure
- //
- // This file is distributed under the University of Illinois Open Source
- // License. See LICENSE.TXT for details.
- //
- //===----------------------------------------------------------------------===//
- #include "clang/Format/Format.h"
- #include "clang/Frontend/TextDiagnosticPrinter.h"
- #include "llvm/Support/Debug.h"
- #include "gtest/gtest.h"
- #define DEBUG_TYPE "namespace-end-comments-fixer-test"
- namespace clang {
- namespace format {
- namespace {
- class NamespaceEndCommentsFixerTest : public ::testing::Test {
- protected:
- std::string
- fixNamespaceEndComments(llvm::StringRef Code,
- std::vector<tooling::Range> Ranges,
- const FormatStyle &Style = getLLVMStyle()) {
- DEBUG(llvm::errs() << "---\n");
- DEBUG(llvm::errs() << Code << "\n\n");
- tooling::Replacements Replaces =
- clang::format::fixNamespaceEndComments(Style, Code, Ranges, "<stdin>");
- auto Result = applyAllReplacements(Code, Replaces);
- EXPECT_TRUE(static_cast<bool>(Result));
- DEBUG(llvm::errs() << "\n" << *Result << "\n\n");
- return *Result;
- }
- std::string
- fixNamespaceEndComments(llvm::StringRef Code,
- const FormatStyle &Style = getLLVMStyle()) {
- return fixNamespaceEndComments(
- Code,
- /*Ranges=*/{1, tooling::Range(0, Code.size())}, Style);
- }
- };
- TEST_F(NamespaceEndCommentsFixerTest, AddsEndComment) {
- EXPECT_EQ("namespace {\n"
- " int i;\n"
- "}// namespace",
- fixNamespaceEndComments("namespace {\n"
- " int i;\n"
- "}"));
- EXPECT_EQ("namespace {\n"
- " int i;\n"
- "}// namespace\n",
- fixNamespaceEndComments("namespace {\n"
- " int i;\n"
- "}\n"));
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "}// namespace A",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "}"));
- EXPECT_EQ("inline namespace A {\n"
- " int i;\n"
- "}// namespace A",
- fixNamespaceEndComments("inline namespace A {\n"
- " int i;\n"
- "}"));
- EXPECT_EQ("namespace ::A {\n"
- " int i;\n"
- "}// namespace ::A",
- fixNamespaceEndComments("namespace ::A {\n"
- " int i;\n"
- "}"));
- EXPECT_EQ("namespace ::A::B {\n"
- " int i;\n"
- "}// namespace ::A::B",
- fixNamespaceEndComments("namespace ::A::B {\n"
- " int i;\n"
- "}"));
- EXPECT_EQ("namespace /**/::/**/A/**/::/**/B/**/ {\n"
- " int i;\n"
- "}// namespace ::A::B",
- fixNamespaceEndComments("namespace /**/::/**/A/**/::/**/B/**/ {\n"
- " int i;\n"
- "}"));
- EXPECT_EQ("namespace A {\n"
- "namespace B {\n"
- " int i;\n"
- "}// namespace B\n"
- "}// namespace A",
- fixNamespaceEndComments("namespace A {\n"
- "namespace B {\n"
- " int i;\n"
- "}\n"
- "}"));
- EXPECT_EQ("namespace A {\n"
- " int a;\n"
- "}// namespace A\n"
- "namespace B {\n"
- " int b;\n"
- "}// namespace B",
- fixNamespaceEndComments("namespace A {\n"
- " int a;\n"
- "}\n"
- "namespace B {\n"
- " int b;\n"
- "}"));
- EXPECT_EQ("namespace A {\n"
- " int a1;\n"
- "}// namespace A\n"
- "namespace A {\n"
- " int a2;\n"
- "}// namespace A",
- fixNamespaceEndComments("namespace A {\n"
- " int a1;\n"
- "}\n"
- "namespace A {\n"
- " int a2;\n"
- "}"));
- EXPECT_EQ("namespace A {\n"
- " int a;\n"
- "}// namespace A\n"
- "// comment about b\n"
- "int b;",
- fixNamespaceEndComments("namespace A {\n"
- " int a;\n"
- "}\n"
- "// comment about b\n"
- "int b;"));
- EXPECT_EQ("namespace A {\n"
- "namespace B {\n"
- "namespace C {\n"
- "namespace D {\n"
- "}// namespace D\n"
- "}// namespace C\n"
- "}// namespace B\n"
- "}// namespace A",
- fixNamespaceEndComments("namespace A {\n"
- "namespace B {\n"
- "namespace C {\n"
- "namespace D {\n"
- "}\n"
- "}\n"
- "}\n"
- "}"));
- }
- TEST_F(NamespaceEndCommentsFixerTest, AddsNewlineIfNeeded) {
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "}// namespace A\n"
- " int j;",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "} int j;"));
- EXPECT_EQ("namespace {\n"
- " int i;\n"
- "}// namespace\n"
- " int j;",
- fixNamespaceEndComments("namespace {\n"
- " int i;\n"
- "} int j;"));
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "}// namespace A\n"
- " namespace B {\n"
- " int j;\n"
- "}// namespace B",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "} namespace B {\n"
- " int j;\n"
- "}"));
- }
- TEST_F(NamespaceEndCommentsFixerTest, DoesNotAddEndCommentForShortNamespace) {
- EXPECT_EQ("namespace {}", fixNamespaceEndComments("namespace {}"));
- EXPECT_EQ("namespace A {}", fixNamespaceEndComments("namespace A {}"));
- EXPECT_EQ("namespace A { int i; }",
- fixNamespaceEndComments("namespace A { int i; }"));
- }
- TEST_F(NamespaceEndCommentsFixerTest, DoesNotAddCommentAfterUnaffectedRBrace) {
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "}",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "}",
- // The range (16, 3) spans the 'int' above.
- /*Ranges=*/{1, tooling::Range(16, 3)}));
- }
- TEST_F(NamespaceEndCommentsFixerTest, DoesNotAddCommentAfterRBraceInPPDirective) {
- EXPECT_EQ("#define SAD \\\n"
- "namespace A { \\\n"
- " int i; \\\n"
- "}",
- fixNamespaceEndComments("#define SAD \\\n"
- "namespace A { \\\n"
- " int i; \\\n"
- "}"));
- }
- TEST_F(NamespaceEndCommentsFixerTest, KeepsValidEndComment) {
- EXPECT_EQ("namespace {\n"
- " int i;\n"
- "} // end anonymous namespace",
- fixNamespaceEndComments("namespace {\n"
- " int i;\n"
- "} // end anonymous namespace"));
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "} /* end of namespace A */",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "} /* end of namespace A */"));
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "} // namespace A",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "} // namespace A"));
- EXPECT_EQ("namespace A::B {\n"
- " int i;\n"
- "} // end namespace A::B",
- fixNamespaceEndComments("namespace A::B {\n"
- " int i;\n"
- "} // end namespace A::B"));
- }
- TEST_F(NamespaceEndCommentsFixerTest, UpdatesInvalidEndLineComment) {
- EXPECT_EQ("namespace {\n"
- " int i;\n"
- "} // namespace",
- fixNamespaceEndComments("namespace {\n"
- " int i;\n"
- "} // namespace A"));
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "} // namespace A",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "} // namespace"));
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "} // namespace A",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "} //"));
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "} // namespace A",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "} //"));
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "} // namespace A",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "} // banamespace A"));
- // Updates invalid line comments even for short namespaces.
- EXPECT_EQ("namespace A {} // namespace A",
- fixNamespaceEndComments("namespace A {} // namespace"));
- }
- TEST_F(NamespaceEndCommentsFixerTest, UpdatesInvalidEndBlockComment) {
- EXPECT_EQ("namespace {\n"
- " int i;\n"
- "} // namespace",
- fixNamespaceEndComments("namespace {\n"
- " int i;\n"
- "} /* namespace A */"));
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "} // namespace A",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "} /* end namespace */"));
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "} // namespace A",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "} /**/"));
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "} // namespace A",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "} /* end unnamed namespace */"));
- EXPECT_EQ("namespace A {\n"
- " int i;\n"
- "} // namespace A",
- fixNamespaceEndComments("namespace A {\n"
- " int i;\n"
- "} /* banamespace A */"));
- EXPECT_EQ("namespace A {} // namespace A",
- fixNamespaceEndComments("namespace A {} /**/"));
- }
- TEST_F(NamespaceEndCommentsFixerTest,
- DoesNotAddEndCommentForNamespacesControlledByMacros) {
- EXPECT_EQ("#ifdef 1\n"
- "namespace A {\n"
- "#elseif\n"
- "namespace B {\n"
- "#endif\n"
- " int i;\n"
- "}\n"
- "}\n",
- fixNamespaceEndComments("#ifdef 1\n"
- "namespace A {\n"
- "#elseif\n"
- "namespace B {\n"
- "#endif\n"
- " int i;\n"
- "}\n"
- "}\n"));
- }
- TEST_F(NamespaceEndCommentsFixerTest,
- DoesNotAddEndCommentForNamespacesInMacroDeclarations) {
- EXPECT_EQ("#ifdef 1\n"
- "namespace A {\n"
- "#elseif\n"
- "namespace B {\n"
- "#endif\n"
- " int i;\n"
- "}\n"
- "}\n",
- fixNamespaceEndComments("#ifdef 1\n"
- "namespace A {\n"
- "#elseif\n"
- "namespace B {\n"
- "#endif\n"
- " int i;\n"
- "}\n"
- "}\n"));
- }
- TEST_F(NamespaceEndCommentsFixerTest,
- DoesNotAddEndCommentForUnbalancedRBracesAfterNamespaceEnd) {
- EXPECT_EQ("namespace {\n"
- " int i;\n"
- "} // namespace\n"
- "}",
- fixNamespaceEndComments("namespace {\n"
- " int i;\n"
- "} // namespace\n"
- "}"));
- }
- } // end namespace
- } // end namespace format
- } // end namespace clang
|