| //===-- ChangeNamespaceTests.cpp - Change namespace unit tests ---*- C++ -*-===// |
| // |
| // The LLVM Compiler Infrastructure |
| // |
| // This file is distributed under the University of Illinois Open Source |
| // License. See LICENSE.TXT for details. |
| // |
| //===----------------------------------------------------------------------===// |
| |
| #include "ChangeNamespace.h" |
| #include "unittests/Tooling/RewriterTestContext.h" |
| #include "clang/ASTMatchers/ASTMatchFinder.h" |
| #include "clang/Basic/FileManager.h" |
| #include "clang/Basic/FileSystemOptions.h" |
| #include "clang/Basic/VirtualFileSystem.h" |
| #include "clang/Format/Format.h" |
| #include "clang/Frontend/CompilerInstance.h" |
| #include "clang/Frontend/PCHContainerOperations.h" |
| #include "clang/Tooling/Refactoring.h" |
| #include "clang/Tooling/Tooling.h" |
| #include "llvm/ADT/IntrusiveRefCntPtr.h" |
| #include "llvm/ADT/StringRef.h" |
| #include "llvm/Support/MemoryBuffer.h" |
| #include "gtest/gtest.h" |
| #include <memory> |
| #include <string> |
| #include <vector> |
| |
| namespace clang { |
| namespace change_namespace { |
| namespace { |
| |
| class ChangeNamespaceTest : public ::testing::Test { |
| public: |
| std::string runChangeNamespaceOnCode(llvm::StringRef Code) { |
| clang::RewriterTestContext Context; |
| clang::FileID ID = Context.createInMemoryFile(FileName, Code); |
| |
| std::map<std::string, tooling::Replacements> FileToReplacements; |
| change_namespace::ChangeNamespaceTool NamespaceTool( |
| OldNamespace, NewNamespace, FilePattern, &FileToReplacements); |
| ast_matchers::MatchFinder Finder; |
| NamespaceTool.registerMatchers(&Finder); |
| std::unique_ptr<tooling::FrontendActionFactory> Factory = |
| tooling::newFrontendActionFactory(&Finder); |
| tooling::runToolOnCodeWithArgs(Factory->create(), Code, {"-std=c++11"}, |
| FileName); |
| formatAndApplyAllReplacements(FileToReplacements, Context.Rewrite); |
| return format(Context.getRewrittenText(ID)); |
| } |
| |
| std::string format(llvm::StringRef Code) { |
| tooling::Replacements Replaces = format::reformat( |
| format::getLLVMStyle(), Code, {tooling::Range(0, Code.size())}); |
| auto ChangedCode = tooling::applyAllReplacements(Code, Replaces); |
| EXPECT_TRUE(static_cast<bool>(ChangedCode)); |
| if (!ChangedCode) { |
| llvm::errs() << llvm::toString(ChangedCode.takeError()); |
| return ""; |
| } |
| return *ChangedCode; |
| } |
| |
| protected: |
| std::string FileName = "input.cc"; |
| std::string OldNamespace = "na::nb"; |
| std::string NewNamespace = "x::y"; |
| std::string FilePattern = "input.cc"; |
| }; |
| |
| TEST_F(ChangeNamespaceTest, NoMatchingNamespace) { |
| std::string Code = "namespace na {\n" |
| "namespace nx {\n" |
| "class A {};\n" |
| "} // namespace nx\n" |
| "} // namespace na\n"; |
| std::string Expected = "namespace na {\n" |
| "namespace nx {\n" |
| "class A {};\n" |
| "} // namespace nx\n" |
| "} // namespace na\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, SimpleMoveWithoutTypeRefs) { |
| std::string Code = "namespace na {\n" |
| "namespace nb {\n" |
| "class A {};\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| std::string Expected = "\n\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "class A {};\n" |
| "} // namespace y\n" |
| "} // namespace x\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, SimpleMoveIntoAnotherNestedNamespace) { |
| NewNamespace = "na::nc"; |
| std::string Code = "namespace na {\n" |
| "namespace nb {\n" |
| "class A {};\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| std::string Expected = "namespace na {\n" |
| "\n" |
| "namespace nc {\n" |
| "class A {};\n" |
| "} // namespace nc\n" |
| "} // namespace na\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, SimpleMoveNestedNamespace) { |
| NewNamespace = "na::x::y"; |
| std::string Code = "namespace na {\n" |
| "class A {};\n" |
| "namespace nb {\n" |
| "class B {};\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| std::string Expected = "namespace na {\n" |
| "class A {};\n" |
| "\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "class B {};\n" |
| "} // namespace y\n" |
| "} // namespace x\n" |
| "} // namespace na\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, SimpleMoveWithTypeRefs) { |
| std::string Code = "namespace na {\n" |
| "class C_A {};\n" |
| "namespace nc {\n" |
| "class C_C {};" |
| "} // namespace nc\n" |
| "namespace nb {\n" |
| "class C_X {\n" |
| "public:\n" |
| " C_A a;\n" |
| " nc::C_C c;\n" |
| "};\n" |
| "class C_Y {\n" |
| " C_X x;\n" |
| "};\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| std::string Expected = "namespace na {\n" |
| "class C_A {};\n" |
| "namespace nc {\n" |
| "class C_C {};" |
| "} // namespace nc\n" |
| "\n" |
| "} // namespace na\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "class C_X {\n" |
| "public:\n" |
| " na::C_A a;\n" |
| " na::nc::C_C c;\n" |
| "};\n" |
| "class C_Y {\n" |
| " C_X x;\n" |
| "};\n" |
| "} // namespace y\n" |
| "} // namespace x\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, LeaveForwardDeclarationBehind) { |
| std::string Code = "namespace na {\n" |
| "namespace nb {\n" |
| "class FWD;\n" |
| "class A {\n" |
| " FWD *fwd;\n" |
| "};\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| std::string Expected = "namespace na {\n" |
| "namespace nb {\n" |
| "class FWD;\n" |
| "} // namespace nb\n" |
| "} // namespace na\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "\n" |
| "class A {\n" |
| " na::nb::FWD *fwd;\n" |
| "};\n" |
| "} // namespace y\n" |
| "} // namespace x\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, MoveFunctions) { |
| std::string Code = "namespace na {\n" |
| "class C_A {};\n" |
| "namespace nc {\n" |
| "class C_C {};" |
| "} // namespace nc\n" |
| "namespace nb {\n" |
| "void fwd();\n" |
| "void f(C_A ca, nc::C_C cc) {\n" |
| " C_A ca_1 = ca;\n" |
| "}\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| |
| std::string Expected = "namespace na {\n" |
| "class C_A {};\n" |
| "namespace nc {\n" |
| "class C_C {};" |
| "} // namespace nc\n" |
| "\n" |
| "} // namespace na\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "void fwd();\n" |
| "void f(na::C_A ca, na::nc::C_C cc) {\n" |
| " na::C_A ca_1 = ca;\n" |
| "}\n" |
| "} // namespace y\n" |
| "} // namespace x\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, DoNotCrashWithLambdaAsParameter) { |
| std::string Code = |
| "#include <functional>\n" |
| "void f(std::function<void(int)> func, int param) { func(param); } " |
| "void g() { f([](int x) {}, 1); }"; |
| |
| std::string Expected = |
| "#include <functional>\n" |
| "void f(std::function<void(int)> func, int param) { func(param); } " |
| "void g() { f([](int x) {}, 1); }"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, FixUsingShadowDecl) { |
| std::string Code = "class GLOB {};\n" |
| "using BLOG = GLOB;\n" |
| "namespace na {\n" |
| "namespace nc {\n" |
| "class SAME {};\n" |
| "}\n" |
| "namespace nd {\n" |
| "class SAME {};\n" |
| "}\n" |
| "namespace nb {\n" |
| "using nc::SAME;\n" |
| "using YO = nc::SAME;\n" |
| "typedef nc::SAME IDENTICAL;\n" |
| "void f(nd::SAME Same) {}\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| |
| std::string Expected = "class GLOB {};\n" |
| "using BLOG = GLOB;\n" |
| "namespace na {\n" |
| "namespace nc {\n" |
| "class SAME {};\n" |
| "}\n" |
| "namespace nd {\n" |
| "class SAME {};\n" |
| "}\n" |
| "\n" |
| "} // namespace na\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "using ::na::nc::SAME;\n" |
| "using YO = na::nc::SAME;\n" |
| "typedef na::nc::SAME IDENTICAL;\n" |
| "void f(na::nd::SAME Same) {}\n" |
| "} // namespace y\n" |
| "} // namespace x\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, UsingShadowDeclInFunction) { |
| std::string Code = "namespace glob {\n" |
| "class Glob {};\n" |
| "}\n" |
| "namespace na {\n" |
| "namespace nb {\n" |
| "void f() {\n" |
| " using glob::Glob;\n" |
| " Glob g;\n" |
| "}\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| |
| // FIXME: don't add namespace qualifier when there is UsingShadowDecl. |
| std::string Expected = "namespace glob {\n" |
| "class Glob {};\n" |
| "}\n" |
| "\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "void f() {\n" |
| " using ::glob::Glob;\n" |
| " glob::Glob g;\n" |
| "}\n" |
| "} // namespace y\n" |
| "} // namespace x\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, UsingShadowDeclInGlobal) { |
| std::string Code = "namespace glob {\n" |
| "class Glob {};\n" |
| "}\n" |
| "using glob::Glob;\n" |
| "namespace na {\n" |
| "namespace nb {\n" |
| "void f() { Glob g; }\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| |
| // FIXME: don't add namespace qualifier when there is UsingShadowDecl. |
| std::string Expected = "namespace glob {\n" |
| "class Glob {};\n" |
| "}\n" |
| "using glob::Glob;\n" |
| "\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "void f() { glob::Glob g; }\n" |
| "} // namespace y\n" |
| "} // namespace x\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, UsingNamespace) { |
| std::string Code = "namespace glob {\n" |
| "class Glob {};\n" |
| "}\n" |
| "using namespace glob;\n" |
| "namespace na {\n" |
| "namespace nb {\n" |
| "void f() { Glob g; }\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| |
| // FIXME: don't add namespace qualifier when there is "using namespace" decl. |
| std::string Expected = "namespace glob {\n" |
| "class Glob {};\n" |
| "}\n" |
| "using namespace glob;\n" |
| "\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "void f() { glob::Glob g; }\n" |
| "} // namespace y\n" |
| "} // namespace x\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, TypeInNestedNameSpecifier) { |
| std::string Code = |
| "namespace na {\n" |
| "class C_A {\n" |
| "public:\n" |
| " class Nested {\n" |
| " public:\n" |
| " static int NestedX;\n" |
| " static void nestedFunc() {}\n" |
| " };\n" |
| "};\n" |
| "namespace nb {\n" |
| "class C_X {\n" |
| " C_A na;\n" |
| " C_A::Nested nested;\n" |
| " void f() {\n" |
| " C_A::Nested::nestedFunc();\n" |
| " int X = C_A::Nested::NestedX;\n" |
| " }\n" |
| "};\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| std::string Expected = |
| "namespace na {\n" |
| "class C_A {\n" |
| "public:\n" |
| " class Nested {\n" |
| " public:\n" |
| " static int NestedX;\n" |
| " static void nestedFunc() {}\n" |
| " };\n" |
| "};\n" |
| "\n" |
| "} // namespace na\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "class C_X {\n" |
| " na::C_A na;\n" |
| " na::C_A::Nested nested;\n" |
| " void f() {\n" |
| " na::C_A::Nested::nestedFunc();\n" |
| " int X = na::C_A::Nested::NestedX;\n" |
| " }\n" |
| "};\n" |
| "} // namespace y\n" |
| "} // namespace x\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, FixFunctionNameSpecifiers) { |
| std::string Code = |
| "namespace na {\n" |
| "class A {\n" |
| "public:\n" |
| " static void f() {}\n" |
| " static void g();\n" |
| "};\n" |
| "void A::g() {}" |
| "void a_f() {}\n" |
| "static void static_f() {}\n" |
| "namespace nb {\n" |
| "void f() { a_f(); static_f(); A::f(); }\n" |
| "void g() { f(); A::g(); }\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| std::string Expected = |
| "namespace na {\n" |
| "class A {\n" |
| "public:\n" |
| " static void f() {}\n" |
| " static void g();\n" |
| "};\n" |
| "void A::g() {}" |
| "void a_f() {}\n" |
| "static void static_f() {}\n" |
| "\n" |
| "} // namespace na\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "void f() { na::a_f(); na::static_f(); na::A::f(); }\n" |
| "void g() { f(); na::A::g(); }\n" |
| "} // namespace y\n" |
| "} // namespace x\n"; |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, MoveAndFixGlobalVariables) { |
| std::string Code = "namespace na {\n" |
| "int GlobA;\n" |
| "static int GlobAStatic = 0;\n" |
| "namespace nc { int GlobC; }\n" |
| "namespace nb {\n" |
| "int GlobB;\n" |
| "void f() {\n" |
| " int a = GlobA;\n" |
| " int b = GlobAStatic;\n" |
| " int c = nc::GlobC;\n" |
| "}\n" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| |
| std::string Expected = "namespace na {\n" |
| "int GlobA;\n" |
| "static int GlobAStatic = 0;\n" |
| "namespace nc { int GlobC; }\n" |
| "\n" |
| "} // namespace na\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "int GlobB;\n" |
| "void f() {\n" |
| " int a = na::GlobA;\n" |
| " int b = na::GlobAStatic;\n" |
| " int c = na::nc::GlobC;\n" |
| "}\n" |
| "} // namespace y\n" |
| "} // namespace x\n"; |
| |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| TEST_F(ChangeNamespaceTest, DoNotFixStaticVariableOfClass) { |
| std::string Code = "namespace na {\n" |
| "class A {\n" |
| "public:\n" |
| "static int A1;\n" |
| "static int A2;\n" |
| "}\n" |
| "static int A::A1 = 0;\n" |
| "namespace nb {\n" |
| "void f() { int a = A::A1; int b = A::A2; }" |
| "} // namespace nb\n" |
| "} // namespace na\n"; |
| |
| std::string Expected = "namespace na {\n" |
| "class A {\n" |
| "public:\n" |
| "static int A1;\n" |
| "static int A2;\n" |
| "}\n" |
| "static int A::A1 = 0;\n" |
| "\n" |
| "} // namespace na\n" |
| "namespace x {\n" |
| "namespace y {\n" |
| "void f() { int a = na::A::A1; int b = na::A::A2; }" |
| "} // namespace y\n" |
| "} // namespace x\n"; |
| |
| EXPECT_EQ(format(Expected), runChangeNamespaceOnCode(Code)); |
| } |
| |
| } // anonymous namespace |
| } // namespace change_namespace |
| } // namespace clang |