Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 1 | //===-- ClangMoveTest.cpp - clang-move unit tests -------------------------===// |
| 2 | // |
| 3 | // The LLVM Compiler Infrastructure |
| 4 | // |
| 5 | // This file is distributed under the University of Illinois Open Source |
| 6 | // License. See LICENSE.TXT for details. |
| 7 | // |
| 8 | //===----------------------------------------------------------------------===// |
| 9 | |
| 10 | #include "ClangMove.h" |
| 11 | #include "unittests/Tooling/RewriterTestContext.h" |
| 12 | #include "clang/Format/Format.h" |
| 13 | #include "clang/Frontend/FrontendActions.h" |
| 14 | #include "clang/Frontend/TextDiagnosticPrinter.h" |
| 15 | #include "clang/Rewrite/Core/Rewriter.h" |
| 16 | #include "clang/Tooling/Refactoring.h" |
| 17 | #include "clang/Tooling/Tooling.h" |
| 18 | #include "llvm/ADT/StringRef.h" |
| 19 | #include "gtest/gtest.h" |
| 20 | #include <string> |
| 21 | #include <vector> |
| 22 | |
| 23 | namespace clang { |
| 24 | namespace move { |
| 25 | namespace { |
| 26 | |
| 27 | const char TestHeaderName[] = "foo.h"; |
| 28 | |
| 29 | const char TestCCName[] = "foo.cc"; |
| 30 | |
| 31 | const char TestHeader[] = "namespace a {\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 32 | "class C1; // test\n" |
Haojian Wu | 29c38f7 | 2016-10-21 19:26:43 +0000 | [diff] [blame] | 33 | "template <typename T> class C2;\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 34 | "namespace b {\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 35 | "// This is a Foo class\n" |
| 36 | "// which is used in\n" |
| 37 | "// test.\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 38 | "class Foo {\n" |
| 39 | "public:\n" |
| 40 | " void f();\n" |
| 41 | "\n" |
| 42 | "private:\n" |
| 43 | " C1 *c1;\n" |
| 44 | " static int b;\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 45 | "}; // abc\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 46 | "\n" |
| 47 | "class Foo2 {\n" |
| 48 | "public:\n" |
| 49 | " int f();\n" |
| 50 | "};\n" |
| 51 | "} // namespace b\n" |
| 52 | "} // namespace a\n"; |
| 53 | |
| 54 | const char TestCC[] = "#include \"foo.h\"\n" |
| 55 | "namespace a {\n" |
| 56 | "namespace b {\n" |
| 57 | "namespace {\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 58 | "// comment1.\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 59 | "void f1() {}\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 60 | "/// comment2.\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 61 | "int kConstInt1 = 0;\n" |
| 62 | "} // namespace\n" |
| 63 | "\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 64 | "/* comment 3*/\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 65 | "static int kConstInt2 = 1;\n" |
| 66 | "\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 67 | "/** comment4\n" |
| 68 | "*/\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 69 | "static int help() {\n" |
| 70 | " int a = 0;\n" |
| 71 | " return a;\n" |
| 72 | "}\n" |
| 73 | "\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 74 | "// comment5\n" |
| 75 | "// comment5\n" |
Haojian Wu | 3626516 | 2017-01-03 09:00:51 +0000 | [diff] [blame] | 76 | "void Foo::f() {\n" |
| 77 | " f1();\n" |
| 78 | " kConstInt1;\n" |
| 79 | " kConstInt2;\n" |
| 80 | " help();\n" |
| 81 | "}\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 82 | "\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 83 | "/////////////\n" |
| 84 | "// comment //\n" |
| 85 | "/////////////\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 86 | "int Foo::b = 2;\n" |
| 87 | "int Foo2::f() {\n" |
Haojian Wu | 3626516 | 2017-01-03 09:00:51 +0000 | [diff] [blame] | 88 | " kConstInt1;\n" |
| 89 | " kConstInt2;\n" |
| 90 | " help();\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 91 | " f1();\n" |
| 92 | " return 1;\n" |
| 93 | "}\n" |
| 94 | "} // namespace b\n" |
| 95 | "} // namespace a\n"; |
| 96 | |
| 97 | const char ExpectedTestHeader[] = "namespace a {\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 98 | "class C1; // test\n" |
Haojian Wu | 29c38f7 | 2016-10-21 19:26:43 +0000 | [diff] [blame] | 99 | "template <typename T> class C2;\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 100 | "namespace b {\n" |
| 101 | "\n" |
| 102 | "class Foo2 {\n" |
| 103 | "public:\n" |
| 104 | " int f();\n" |
| 105 | "};\n" |
| 106 | "} // namespace b\n" |
| 107 | "} // namespace a\n"; |
| 108 | |
| 109 | const char ExpectedTestCC[] = "#include \"foo.h\"\n" |
| 110 | "namespace a {\n" |
| 111 | "namespace b {\n" |
| 112 | "namespace {\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 113 | "// comment1.\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 114 | "void f1() {}\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 115 | "/// comment2.\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 116 | "int kConstInt1 = 0;\n" |
| 117 | "} // namespace\n" |
| 118 | "\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 119 | "/* comment 3*/\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 120 | "static int kConstInt2 = 1;\n" |
| 121 | "\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 122 | "/** comment4\n" |
| 123 | "*/\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 124 | "static int help() {\n" |
| 125 | " int a = 0;\n" |
| 126 | " return a;\n" |
| 127 | "}\n" |
| 128 | "\n" |
| 129 | "int Foo2::f() {\n" |
Haojian Wu | 3626516 | 2017-01-03 09:00:51 +0000 | [diff] [blame] | 130 | " kConstInt1;\n" |
| 131 | " kConstInt2;\n" |
| 132 | " help();\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 133 | " f1();\n" |
| 134 | " return 1;\n" |
| 135 | "}\n" |
| 136 | "} // namespace b\n" |
| 137 | "} // namespace a\n"; |
| 138 | |
Haojian Wu | 220c755 | 2016-10-14 13:01:36 +0000 | [diff] [blame] | 139 | const char ExpectedNewHeader[] = "#ifndef NEW_FOO_H\n" |
| 140 | "#define NEW_FOO_H\n" |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 141 | "\n" |
Haojian Wu | 220c755 | 2016-10-14 13:01:36 +0000 | [diff] [blame] | 142 | "namespace a {\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 143 | "class C1; // test\n" |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 144 | "\n" |
Haojian Wu | 29c38f7 | 2016-10-21 19:26:43 +0000 | [diff] [blame] | 145 | "template <typename T> class C2;\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 146 | "namespace b {\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 147 | "// This is a Foo class\n" |
| 148 | "// which is used in\n" |
| 149 | "// test.\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 150 | "class Foo {\n" |
| 151 | "public:\n" |
| 152 | " void f();\n" |
| 153 | "\n" |
| 154 | "private:\n" |
| 155 | " C1 *c1;\n" |
| 156 | " static int b;\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 157 | "}; // abc\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 158 | "} // namespace b\n" |
Haojian Wu | 220c755 | 2016-10-14 13:01:36 +0000 | [diff] [blame] | 159 | "} // namespace a\n" |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 160 | "\n" |
Haojian Wu | 220c755 | 2016-10-14 13:01:36 +0000 | [diff] [blame] | 161 | "#endif // NEW_FOO_H\n"; |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 162 | |
Haojian Wu | daf4cb8 | 2016-09-23 13:28:38 +0000 | [diff] [blame] | 163 | const char ExpectedNewCC[] = "namespace a {\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 164 | "namespace b {\n" |
| 165 | "namespace {\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 166 | "// comment1.\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 167 | "void f1() {}\n" |
Haojian Wu | 3626516 | 2017-01-03 09:00:51 +0000 | [diff] [blame] | 168 | "\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 169 | "/// comment2.\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 170 | "int kConstInt1 = 0;\n" |
| 171 | "} // namespace\n" |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 172 | "\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 173 | "/* comment 3*/\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 174 | "static int kConstInt2 = 1;\n" |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 175 | "\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 176 | "/** comment4\n" |
| 177 | "*/\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 178 | "static int help() {\n" |
| 179 | " int a = 0;\n" |
| 180 | " return a;\n" |
| 181 | "}\n" |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 182 | "\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 183 | "// comment5\n" |
| 184 | "// comment5\n" |
Haojian Wu | 3626516 | 2017-01-03 09:00:51 +0000 | [diff] [blame] | 185 | "void Foo::f() {\n" |
| 186 | " f1();\n" |
| 187 | " kConstInt1;\n" |
| 188 | " kConstInt2;\n" |
| 189 | " help();\n" |
| 190 | "}\n" |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 191 | "\n" |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 192 | "/////////////\n" |
| 193 | "// comment //\n" |
| 194 | "/////////////\n" |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 195 | "int Foo::b = 2;\n" |
| 196 | "} // namespace b\n" |
| 197 | "} // namespace a\n"; |
| 198 | |
| 199 | std::map<std::string, std::string> |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 200 | runClangMoveOnCode(const move::MoveDefinitionSpec &Spec, |
Haojian Wu | 2930be1 | 2016-11-08 19:55:13 +0000 | [diff] [blame] | 201 | const char *const Header = TestHeader, |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 202 | const char *const CC = TestCC, |
| 203 | DeclarationReporter *const Reporter = nullptr) { |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 204 | clang::RewriterTestContext Context; |
| 205 | |
| 206 | std::map<llvm::StringRef, clang::FileID> FileToFileID; |
| 207 | std::vector<std::pair<std::string, std::string>> FileToSourceText = { |
Haojian Wu | 2930be1 | 2016-11-08 19:55:13 +0000 | [diff] [blame] | 208 | {TestHeaderName, Header}, {TestCCName, CC}}; |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 209 | |
Malcolm Parsons | e65b102 | 2017-01-13 19:02:50 +0000 | [diff] [blame] | 210 | auto CreateFiles = [&Context, &FileToFileID](llvm::StringRef Name, |
| 211 | llvm::StringRef Code) { |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 212 | if (!Name.empty()) { |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 213 | FileToFileID[Name] = Context.createInMemoryFile(Name, Code); |
| 214 | } |
| 215 | }; |
| 216 | CreateFiles(Spec.NewCC, ""); |
| 217 | CreateFiles(Spec.NewHeader, ""); |
Haojian Wu | 2930be1 | 2016-11-08 19:55:13 +0000 | [diff] [blame] | 218 | CreateFiles(Spec.OldHeader, Header); |
| 219 | CreateFiles(Spec.OldCC, CC); |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 220 | |
| 221 | std::map<std::string, tooling::Replacements> FileToReplacements; |
Haojian Wu | d2a6d7b | 2016-10-04 09:05:31 +0000 | [diff] [blame] | 222 | llvm::SmallString<128> InitialDirectory; |
| 223 | std::error_code EC = llvm::sys::fs::current_path(InitialDirectory); |
| 224 | assert(!EC); |
| 225 | (void)EC; |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 226 | ClangMoveContext MoveContext = {Spec, FileToReplacements, |
| 227 | InitialDirectory.str(), "LLVM", |
| 228 | Reporter != nullptr}; |
| 229 | |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 230 | auto Factory = llvm::make_unique<clang::move::ClangMoveActionFactory>( |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 231 | &MoveContext, Reporter); |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 232 | |
| 233 | tooling::runToolOnCodeWithArgs( |
Haojian Wu | 2930be1 | 2016-11-08 19:55:13 +0000 | [diff] [blame] | 234 | Factory->create(), CC, {"-std=c++11", "-fparse-all-comments"}, |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 235 | TestCCName, "clang-move", std::make_shared<PCHContainerOperations>(), |
| 236 | FileToSourceText); |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 237 | formatAndApplyAllReplacements(FileToReplacements, Context.Rewrite, "llvm"); |
| 238 | // The Key is file name, value is the new code after moving the class. |
| 239 | std::map<std::string, std::string> Results; |
| 240 | for (const auto &It : FileToReplacements) { |
| 241 | StringRef FilePath = It.first; |
| 242 | Results[FilePath] = Context.getRewrittenText(FileToFileID[FilePath]); |
| 243 | } |
| 244 | return Results; |
| 245 | } |
| 246 | |
| 247 | TEST(ClangMove, MoveHeaderAndCC) { |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 248 | move::MoveDefinitionSpec Spec; |
NAKAMURA Takumi | 0cfbbae | 2016-10-17 05:09:58 +0000 | [diff] [blame] | 249 | Spec.Names = {std::string("a::b::Foo")}; |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 250 | Spec.OldHeader = "foo.h"; |
| 251 | Spec.OldCC = "foo.cc"; |
| 252 | Spec.NewHeader = "new_foo.h"; |
| 253 | Spec.NewCC = "new_foo.cc"; |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 254 | std::string ExpectedHeader = "#include \"" + Spec.NewHeader + "\"\n\n"; |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 255 | auto Results = runClangMoveOnCode(Spec); |
| 256 | EXPECT_EQ(ExpectedTestHeader, Results[Spec.OldHeader]); |
| 257 | EXPECT_EQ(ExpectedTestCC, Results[Spec.OldCC]); |
| 258 | EXPECT_EQ(ExpectedNewHeader, Results[Spec.NewHeader]); |
Haojian Wu | daf4cb8 | 2016-09-23 13:28:38 +0000 | [diff] [blame] | 259 | EXPECT_EQ(ExpectedHeader + ExpectedNewCC, Results[Spec.NewCC]); |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 260 | } |
| 261 | |
| 262 | TEST(ClangMove, MoveHeaderOnly) { |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 263 | move::MoveDefinitionSpec Spec; |
NAKAMURA Takumi | 0cfbbae | 2016-10-17 05:09:58 +0000 | [diff] [blame] | 264 | Spec.Names = {std::string("a::b::Foo")}; |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 265 | Spec.OldHeader = "foo.h"; |
| 266 | Spec.NewHeader = "new_foo.h"; |
| 267 | auto Results = runClangMoveOnCode(Spec); |
Haojian Wu | 62ba446 | 2016-09-21 15:19:04 +0000 | [diff] [blame] | 268 | EXPECT_EQ(2u, Results.size()); |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 269 | EXPECT_EQ(ExpectedTestHeader, Results[Spec.OldHeader]); |
| 270 | EXPECT_EQ(ExpectedNewHeader, Results[Spec.NewHeader]); |
| 271 | } |
| 272 | |
| 273 | TEST(ClangMove, MoveCCOnly) { |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 274 | move::MoveDefinitionSpec Spec; |
NAKAMURA Takumi | 0cfbbae | 2016-10-17 05:09:58 +0000 | [diff] [blame] | 275 | Spec.Names = {std::string("a::b::Foo")}; |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 276 | Spec.OldCC = "foo.cc"; |
| 277 | Spec.NewCC = "new_foo.cc"; |
Haojian Wu | 9abbeaa | 2016-10-06 08:59:24 +0000 | [diff] [blame] | 278 | std::string ExpectedHeader = "#include \"foo.h\"\n\n"; |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 279 | auto Results = runClangMoveOnCode(Spec); |
Haojian Wu | 62ba446 | 2016-09-21 15:19:04 +0000 | [diff] [blame] | 280 | EXPECT_EQ(2u, Results.size()); |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 281 | EXPECT_EQ(ExpectedTestCC, Results[Spec.OldCC]); |
Haojian Wu | daf4cb8 | 2016-09-23 13:28:38 +0000 | [diff] [blame] | 282 | EXPECT_EQ(ExpectedHeader + ExpectedNewCC, Results[Spec.NewCC]); |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 283 | } |
| 284 | |
| 285 | TEST(ClangMove, MoveNonExistClass) { |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 286 | move::MoveDefinitionSpec Spec; |
NAKAMURA Takumi | 0cfbbae | 2016-10-17 05:09:58 +0000 | [diff] [blame] | 287 | Spec.Names = {std::string("NonExistFoo")}; |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 288 | Spec.OldHeader = "foo.h"; |
| 289 | Spec.OldCC = "foo.cc"; |
| 290 | Spec.NewHeader = "new_foo.h"; |
| 291 | Spec.NewCC = "new_foo.cc"; |
| 292 | auto Results = runClangMoveOnCode(Spec); |
Haojian Wu | 62ba446 | 2016-09-21 15:19:04 +0000 | [diff] [blame] | 293 | EXPECT_EQ(0u, Results.size()); |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 294 | } |
| 295 | |
Haojian Wu | 2930be1 | 2016-11-08 19:55:13 +0000 | [diff] [blame] | 296 | TEST(ClangMove, MoveAll) { |
| 297 | std::vector<std::string> TestHeaders = { |
| 298 | "class A {\npublic:\n int f();\n};", |
| 299 | // forward declaration. |
| 300 | "class B;\nclass A {\npublic:\n int f();\n};", |
| 301 | // template forward declaration. |
| 302 | "template <typename T> class B;\nclass A {\npublic:\n int f();\n};", |
| 303 | "namespace a {}\nclass A {\npublic:\n int f();\n};", |
| 304 | "namespace a {}\nusing namespace a;\nclass A {\npublic:\n int f();\n};", |
| 305 | }; |
| 306 | const char Code[] = "#include \"foo.h\"\nint A::f() { return 0; }"; |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 307 | move::MoveDefinitionSpec Spec; |
Haojian Wu | 2930be1 | 2016-11-08 19:55:13 +0000 | [diff] [blame] | 308 | Spec.Names.push_back("A"); |
| 309 | Spec.OldHeader = "foo.h"; |
| 310 | Spec.OldCC = "foo.cc"; |
| 311 | Spec.NewHeader = "new_foo.h"; |
| 312 | Spec.NewCC = "new_foo.cc"; |
| 313 | for (const auto& Header : TestHeaders) { |
| 314 | auto Results = runClangMoveOnCode(Spec, Header.c_str(), Code); |
| 315 | EXPECT_EQ(Header, Results[Spec.NewHeader]); |
| 316 | EXPECT_EQ("", Results[Spec.OldHeader]); |
| 317 | EXPECT_EQ("", Results[Spec.OldCC]); |
| 318 | } |
| 319 | } |
| 320 | |
| 321 | TEST(ClangMove, MoveAllMultipleClasses) { |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 322 | move::MoveDefinitionSpec Spec; |
Haojian Wu | 2930be1 | 2016-11-08 19:55:13 +0000 | [diff] [blame] | 323 | std::vector<std::string> TestHeaders = { |
| 324 | "class C;\nclass A {\npublic:\n int f();\n};\nclass B {};", |
| 325 | "class C;\nclass B;\nclass A {\npublic:\n int f();\n};\nclass B {};", |
| 326 | }; |
| 327 | const char Code[] = "#include \"foo.h\"\nint A::f() { return 0; }"; |
| 328 | Spec.Names = {std::string("A"), std::string("B")}; |
| 329 | Spec.OldHeader = "foo.h"; |
| 330 | Spec.OldCC = "foo.cc"; |
| 331 | Spec.NewHeader = "new_foo.h"; |
| 332 | Spec.NewCC = "new_foo.cc"; |
| 333 | for (const auto& Header : TestHeaders) { |
| 334 | auto Results = runClangMoveOnCode(Spec, Header.c_str(), Code); |
| 335 | EXPECT_EQ(Header, Results[Spec.NewHeader]); |
| 336 | EXPECT_EQ("", Results[Spec.OldHeader]); |
| 337 | EXPECT_EQ("", Results[Spec.OldCC]); |
| 338 | } |
| 339 | } |
| 340 | |
| 341 | TEST(ClangMove, DontMoveAll) { |
| 342 | const char ExpectedHeader[] = "#ifndef NEW_FOO_H\n" |
| 343 | "#define NEW_FOO_H\n" |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 344 | "\n" |
Haojian Wu | 2930be1 | 2016-11-08 19:55:13 +0000 | [diff] [blame] | 345 | "class A {\npublic:\n int f();\n};\n" |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 346 | "\n" |
Haojian Wu | 2930be1 | 2016-11-08 19:55:13 +0000 | [diff] [blame] | 347 | "#endif // NEW_FOO_H\n"; |
| 348 | const char Code[] = "#include \"foo.h\"\nint A::f() { return 0; }"; |
| 349 | std::vector<std::string> TestHeaders = { |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 350 | "class B {};\nclass A {\npublic:\n int f();\n};\n", |
| 351 | "void f() {};\nclass A {\npublic:\n int f();\n};\n", |
Haojian Wu | 2930be1 | 2016-11-08 19:55:13 +0000 | [diff] [blame] | 352 | }; |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 353 | move::MoveDefinitionSpec Spec; |
Haojian Wu | 2930be1 | 2016-11-08 19:55:13 +0000 | [diff] [blame] | 354 | Spec.Names.push_back("A"); |
| 355 | Spec.OldHeader = "foo.h"; |
| 356 | Spec.OldCC = "foo.cc"; |
| 357 | Spec.NewHeader = "new_foo.h"; |
| 358 | Spec.NewCC = "new_foo.cc"; |
| 359 | for (const auto& Header : TestHeaders) { |
| 360 | auto Results = runClangMoveOnCode(Spec, Header.c_str(), Code); |
| 361 | EXPECT_EQ(ExpectedHeader, Results[Spec.NewHeader]); |
| 362 | // The expected old header should not contain class A definition. |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 363 | std::string ExpectedOldHeader = Header.substr(0, Header.size() - 32); |
Haojian Wu | 2930be1 | 2016-11-08 19:55:13 +0000 | [diff] [blame] | 364 | EXPECT_EQ(ExpectedOldHeader, Results[Spec.OldHeader]); |
| 365 | } |
| 366 | } |
| 367 | |
Haojian Wu | 2467539 | 2016-11-14 14:46:48 +0000 | [diff] [blame] | 368 | TEST(ClangMove, MacroInFunction) { |
| 369 | const char TestHeader[] = "#define INT int\n" |
| 370 | "class A {\npublic:\n int f();\n};\n" |
| 371 | "class B {};\n"; |
| 372 | const char TestCode[] = "#include \"foo.h\"\n" |
| 373 | "INT A::f() { return 0; }\n"; |
| 374 | const char ExpectedNewCode[] = "#include \"new_foo.h\"\n\n" |
| 375 | "INT A::f() { return 0; }\n"; |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 376 | move::MoveDefinitionSpec Spec; |
Haojian Wu | 2467539 | 2016-11-14 14:46:48 +0000 | [diff] [blame] | 377 | Spec.Names.push_back("A"); |
| 378 | Spec.OldHeader = "foo.h"; |
| 379 | Spec.OldCC = "foo.cc"; |
| 380 | Spec.NewHeader = "new_foo.h"; |
| 381 | Spec.NewCC = "new_foo.cc"; |
| 382 | auto Results = runClangMoveOnCode(Spec, TestHeader, TestCode); |
| 383 | EXPECT_EQ(ExpectedNewCode, Results[Spec.NewCC]); |
| 384 | } |
| 385 | |
Haojian Wu | dc4edba | 2016-12-13 15:35:47 +0000 | [diff] [blame] | 386 | TEST(ClangMove, DefinitionInMacro) { |
| 387 | const char TestHeader[] = "#define DEF(CLASS) void CLASS##_::f() {}\n" |
| 388 | "class A_ {\nvoid f();\n};\n" |
| 389 | "class B {};\n"; |
| 390 | const char TestCode[] = "#include \"foo.h\"\n" |
| 391 | "DEF(A)\n"; |
| 392 | const char ExpectedNewCode[] = "#include \"new_foo.h\"\n\n" |
| 393 | "DEF(A)\n"; |
| 394 | move::MoveDefinitionSpec Spec; |
| 395 | Spec.Names.push_back("A_"); |
| 396 | Spec.OldHeader = "foo.h"; |
| 397 | Spec.OldCC = "foo.cc"; |
| 398 | Spec.NewHeader = "new_foo.h"; |
| 399 | Spec.NewCC = "new_foo.cc"; |
| 400 | auto Results = runClangMoveOnCode(Spec, TestHeader, TestCode); |
| 401 | EXPECT_EQ(ExpectedNewCode, Results[Spec.NewCC]); |
| 402 | } |
| 403 | |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 404 | TEST(ClangMove, WellFormattedCode) { |
| 405 | const std::string CommonHeader = |
| 406 | "namespace a {\n" |
| 407 | "namespace b {\n" |
| 408 | "namespace c {\n" |
| 409 | "class C;\n" |
| 410 | "\n" |
| 411 | "class A {\npublic:\n void f();\n void f2();\n};\n" |
| 412 | "} // namespace c\n" |
| 413 | "} // namespace b\n" |
| 414 | "\n" |
| 415 | "namespace d {\n" |
| 416 | "namespace e {\n" |
| 417 | "class B {\npublic:\n void f();\n};\n" |
| 418 | "} // namespace e\n" |
| 419 | "} // namespace d\n" |
| 420 | "} // namespace a\n"; |
| 421 | const std::string CommonCode = "\n" |
| 422 | "namespace a {\n" |
| 423 | "namespace b {\n" |
| 424 | "namespace c {\n" |
| 425 | "void A::f() {}\n" |
| 426 | "\n" |
| 427 | "void A::f2() {}\n" |
| 428 | "} // namespace c\n" |
| 429 | "} // namespace b\n" |
| 430 | "\n" |
| 431 | "namespace d {\n" |
| 432 | "namespace e {\n" |
| 433 | "void B::f() {}\n" |
| 434 | "} // namespace e\n" |
| 435 | "} // namespace d\n" |
| 436 | "} // namespace a\n"; |
| 437 | // Add dummy class to prevent behavior of moving all declarations from header. |
| 438 | const std::string TestHeader = CommonHeader + "class D {};\n"; |
| 439 | const std::string TestCode = "#include \"foo.h\"\n" + CommonCode; |
| 440 | const std::string ExpectedNewHeader = "#ifndef NEW_FOO_H\n" |
| 441 | "#define NEW_FOO_H\n" |
| 442 | "\n" + |
| 443 | CommonHeader + |
| 444 | "\n" |
| 445 | "#endif // NEW_FOO_H\n"; |
| 446 | const std::string ExpectedNewCC = "#include \"new_foo.h\"\n" + CommonCode; |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 447 | move::MoveDefinitionSpec Spec; |
Haojian Wu | 53315a7 | 2016-11-15 09:06:59 +0000 | [diff] [blame] | 448 | Spec.Names.push_back("a::b::c::A"); |
| 449 | Spec.Names.push_back("a::d::e::B"); |
| 450 | Spec.OldHeader = "foo.h"; |
| 451 | Spec.OldCC = "foo.cc"; |
| 452 | Spec.NewHeader = "new_foo.h"; |
| 453 | Spec.NewCC = "new_foo.cc"; |
| 454 | auto Results = runClangMoveOnCode(Spec, TestHeader.c_str(), TestCode.c_str()); |
| 455 | EXPECT_EQ(ExpectedNewCC, Results[Spec.NewCC]); |
| 456 | EXPECT_EQ(ExpectedNewHeader, Results[Spec.NewHeader]); |
| 457 | } |
| 458 | |
Haojian Wu | 48ac304 | 2016-11-23 10:04:19 +0000 | [diff] [blame] | 459 | TEST(ClangMove, AddDependentNewHeader) { |
| 460 | const char TestHeader[] = "class A {};\n" |
| 461 | "class B {};\n"; |
| 462 | const char TestCode[] = "#include \"foo.h\"\n"; |
| 463 | const char ExpectedOldHeader[] = "#include \"new_foo.h\"\nclass B {};\n"; |
| 464 | const char ExpectedNewHeader[] = "#ifndef NEW_FOO_H\n" |
| 465 | "#define NEW_FOO_H\n" |
| 466 | "\n" |
| 467 | "class A {};\n" |
| 468 | "\n" |
| 469 | "#endif // NEW_FOO_H\n"; |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 470 | move::MoveDefinitionSpec Spec; |
Haojian Wu | 48ac304 | 2016-11-23 10:04:19 +0000 | [diff] [blame] | 471 | Spec.Names.push_back("A"); |
| 472 | Spec.OldHeader = "foo.h"; |
| 473 | Spec.OldCC = "foo.cc"; |
| 474 | Spec.NewHeader = "new_foo.h"; |
| 475 | Spec.NewCC = "new_foo.cc"; |
| 476 | Spec.OldDependOnNew = true; |
| 477 | auto Results = runClangMoveOnCode(Spec, TestHeader, TestCode); |
| 478 | EXPECT_EQ(ExpectedOldHeader, Results[Spec.OldHeader]); |
| 479 | EXPECT_EQ(ExpectedNewHeader, Results[Spec.NewHeader]); |
| 480 | } |
| 481 | |
| 482 | TEST(ClangMove, AddDependentOldHeader) { |
| 483 | const char TestHeader[] = "class A {};\n" |
| 484 | "class B {};\n"; |
| 485 | const char TestCode[] = "#include \"foo.h\"\n"; |
| 486 | const char ExpectedNewHeader[] = "#ifndef NEW_FOO_H\n" |
| 487 | "#define NEW_FOO_H\n" |
| 488 | "\n" |
| 489 | "#include \"foo.h\"\n" |
| 490 | "\n" |
| 491 | "class B {};\n" |
| 492 | "\n" |
| 493 | "#endif // NEW_FOO_H\n"; |
| 494 | const char ExpectedOldHeader[] = "class A {};\n"; |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 495 | move::MoveDefinitionSpec Spec; |
Haojian Wu | 48ac304 | 2016-11-23 10:04:19 +0000 | [diff] [blame] | 496 | Spec.Names.push_back("B"); |
| 497 | Spec.OldHeader = "foo.h"; |
| 498 | Spec.OldCC = "foo.cc"; |
| 499 | Spec.NewHeader = "new_foo.h"; |
| 500 | Spec.NewCC = "new_foo.cc"; |
| 501 | Spec.NewDependOnOld = true; |
| 502 | auto Results = runClangMoveOnCode(Spec, TestHeader, TestCode); |
| 503 | EXPECT_EQ(ExpectedNewHeader, Results[Spec.NewHeader]); |
| 504 | EXPECT_EQ(ExpectedOldHeader, Results[Spec.OldHeader]); |
| 505 | } |
| 506 | |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 507 | TEST(ClangMove, DumpDecls) { |
| 508 | const char TestHeader[] = "template <typename T>\n" |
| 509 | "class A {\n" |
| 510 | " public:\n" |
| 511 | " void f();\n" |
| 512 | " template <typename U> void h();\n" |
| 513 | " static int b;\n" |
| 514 | "};\n" |
| 515 | "\n" |
| 516 | "template <typename T> void A<T>::f() {}\n" |
| 517 | "\n" |
| 518 | "template <typename T>\n" |
| 519 | "template <typename U>\n" |
| 520 | "void A<T>::h() {}\n" |
| 521 | "\n" |
| 522 | "template <typename T> int A<T>::b = 2;\n" |
| 523 | "\n" |
| 524 | "template <> class A<int> {};\n" |
| 525 | "\n" |
| 526 | "class B {};\n" |
| 527 | "\n" |
| 528 | "namespace a {\n" |
| 529 | "class Move1 {};\n" |
| 530 | "void f1() {}\n" |
| 531 | "void f2();\n" |
| 532 | "} // namespace a\n" |
| 533 | "\n" |
| 534 | "namespace a {\n" |
| 535 | "namespace b {\n" |
| 536 | "class Move1 { public : void f(); };\n" |
| 537 | "void f() {}\n" |
Haojian Wu | 8586772 | 2017-01-16 09:34:07 +0000 | [diff] [blame] | 538 | "enum E1 { Green };\n" |
| 539 | "enum class E2 { Red };\n" |
| 540 | "typedef int Int2;\n" |
| 541 | "using Int = int;\n" |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 542 | "} // namespace b\n" |
| 543 | "} // namespace a\n"; |
| 544 | const char TestCode[] = "#include \"foo.h\"\n"; |
| 545 | move::MoveDefinitionSpec Spec; |
| 546 | Spec.Names.push_back("B"); |
| 547 | Spec.OldHeader = "foo.h"; |
| 548 | Spec.OldCC = "foo.cc"; |
| 549 | Spec.NewHeader = "new_foo.h"; |
| 550 | Spec.NewCC = "new_foo.cc"; |
| 551 | DeclarationReporter Reporter; |
Haojian Wu | 8586772 | 2017-01-16 09:34:07 +0000 | [diff] [blame] | 552 | std::set<DeclarationReporter::DeclarationPair> ExpectedDeclarations = { |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 553 | {"A", "Class"}, {"B", "Class"}, {"a::Move1", "Class"}, |
| 554 | {"a::f1", "Function"}, {"a::f2", "Function"}, {"a::b::Move1", "Class"}, |
Haojian Wu | 8586772 | 2017-01-16 09:34:07 +0000 | [diff] [blame] | 555 | {"a::b::f", "Function"}, {"a::b::E1", "Enum"}, {"a::b::E2", "Enum"}, |
| 556 | {"a::b::Int2", "TypeAlias"}, {"a::b::Int", "TypeAlias"} }; |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 557 | runClangMoveOnCode(Spec, TestHeader, TestCode, &Reporter); |
Haojian Wu | 8586772 | 2017-01-16 09:34:07 +0000 | [diff] [blame] | 558 | std::set<DeclarationReporter::DeclarationPair> Results; |
| 559 | for (const auto& DelPair : Reporter.getDeclarationList()) |
| 560 | Results.insert(DelPair); |
| 561 | EXPECT_EQ(ExpectedDeclarations, Results); |
Haojian Wu | b15c8da | 2016-11-24 10:17:17 +0000 | [diff] [blame] | 562 | } |
| 563 | |
Haojian Wu | 357ef99 | 2016-09-21 13:18:19 +0000 | [diff] [blame] | 564 | } // namespace |
| 565 | } // namespce move |
| 566 | } // namespace clang |