Chandler Carruth | 320d966 | 2012-12-04 09:45:34 +0000 | [diff] [blame] | 1 | //===- unittest/Tooling/RefactoringCallbacksTest.cpp ----------------------===// |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 2 | // |
Chandler Carruth | 2946cd7 | 2019-01-19 08:50:56 +0000 | [diff] [blame] | 3 | // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. |
| 4 | // See https://llvm.org/LICENSE.txt for license information. |
| 5 | // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 6 | // |
| 7 | //===----------------------------------------------------------------------===// |
| 8 | |
Daniel Jasper | 1975e03 | 2012-07-17 08:03:01 +0000 | [diff] [blame] | 9 | #include "RewriterTestContext.h" |
Chandler Carruth | 320d966 | 2012-12-04 09:45:34 +0000 | [diff] [blame] | 10 | #include "clang/ASTMatchers/ASTMatchFinder.h" |
Chandler Carruth | fa0b3bb | 2012-12-04 09:53:37 +0000 | [diff] [blame] | 11 | #include "clang/ASTMatchers/ASTMatchers.h" |
Eric Liu | fc9213e | 2017-05-10 07:48:45 +0000 | [diff] [blame] | 12 | #include "clang/Tooling/RefactoringCallbacks.h" |
Douglas Gregor | 949cc50 | 2012-10-23 23:13:50 +0000 | [diff] [blame] | 13 | #include "gtest/gtest.h" |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 14 | |
| 15 | namespace clang { |
Daniel Jasper | 6389dd1 | 2012-07-17 08:37:03 +0000 | [diff] [blame] | 16 | namespace tooling { |
| 17 | |
| 18 | using namespace ast_matchers; |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 19 | |
| 20 | template <typename T> |
Eric Liu | fc9213e | 2017-05-10 07:48:45 +0000 | [diff] [blame] | 21 | void expectRewritten(const std::string &Code, const std::string &Expected, |
| 22 | const T &AMatcher, RefactoringCallback &Callback) { |
| 23 | std::map<std::string, Replacements> FileToReplace; |
| 24 | ASTMatchRefactorer Finder(FileToReplace); |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 25 | Finder.addMatcher(AMatcher, &Callback); |
Ahmed Charles | b898432 | 2014-03-07 20:03:18 +0000 | [diff] [blame] | 26 | std::unique_ptr<tooling::FrontendActionFactory> Factory( |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 27 | tooling::newFrontendActionFactory(&Finder)); |
| 28 | ASSERT_TRUE(tooling::runToolOnCode(Factory->create(), Code)) |
| 29 | << "Parsing error in \"" << Code << "\""; |
| 30 | RewriterTestContext Context; |
| 31 | FileID ID = Context.createInMemoryFile("input.cc", Code); |
Eric Liu | fc9213e | 2017-05-10 07:48:45 +0000 | [diff] [blame] | 32 | EXPECT_TRUE(tooling::applyAllReplacements(FileToReplace["input.cc"], |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 33 | Context.Rewrite)); |
| 34 | EXPECT_EQ(Expected, Context.getRewrittenText(ID)); |
| 35 | } |
| 36 | |
| 37 | TEST(RefactoringCallbacksTest, ReplacesStmtsWithString) { |
| 38 | std::string Code = "void f() { int i = 1; }"; |
| 39 | std::string Expected = "void f() { ; }"; |
| 40 | ReplaceStmtWithText Callback("id", ";"); |
Dmitri Gribenko | dee011b | 2019-08-20 13:02:28 +0000 | [diff] [blame] | 41 | expectRewritten(Code, Expected, declStmt().bind("id"), Callback); |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 42 | } |
| 43 | |
| 44 | TEST(RefactoringCallbacksTest, ReplacesStmtsInCalledMacros) { |
| 45 | std::string Code = "#define A void f() { int i = 1; }\nA"; |
| 46 | std::string Expected = "#define A void f() { ; }\nA"; |
| 47 | ReplaceStmtWithText Callback("id", ";"); |
Dmitri Gribenko | dee011b | 2019-08-20 13:02:28 +0000 | [diff] [blame] | 48 | expectRewritten(Code, Expected, declStmt().bind("id"), Callback); |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 49 | } |
| 50 | |
| 51 | TEST(RefactoringCallbacksTest, IgnoresStmtsInUncalledMacros) { |
| 52 | std::string Code = "#define A void f() { int i = 1; }"; |
| 53 | std::string Expected = "#define A void f() { int i = 1; }"; |
| 54 | ReplaceStmtWithText Callback("id", ";"); |
Dmitri Gribenko | dee011b | 2019-08-20 13:02:28 +0000 | [diff] [blame] | 55 | expectRewritten(Code, Expected, declStmt().bind("id"), Callback); |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 56 | } |
| 57 | |
| 58 | TEST(RefactoringCallbacksTest, ReplacesInteger) { |
| 59 | std::string Code = "void f() { int i = 1; }"; |
| 60 | std::string Expected = "void f() { int i = 2; }"; |
| 61 | ReplaceStmtWithText Callback("id", "2"); |
Dmitri Gribenko | dee011b | 2019-08-20 13:02:28 +0000 | [diff] [blame] | 62 | expectRewritten(Code, Expected, expr(integerLiteral()).bind("id"), Callback); |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 63 | } |
| 64 | |
| 65 | TEST(RefactoringCallbacksTest, ReplacesStmtWithStmt) { |
| 66 | std::string Code = "void f() { int i = false ? 1 : i * 2; }"; |
| 67 | std::string Expected = "void f() { int i = i * 2; }"; |
| 68 | ReplaceStmtWithStmt Callback("always-false", "should-be"); |
Eric Liu | fc9213e | 2017-05-10 07:48:45 +0000 | [diff] [blame] | 69 | expectRewritten( |
| 70 | Code, Expected, |
Dmitri Gribenko | dee011b | 2019-08-20 13:02:28 +0000 | [diff] [blame] | 71 | conditionalOperator(hasCondition(cxxBoolLiteral(equals(false))), |
| 72 | hasFalseExpression(expr().bind("should-be"))) |
| 73 | .bind("always-false"), |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 74 | Callback); |
| 75 | } |
| 76 | |
| 77 | TEST(RefactoringCallbacksTest, ReplacesIfStmt) { |
| 78 | std::string Code = "bool a; void f() { if (a) f(); else a = true; }"; |
| 79 | std::string Expected = "bool a; void f() { f(); }"; |
| 80 | ReplaceIfStmtWithItsBody Callback("id", true); |
Dmitri Gribenko | dee011b | 2019-08-20 13:02:28 +0000 | [diff] [blame] | 81 | expectRewritten(Code, Expected, |
| 82 | ifStmt(hasCondition(implicitCastExpr(hasSourceExpression( |
| 83 | declRefExpr(to(varDecl(hasName("a")))))))) |
| 84 | .bind("id"), |
| 85 | Callback); |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 86 | } |
| 87 | |
| 88 | TEST(RefactoringCallbacksTest, RemovesEntireIfOnEmptyElse) { |
| 89 | std::string Code = "void f() { if (false) int i = 0; }"; |
| 90 | std::string Expected = "void f() { }"; |
| 91 | ReplaceIfStmtWithItsBody Callback("id", false); |
Dmitri Gribenko | dee011b | 2019-08-20 13:02:28 +0000 | [diff] [blame] | 92 | expectRewritten( |
| 93 | Code, Expected, |
| 94 | ifStmt(hasCondition(cxxBoolLiteral(equals(false)))).bind("id"), Callback); |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 95 | } |
| 96 | |
Eric Liu | fc9213e | 2017-05-10 07:48:45 +0000 | [diff] [blame] | 97 | TEST(RefactoringCallbacksTest, TemplateJustText) { |
| 98 | std::string Code = "void f() { int i = 1; }"; |
| 99 | std::string Expected = "void f() { FOO }"; |
| 100 | auto Callback = ReplaceNodeWithTemplate::create("id", "FOO"); |
| 101 | EXPECT_FALSE(Callback.takeError()); |
Dmitri Gribenko | dee011b | 2019-08-20 13:02:28 +0000 | [diff] [blame] | 102 | expectRewritten(Code, Expected, declStmt().bind("id"), **Callback); |
Eric Liu | fc9213e | 2017-05-10 07:48:45 +0000 | [diff] [blame] | 103 | } |
| 104 | |
| 105 | TEST(RefactoringCallbacksTest, TemplateSimpleSubst) { |
| 106 | std::string Code = "void f() { int i = 1; }"; |
| 107 | std::string Expected = "void f() { long x = 1; }"; |
| 108 | auto Callback = ReplaceNodeWithTemplate::create("decl", "long x = ${init}"); |
| 109 | EXPECT_FALSE(Callback.takeError()); |
| 110 | expectRewritten(Code, Expected, |
Dmitri Gribenko | dee011b | 2019-08-20 13:02:28 +0000 | [diff] [blame] | 111 | varDecl(hasInitializer(expr().bind("init"))).bind("decl"), |
Eric Liu | fc9213e | 2017-05-10 07:48:45 +0000 | [diff] [blame] | 112 | **Callback); |
| 113 | } |
| 114 | |
| 115 | TEST(RefactoringCallbacksTest, TemplateLiteral) { |
| 116 | std::string Code = "void f() { int i = 1; }"; |
| 117 | std::string Expected = "void f() { string x = \"$-1\"; }"; |
| 118 | auto Callback = ReplaceNodeWithTemplate::create("decl", |
| 119 | "string x = \"$$-${init}\""); |
| 120 | EXPECT_FALSE(Callback.takeError()); |
| 121 | expectRewritten(Code, Expected, |
Dmitri Gribenko | dee011b | 2019-08-20 13:02:28 +0000 | [diff] [blame] | 122 | varDecl(hasInitializer(expr().bind("init"))).bind("decl"), |
Eric Liu | fc9213e | 2017-05-10 07:48:45 +0000 | [diff] [blame] | 123 | **Callback); |
| 124 | } |
| 125 | |
| 126 | static void ExpectStringError(const std::string &Expected, |
| 127 | llvm::Error E) { |
| 128 | std::string Found; |
| 129 | handleAllErrors(std::move(E), [&](const llvm::StringError &SE) { |
| 130 | llvm::raw_string_ostream Stream(Found); |
| 131 | SE.log(Stream); |
| 132 | }); |
| 133 | EXPECT_EQ(Expected, Found); |
| 134 | } |
| 135 | |
| 136 | TEST(RefactoringCallbacksTest, TemplateUnterminated) { |
| 137 | auto Callback = ReplaceNodeWithTemplate::create("decl", |
| 138 | "string x = \"$$-${init\""); |
| 139 | ExpectStringError("Unterminated ${...} in replacement template near ${init\"", |
| 140 | Callback.takeError()); |
| 141 | } |
| 142 | |
| 143 | TEST(RefactoringCallbacksTest, TemplateUnknownDollar) { |
| 144 | auto Callback = ReplaceNodeWithTemplate::create("decl", |
| 145 | "string x = \"$<"); |
| 146 | ExpectStringError("Invalid $ in replacement template near $<", |
| 147 | Callback.takeError()); |
| 148 | } |
| 149 | |
| 150 | |
Alexander Kornienko | ab9db51 | 2015-06-22 23:07:51 +0000 | [diff] [blame] | 151 | } // end namespace ast_matchers |
Daniel Jasper | 7e22282 | 2012-07-16 09:18:17 +0000 | [diff] [blame] | 152 | } // end namespace clang |