1*67e74705SXin Li //===- unittest/Tooling/RefactoringTest.cpp - Refactoring unit tests ------===//
2*67e74705SXin Li //
3*67e74705SXin Li // The LLVM Compiler Infrastructure
4*67e74705SXin Li //
5*67e74705SXin Li // This file is distributed under the University of Illinois Open Source
6*67e74705SXin Li // License. See LICENSE.TXT for details.
7*67e74705SXin Li //
8*67e74705SXin Li //===----------------------------------------------------------------------===//
9*67e74705SXin Li
10*67e74705SXin Li #include "RewriterTestContext.h"
11*67e74705SXin Li #include "clang/AST/ASTConsumer.h"
12*67e74705SXin Li #include "clang/AST/ASTContext.h"
13*67e74705SXin Li #include "clang/AST/DeclCXX.h"
14*67e74705SXin Li #include "clang/AST/DeclGroup.h"
15*67e74705SXin Li #include "clang/AST/RecursiveASTVisitor.h"
16*67e74705SXin Li #include "clang/Basic/Diagnostic.h"
17*67e74705SXin Li #include "clang/Basic/DiagnosticOptions.h"
18*67e74705SXin Li #include "clang/Basic/FileManager.h"
19*67e74705SXin Li #include "clang/Basic/LangOptions.h"
20*67e74705SXin Li #include "clang/Basic/SourceManager.h"
21*67e74705SXin Li #include "clang/Format/Format.h"
22*67e74705SXin Li #include "clang/Frontend/CompilerInstance.h"
23*67e74705SXin Li #include "clang/Frontend/FrontendAction.h"
24*67e74705SXin Li #include "clang/Frontend/TextDiagnosticPrinter.h"
25*67e74705SXin Li #include "clang/Rewrite/Core/Rewriter.h"
26*67e74705SXin Li #include "clang/Tooling/Refactoring.h"
27*67e74705SXin Li #include "clang/Tooling/Tooling.h"
28*67e74705SXin Li #include "llvm/ADT/SmallString.h"
29*67e74705SXin Li #include "llvm/Support/Path.h"
30*67e74705SXin Li #include "gtest/gtest.h"
31*67e74705SXin Li
32*67e74705SXin Li namespace clang {
33*67e74705SXin Li namespace tooling {
34*67e74705SXin Li
35*67e74705SXin Li class ReplacementTest : public ::testing::Test {
36*67e74705SXin Li protected:
createReplacement(SourceLocation Start,unsigned Length,llvm::StringRef ReplacementText)37*67e74705SXin Li Replacement createReplacement(SourceLocation Start, unsigned Length,
38*67e74705SXin Li llvm::StringRef ReplacementText) {
39*67e74705SXin Li return Replacement(Context.Sources, Start, Length, ReplacementText);
40*67e74705SXin Li }
41*67e74705SXin Li
42*67e74705SXin Li RewriterTestContext Context;
43*67e74705SXin Li };
44*67e74705SXin Li
TEST_F(ReplacementTest,CanDeleteAllText)45*67e74705SXin Li TEST_F(ReplacementTest, CanDeleteAllText) {
46*67e74705SXin Li FileID ID = Context.createInMemoryFile("input.cpp", "text");
47*67e74705SXin Li SourceLocation Location = Context.getLocation(ID, 1, 1);
48*67e74705SXin Li Replacement Replace(createReplacement(Location, 4, ""));
49*67e74705SXin Li EXPECT_TRUE(Replace.apply(Context.Rewrite));
50*67e74705SXin Li EXPECT_EQ("", Context.getRewrittenText(ID));
51*67e74705SXin Li }
52*67e74705SXin Li
TEST_F(ReplacementTest,CanDeleteAllTextInTextWithNewlines)53*67e74705SXin Li TEST_F(ReplacementTest, CanDeleteAllTextInTextWithNewlines) {
54*67e74705SXin Li FileID ID = Context.createInMemoryFile("input.cpp", "line1\nline2\nline3");
55*67e74705SXin Li SourceLocation Location = Context.getLocation(ID, 1, 1);
56*67e74705SXin Li Replacement Replace(createReplacement(Location, 17, ""));
57*67e74705SXin Li EXPECT_TRUE(Replace.apply(Context.Rewrite));
58*67e74705SXin Li EXPECT_EQ("", Context.getRewrittenText(ID));
59*67e74705SXin Li }
60*67e74705SXin Li
TEST_F(ReplacementTest,CanAddText)61*67e74705SXin Li TEST_F(ReplacementTest, CanAddText) {
62*67e74705SXin Li FileID ID = Context.createInMemoryFile("input.cpp", "");
63*67e74705SXin Li SourceLocation Location = Context.getLocation(ID, 1, 1);
64*67e74705SXin Li Replacement Replace(createReplacement(Location, 0, "result"));
65*67e74705SXin Li EXPECT_TRUE(Replace.apply(Context.Rewrite));
66*67e74705SXin Li EXPECT_EQ("result", Context.getRewrittenText(ID));
67*67e74705SXin Li }
68*67e74705SXin Li
TEST_F(ReplacementTest,CanReplaceTextAtPosition)69*67e74705SXin Li TEST_F(ReplacementTest, CanReplaceTextAtPosition) {
70*67e74705SXin Li FileID ID = Context.createInMemoryFile("input.cpp",
71*67e74705SXin Li "line1\nline2\nline3\nline4");
72*67e74705SXin Li SourceLocation Location = Context.getLocation(ID, 2, 3);
73*67e74705SXin Li Replacement Replace(createReplacement(Location, 12, "x"));
74*67e74705SXin Li EXPECT_TRUE(Replace.apply(Context.Rewrite));
75*67e74705SXin Li EXPECT_EQ("line1\nlixne4", Context.getRewrittenText(ID));
76*67e74705SXin Li }
77*67e74705SXin Li
TEST_F(ReplacementTest,CanReplaceTextAtPositionMultipleTimes)78*67e74705SXin Li TEST_F(ReplacementTest, CanReplaceTextAtPositionMultipleTimes) {
79*67e74705SXin Li FileID ID = Context.createInMemoryFile("input.cpp",
80*67e74705SXin Li "line1\nline2\nline3\nline4");
81*67e74705SXin Li SourceLocation Location1 = Context.getLocation(ID, 2, 3);
82*67e74705SXin Li Replacement Replace1(createReplacement(Location1, 12, "x\ny\n"));
83*67e74705SXin Li EXPECT_TRUE(Replace1.apply(Context.Rewrite));
84*67e74705SXin Li EXPECT_EQ("line1\nlix\ny\nne4", Context.getRewrittenText(ID));
85*67e74705SXin Li
86*67e74705SXin Li // Since the original source has not been modified, the (4, 4) points to the
87*67e74705SXin Li // 'e' in the original content.
88*67e74705SXin Li SourceLocation Location2 = Context.getLocation(ID, 4, 4);
89*67e74705SXin Li Replacement Replace2(createReplacement(Location2, 1, "f"));
90*67e74705SXin Li EXPECT_TRUE(Replace2.apply(Context.Rewrite));
91*67e74705SXin Li EXPECT_EQ("line1\nlix\ny\nnf4", Context.getRewrittenText(ID));
92*67e74705SXin Li }
93*67e74705SXin Li
TEST_F(ReplacementTest,ApplyFailsForNonExistentLocation)94*67e74705SXin Li TEST_F(ReplacementTest, ApplyFailsForNonExistentLocation) {
95*67e74705SXin Li Replacement Replace("nonexistent-file.cpp", 0, 1, "");
96*67e74705SXin Li EXPECT_FALSE(Replace.apply(Context.Rewrite));
97*67e74705SXin Li }
98*67e74705SXin Li
TEST_F(ReplacementTest,CanRetrivePath)99*67e74705SXin Li TEST_F(ReplacementTest, CanRetrivePath) {
100*67e74705SXin Li Replacement Replace("/path/to/file.cpp", 0, 1, "");
101*67e74705SXin Li EXPECT_EQ("/path/to/file.cpp", Replace.getFilePath());
102*67e74705SXin Li }
103*67e74705SXin Li
TEST_F(ReplacementTest,ReturnsInvalidPath)104*67e74705SXin Li TEST_F(ReplacementTest, ReturnsInvalidPath) {
105*67e74705SXin Li Replacement Replace1(Context.Sources, SourceLocation(), 0, "");
106*67e74705SXin Li EXPECT_TRUE(Replace1.getFilePath().empty());
107*67e74705SXin Li
108*67e74705SXin Li Replacement Replace2;
109*67e74705SXin Li EXPECT_TRUE(Replace2.getFilePath().empty());
110*67e74705SXin Li }
111*67e74705SXin Li
TEST_F(ReplacementTest,CanApplyReplacements)112*67e74705SXin Li TEST_F(ReplacementTest, CanApplyReplacements) {
113*67e74705SXin Li FileID ID = Context.createInMemoryFile("input.cpp",
114*67e74705SXin Li "line1\nline2\nline3\nline4");
115*67e74705SXin Li Replacements Replaces;
116*67e74705SXin Li Replaces.insert(Replacement(Context.Sources, Context.getLocation(ID, 2, 1),
117*67e74705SXin Li 5, "replaced"));
118*67e74705SXin Li Replaces.insert(Replacement(Context.Sources, Context.getLocation(ID, 3, 1),
119*67e74705SXin Li 5, "other"));
120*67e74705SXin Li EXPECT_TRUE(applyAllReplacements(Replaces, Context.Rewrite));
121*67e74705SXin Li EXPECT_EQ("line1\nreplaced\nother\nline4", Context.getRewrittenText(ID));
122*67e74705SXin Li }
123*67e74705SXin Li
124*67e74705SXin Li // FIXME: Remove this test case when Replacements is implemented as std::vector
125*67e74705SXin Li // instead of std::set. The other ReplacementTest tests will need to be updated
126*67e74705SXin Li // at that point as well.
TEST_F(ReplacementTest,VectorCanApplyReplacements)127*67e74705SXin Li TEST_F(ReplacementTest, VectorCanApplyReplacements) {
128*67e74705SXin Li FileID ID = Context.createInMemoryFile("input.cpp",
129*67e74705SXin Li "line1\nline2\nline3\nline4");
130*67e74705SXin Li std::vector<Replacement> Replaces;
131*67e74705SXin Li Replaces.push_back(Replacement(Context.Sources, Context.getLocation(ID, 2, 1),
132*67e74705SXin Li 5, "replaced"));
133*67e74705SXin Li Replaces.push_back(
134*67e74705SXin Li Replacement(Context.Sources, Context.getLocation(ID, 3, 1), 5, "other"));
135*67e74705SXin Li EXPECT_TRUE(applyAllReplacements(Replaces, Context.Rewrite));
136*67e74705SXin Li EXPECT_EQ("line1\nreplaced\nother\nline4", Context.getRewrittenText(ID));
137*67e74705SXin Li }
138*67e74705SXin Li
TEST_F(ReplacementTest,SkipsDuplicateReplacements)139*67e74705SXin Li TEST_F(ReplacementTest, SkipsDuplicateReplacements) {
140*67e74705SXin Li FileID ID = Context.createInMemoryFile("input.cpp",
141*67e74705SXin Li "line1\nline2\nline3\nline4");
142*67e74705SXin Li Replacements Replaces;
143*67e74705SXin Li Replaces.insert(Replacement(Context.Sources, Context.getLocation(ID, 2, 1),
144*67e74705SXin Li 5, "replaced"));
145*67e74705SXin Li Replaces.insert(Replacement(Context.Sources, Context.getLocation(ID, 2, 1),
146*67e74705SXin Li 5, "replaced"));
147*67e74705SXin Li Replaces.insert(Replacement(Context.Sources, Context.getLocation(ID, 2, 1),
148*67e74705SXin Li 5, "replaced"));
149*67e74705SXin Li EXPECT_TRUE(applyAllReplacements(Replaces, Context.Rewrite));
150*67e74705SXin Li EXPECT_EQ("line1\nreplaced\nline3\nline4", Context.getRewrittenText(ID));
151*67e74705SXin Li }
152*67e74705SXin Li
TEST_F(ReplacementTest,ApplyAllFailsIfOneApplyFails)153*67e74705SXin Li TEST_F(ReplacementTest, ApplyAllFailsIfOneApplyFails) {
154*67e74705SXin Li // This test depends on the value of the file name of an invalid source
155*67e74705SXin Li // location being in the range ]a, z[.
156*67e74705SXin Li FileID IDa = Context.createInMemoryFile("a.cpp", "text");
157*67e74705SXin Li FileID IDz = Context.createInMemoryFile("z.cpp", "text");
158*67e74705SXin Li Replacements Replaces;
159*67e74705SXin Li Replaces.insert(Replacement(Context.Sources, Context.getLocation(IDa, 1, 1),
160*67e74705SXin Li 4, "a"));
161*67e74705SXin Li Replaces.insert(Replacement(Context.Sources, SourceLocation(),
162*67e74705SXin Li 5, "2"));
163*67e74705SXin Li Replaces.insert(Replacement(Context.Sources, Context.getLocation(IDz, 1, 1),
164*67e74705SXin Li 4, "z"));
165*67e74705SXin Li EXPECT_FALSE(applyAllReplacements(Replaces, Context.Rewrite));
166*67e74705SXin Li EXPECT_EQ("a", Context.getRewrittenText(IDa));
167*67e74705SXin Li EXPECT_EQ("z", Context.getRewrittenText(IDz));
168*67e74705SXin Li }
169*67e74705SXin Li
TEST_F(ReplacementTest,MultipleFilesReplaceAndFormat)170*67e74705SXin Li TEST_F(ReplacementTest, MultipleFilesReplaceAndFormat) {
171*67e74705SXin Li // Column limit is 20.
172*67e74705SXin Li std::string Code1 = "Long *a =\n"
173*67e74705SXin Li " new Long();\n"
174*67e74705SXin Li "long x = 1;";
175*67e74705SXin Li std::string Expected1 = "auto a = new Long();\n"
176*67e74705SXin Li "long x =\n"
177*67e74705SXin Li " 12345678901;";
178*67e74705SXin Li std::string Code2 = "int x = 123;\n"
179*67e74705SXin Li "int y = 0;";
180*67e74705SXin Li std::string Expected2 = "int x =\n"
181*67e74705SXin Li " 1234567890123;\n"
182*67e74705SXin Li "int y = 10;";
183*67e74705SXin Li FileID ID1 = Context.createInMemoryFile("format_1.cpp", Code1);
184*67e74705SXin Li FileID ID2 = Context.createInMemoryFile("format_2.cpp", Code2);
185*67e74705SXin Li
186*67e74705SXin Li tooling::Replacements Replaces;
187*67e74705SXin Li // Scrambled the order of replacements.
188*67e74705SXin Li Replaces.insert(tooling::Replacement(
189*67e74705SXin Li Context.Sources, Context.getLocation(ID2, 1, 12), 0, "4567890123"));
190*67e74705SXin Li Replaces.insert(tooling::Replacement(
191*67e74705SXin Li Context.Sources, Context.getLocation(ID1, 1, 1), 6, "auto "));
192*67e74705SXin Li Replaces.insert(tooling::Replacement(
193*67e74705SXin Li Context.Sources, Context.getLocation(ID2, 2, 9), 1, "10"));
194*67e74705SXin Li Replaces.insert(tooling::Replacement(
195*67e74705SXin Li Context.Sources, Context.getLocation(ID1, 3, 10), 1, "12345678901"));
196*67e74705SXin Li
197*67e74705SXin Li EXPECT_TRUE(formatAndApplyAllReplacements(
198*67e74705SXin Li Replaces, Context.Rewrite, "{BasedOnStyle: LLVM, ColumnLimit: 20}"));
199*67e74705SXin Li EXPECT_EQ(Expected1, Context.getRewrittenText(ID1));
200*67e74705SXin Li EXPECT_EQ(Expected2, Context.getRewrittenText(ID2));
201*67e74705SXin Li }
202*67e74705SXin Li
TEST(ShiftedCodePositionTest,FindsNewCodePosition)203*67e74705SXin Li TEST(ShiftedCodePositionTest, FindsNewCodePosition) {
204*67e74705SXin Li Replacements Replaces;
205*67e74705SXin Li Replaces.insert(Replacement("", 0, 1, ""));
206*67e74705SXin Li Replaces.insert(Replacement("", 4, 3, " "));
207*67e74705SXin Li // Assume ' int i;' is turned into 'int i;' and cursor is located at '|'.
208*67e74705SXin Li EXPECT_EQ(0u, shiftedCodePosition(Replaces, 0)); // |int i;
209*67e74705SXin Li EXPECT_EQ(0u, shiftedCodePosition(Replaces, 1)); // |nt i;
210*67e74705SXin Li EXPECT_EQ(1u, shiftedCodePosition(Replaces, 2)); // i|t i;
211*67e74705SXin Li EXPECT_EQ(2u, shiftedCodePosition(Replaces, 3)); // in| i;
212*67e74705SXin Li EXPECT_EQ(3u, shiftedCodePosition(Replaces, 4)); // int| i;
213*67e74705SXin Li EXPECT_EQ(3u, shiftedCodePosition(Replaces, 5)); // int | i;
214*67e74705SXin Li EXPECT_EQ(3u, shiftedCodePosition(Replaces, 6)); // int |i;
215*67e74705SXin Li EXPECT_EQ(4u, shiftedCodePosition(Replaces, 7)); // int |;
216*67e74705SXin Li EXPECT_EQ(5u, shiftedCodePosition(Replaces, 8)); // int i|
217*67e74705SXin Li }
218*67e74705SXin Li
219*67e74705SXin Li // FIXME: Remove this test case when Replacements is implemented as std::vector
220*67e74705SXin Li // instead of std::set. The other ReplacementTest tests will need to be updated
221*67e74705SXin Li // at that point as well.
TEST(ShiftedCodePositionTest,VectorFindsNewCodePositionWithInserts)222*67e74705SXin Li TEST(ShiftedCodePositionTest, VectorFindsNewCodePositionWithInserts) {
223*67e74705SXin Li std::vector<Replacement> Replaces;
224*67e74705SXin Li Replaces.push_back(Replacement("", 0, 1, ""));
225*67e74705SXin Li Replaces.push_back(Replacement("", 4, 3, " "));
226*67e74705SXin Li // Assume ' int i;' is turned into 'int i;' and cursor is located at '|'.
227*67e74705SXin Li EXPECT_EQ(0u, shiftedCodePosition(Replaces, 0)); // |int i;
228*67e74705SXin Li EXPECT_EQ(0u, shiftedCodePosition(Replaces, 1)); // |nt i;
229*67e74705SXin Li EXPECT_EQ(1u, shiftedCodePosition(Replaces, 2)); // i|t i;
230*67e74705SXin Li EXPECT_EQ(2u, shiftedCodePosition(Replaces, 3)); // in| i;
231*67e74705SXin Li EXPECT_EQ(3u, shiftedCodePosition(Replaces, 4)); // int| i;
232*67e74705SXin Li EXPECT_EQ(3u, shiftedCodePosition(Replaces, 5)); // int | i;
233*67e74705SXin Li EXPECT_EQ(3u, shiftedCodePosition(Replaces, 6)); // int |i;
234*67e74705SXin Li EXPECT_EQ(4u, shiftedCodePosition(Replaces, 7)); // int |;
235*67e74705SXin Li EXPECT_EQ(5u, shiftedCodePosition(Replaces, 8)); // int i|
236*67e74705SXin Li }
237*67e74705SXin Li
TEST(ShiftedCodePositionTest,FindsNewCodePositionWithInserts)238*67e74705SXin Li TEST(ShiftedCodePositionTest, FindsNewCodePositionWithInserts) {
239*67e74705SXin Li Replacements Replaces;
240*67e74705SXin Li Replaces.insert(Replacement("", 4, 0, "\"\n\""));
241*67e74705SXin Li // Assume '"12345678"' is turned into '"1234"\n"5678"'.
242*67e74705SXin Li EXPECT_EQ(3u, shiftedCodePosition(Replaces, 3)); // "123|5678"
243*67e74705SXin Li EXPECT_EQ(7u, shiftedCodePosition(Replaces, 4)); // "1234|678"
244*67e74705SXin Li EXPECT_EQ(8u, shiftedCodePosition(Replaces, 5)); // "12345|78"
245*67e74705SXin Li }
246*67e74705SXin Li
TEST(ShiftedCodePositionTest,FindsNewCodePositionInReplacedText)247*67e74705SXin Li TEST(ShiftedCodePositionTest, FindsNewCodePositionInReplacedText) {
248*67e74705SXin Li Replacements Replaces;
249*67e74705SXin Li // Replace the first four characters with "abcd".
250*67e74705SXin Li Replaces.insert(Replacement("", 0, 4, "abcd"));
251*67e74705SXin Li for (unsigned i = 0; i < 3; ++i)
252*67e74705SXin Li EXPECT_EQ(i, shiftedCodePosition(Replaces, i));
253*67e74705SXin Li }
254*67e74705SXin Li
255*67e74705SXin Li class FlushRewrittenFilesTest : public ::testing::Test {
256*67e74705SXin Li public:
FlushRewrittenFilesTest()257*67e74705SXin Li FlushRewrittenFilesTest() {}
258*67e74705SXin Li
~FlushRewrittenFilesTest()259*67e74705SXin Li ~FlushRewrittenFilesTest() override {
260*67e74705SXin Li for (llvm::StringMap<std::string>::iterator I = TemporaryFiles.begin(),
261*67e74705SXin Li E = TemporaryFiles.end();
262*67e74705SXin Li I != E; ++I) {
263*67e74705SXin Li llvm::StringRef Name = I->second;
264*67e74705SXin Li std::error_code EC = llvm::sys::fs::remove(Name);
265*67e74705SXin Li (void)EC;
266*67e74705SXin Li assert(!EC);
267*67e74705SXin Li }
268*67e74705SXin Li }
269*67e74705SXin Li
createFile(llvm::StringRef Name,llvm::StringRef Content)270*67e74705SXin Li FileID createFile(llvm::StringRef Name, llvm::StringRef Content) {
271*67e74705SXin Li SmallString<1024> Path;
272*67e74705SXin Li int FD;
273*67e74705SXin Li std::error_code EC = llvm::sys::fs::createTemporaryFile(Name, "", FD, Path);
274*67e74705SXin Li assert(!EC);
275*67e74705SXin Li (void)EC;
276*67e74705SXin Li
277*67e74705SXin Li llvm::raw_fd_ostream OutStream(FD, true);
278*67e74705SXin Li OutStream << Content;
279*67e74705SXin Li OutStream.close();
280*67e74705SXin Li const FileEntry *File = Context.Files.getFile(Path);
281*67e74705SXin Li assert(File != nullptr);
282*67e74705SXin Li
283*67e74705SXin Li StringRef Found =
284*67e74705SXin Li TemporaryFiles.insert(std::make_pair(Name, Path.str())).first->second;
285*67e74705SXin Li assert(Found == Path);
286*67e74705SXin Li (void)Found;
287*67e74705SXin Li return Context.Sources.createFileID(File, SourceLocation(), SrcMgr::C_User);
288*67e74705SXin Li }
289*67e74705SXin Li
getFileContentFromDisk(llvm::StringRef Name)290*67e74705SXin Li std::string getFileContentFromDisk(llvm::StringRef Name) {
291*67e74705SXin Li std::string Path = TemporaryFiles.lookup(Name);
292*67e74705SXin Li assert(!Path.empty());
293*67e74705SXin Li // We need to read directly from the FileManager without relaying through
294*67e74705SXin Li // a FileEntry, as otherwise we'd read through an already opened file
295*67e74705SXin Li // descriptor, which might not see the changes made.
296*67e74705SXin Li // FIXME: Figure out whether there is a way to get the SourceManger to
297*67e74705SXin Li // reopen the file.
298*67e74705SXin Li auto FileBuffer = Context.Files.getBufferForFile(Path);
299*67e74705SXin Li return (*FileBuffer)->getBuffer();
300*67e74705SXin Li }
301*67e74705SXin Li
302*67e74705SXin Li llvm::StringMap<std::string> TemporaryFiles;
303*67e74705SXin Li RewriterTestContext Context;
304*67e74705SXin Li };
305*67e74705SXin Li
TEST_F(FlushRewrittenFilesTest,StoresChangesOnDisk)306*67e74705SXin Li TEST_F(FlushRewrittenFilesTest, StoresChangesOnDisk) {
307*67e74705SXin Li FileID ID = createFile("input.cpp", "line1\nline2\nline3\nline4");
308*67e74705SXin Li Replacements Replaces;
309*67e74705SXin Li Replaces.insert(Replacement(Context.Sources, Context.getLocation(ID, 2, 1),
310*67e74705SXin Li 5, "replaced"));
311*67e74705SXin Li EXPECT_TRUE(applyAllReplacements(Replaces, Context.Rewrite));
312*67e74705SXin Li EXPECT_FALSE(Context.Rewrite.overwriteChangedFiles());
313*67e74705SXin Li EXPECT_EQ("line1\nreplaced\nline3\nline4",
314*67e74705SXin Li getFileContentFromDisk("input.cpp"));
315*67e74705SXin Li }
316*67e74705SXin Li
317*67e74705SXin Li namespace {
318*67e74705SXin Li template <typename T>
319*67e74705SXin Li class TestVisitor : public clang::RecursiveASTVisitor<T> {
320*67e74705SXin Li public:
runOver(StringRef Code)321*67e74705SXin Li bool runOver(StringRef Code) {
322*67e74705SXin Li return runToolOnCode(new TestAction(this), Code);
323*67e74705SXin Li }
324*67e74705SXin Li
325*67e74705SXin Li protected:
326*67e74705SXin Li clang::SourceManager *SM;
327*67e74705SXin Li clang::ASTContext *Context;
328*67e74705SXin Li
329*67e74705SXin Li private:
330*67e74705SXin Li class FindConsumer : public clang::ASTConsumer {
331*67e74705SXin Li public:
FindConsumer(TestVisitor * Visitor)332*67e74705SXin Li FindConsumer(TestVisitor *Visitor) : Visitor(Visitor) {}
333*67e74705SXin Li
HandleTranslationUnit(clang::ASTContext & Context)334*67e74705SXin Li void HandleTranslationUnit(clang::ASTContext &Context) override {
335*67e74705SXin Li Visitor->TraverseDecl(Context.getTranslationUnitDecl());
336*67e74705SXin Li }
337*67e74705SXin Li
338*67e74705SXin Li private:
339*67e74705SXin Li TestVisitor *Visitor;
340*67e74705SXin Li };
341*67e74705SXin Li
342*67e74705SXin Li class TestAction : public clang::ASTFrontendAction {
343*67e74705SXin Li public:
TestAction(TestVisitor * Visitor)344*67e74705SXin Li TestAction(TestVisitor *Visitor) : Visitor(Visitor) {}
345*67e74705SXin Li
346*67e74705SXin Li std::unique_ptr<clang::ASTConsumer>
CreateASTConsumer(clang::CompilerInstance & compiler,llvm::StringRef dummy)347*67e74705SXin Li CreateASTConsumer(clang::CompilerInstance &compiler,
348*67e74705SXin Li llvm::StringRef dummy) override {
349*67e74705SXin Li Visitor->SM = &compiler.getSourceManager();
350*67e74705SXin Li Visitor->Context = &compiler.getASTContext();
351*67e74705SXin Li /// TestConsumer will be deleted by the framework calling us.
352*67e74705SXin Li return llvm::make_unique<FindConsumer>(Visitor);
353*67e74705SXin Li }
354*67e74705SXin Li
355*67e74705SXin Li private:
356*67e74705SXin Li TestVisitor *Visitor;
357*67e74705SXin Li };
358*67e74705SXin Li };
359*67e74705SXin Li } // end namespace
360*67e74705SXin Li
expectReplacementAt(const Replacement & Replace,StringRef File,unsigned Offset,unsigned Length)361*67e74705SXin Li void expectReplacementAt(const Replacement &Replace,
362*67e74705SXin Li StringRef File, unsigned Offset, unsigned Length) {
363*67e74705SXin Li ASSERT_TRUE(Replace.isApplicable());
364*67e74705SXin Li EXPECT_EQ(File, Replace.getFilePath());
365*67e74705SXin Li EXPECT_EQ(Offset, Replace.getOffset());
366*67e74705SXin Li EXPECT_EQ(Length, Replace.getLength());
367*67e74705SXin Li }
368*67e74705SXin Li
369*67e74705SXin Li class ClassDeclXVisitor : public TestVisitor<ClassDeclXVisitor> {
370*67e74705SXin Li public:
VisitCXXRecordDecl(CXXRecordDecl * Record)371*67e74705SXin Li bool VisitCXXRecordDecl(CXXRecordDecl *Record) {
372*67e74705SXin Li if (Record->getName() == "X") {
373*67e74705SXin Li Replace = Replacement(*SM, Record, "");
374*67e74705SXin Li }
375*67e74705SXin Li return true;
376*67e74705SXin Li }
377*67e74705SXin Li Replacement Replace;
378*67e74705SXin Li };
379*67e74705SXin Li
TEST(Replacement,CanBeConstructedFromNode)380*67e74705SXin Li TEST(Replacement, CanBeConstructedFromNode) {
381*67e74705SXin Li ClassDeclXVisitor ClassDeclX;
382*67e74705SXin Li EXPECT_TRUE(ClassDeclX.runOver(" class X;"));
383*67e74705SXin Li expectReplacementAt(ClassDeclX.Replace, "input.cc", 5, 7);
384*67e74705SXin Li }
385*67e74705SXin Li
TEST(Replacement,ReplacesAtSpellingLocation)386*67e74705SXin Li TEST(Replacement, ReplacesAtSpellingLocation) {
387*67e74705SXin Li ClassDeclXVisitor ClassDeclX;
388*67e74705SXin Li EXPECT_TRUE(ClassDeclX.runOver("#define A(Y) Y\nA(class X);"));
389*67e74705SXin Li expectReplacementAt(ClassDeclX.Replace, "input.cc", 17, 7);
390*67e74705SXin Li }
391*67e74705SXin Li
392*67e74705SXin Li class CallToFVisitor : public TestVisitor<CallToFVisitor> {
393*67e74705SXin Li public:
VisitCallExpr(CallExpr * Call)394*67e74705SXin Li bool VisitCallExpr(CallExpr *Call) {
395*67e74705SXin Li if (Call->getDirectCallee()->getName() == "F") {
396*67e74705SXin Li Replace = Replacement(*SM, Call, "");
397*67e74705SXin Li }
398*67e74705SXin Li return true;
399*67e74705SXin Li }
400*67e74705SXin Li Replacement Replace;
401*67e74705SXin Li };
402*67e74705SXin Li
TEST(Replacement,FunctionCall)403*67e74705SXin Li TEST(Replacement, FunctionCall) {
404*67e74705SXin Li CallToFVisitor CallToF;
405*67e74705SXin Li EXPECT_TRUE(CallToF.runOver("void F(); void G() { F(); }"));
406*67e74705SXin Li expectReplacementAt(CallToF.Replace, "input.cc", 21, 3);
407*67e74705SXin Li }
408*67e74705SXin Li
TEST(Replacement,TemplatedFunctionCall)409*67e74705SXin Li TEST(Replacement, TemplatedFunctionCall) {
410*67e74705SXin Li CallToFVisitor CallToF;
411*67e74705SXin Li EXPECT_TRUE(CallToF.runOver(
412*67e74705SXin Li "template <typename T> void F(); void G() { F<int>(); }"));
413*67e74705SXin Li expectReplacementAt(CallToF.Replace, "input.cc", 43, 8);
414*67e74705SXin Li }
415*67e74705SXin Li
416*67e74705SXin Li class NestedNameSpecifierAVisitor
417*67e74705SXin Li : public TestVisitor<NestedNameSpecifierAVisitor> {
418*67e74705SXin Li public:
TraverseNestedNameSpecifierLoc(NestedNameSpecifierLoc NNSLoc)419*67e74705SXin Li bool TraverseNestedNameSpecifierLoc(NestedNameSpecifierLoc NNSLoc) {
420*67e74705SXin Li if (NNSLoc.getNestedNameSpecifier()) {
421*67e74705SXin Li if (const NamespaceDecl* NS = NNSLoc.getNestedNameSpecifier()->getAsNamespace()) {
422*67e74705SXin Li if (NS->getName() == "a") {
423*67e74705SXin Li Replace = Replacement(*SM, &NNSLoc, "", Context->getLangOpts());
424*67e74705SXin Li }
425*67e74705SXin Li }
426*67e74705SXin Li }
427*67e74705SXin Li return TestVisitor<NestedNameSpecifierAVisitor>::TraverseNestedNameSpecifierLoc(
428*67e74705SXin Li NNSLoc);
429*67e74705SXin Li }
430*67e74705SXin Li Replacement Replace;
431*67e74705SXin Li };
432*67e74705SXin Li
TEST(Replacement,ColonColon)433*67e74705SXin Li TEST(Replacement, ColonColon) {
434*67e74705SXin Li NestedNameSpecifierAVisitor VisitNNSA;
435*67e74705SXin Li EXPECT_TRUE(VisitNNSA.runOver("namespace a { void f() { ::a::f(); } }"));
436*67e74705SXin Li expectReplacementAt(VisitNNSA.Replace, "input.cc", 25, 5);
437*67e74705SXin Li }
438*67e74705SXin Li
TEST(Range,overlaps)439*67e74705SXin Li TEST(Range, overlaps) {
440*67e74705SXin Li EXPECT_TRUE(Range(10, 10).overlapsWith(Range(0, 11)));
441*67e74705SXin Li EXPECT_TRUE(Range(0, 11).overlapsWith(Range(10, 10)));
442*67e74705SXin Li EXPECT_FALSE(Range(10, 10).overlapsWith(Range(0, 10)));
443*67e74705SXin Li EXPECT_FALSE(Range(0, 10).overlapsWith(Range(10, 10)));
444*67e74705SXin Li EXPECT_TRUE(Range(0, 10).overlapsWith(Range(2, 6)));
445*67e74705SXin Li EXPECT_TRUE(Range(2, 6).overlapsWith(Range(0, 10)));
446*67e74705SXin Li }
447*67e74705SXin Li
TEST(Range,contains)448*67e74705SXin Li TEST(Range, contains) {
449*67e74705SXin Li EXPECT_TRUE(Range(0, 10).contains(Range(0, 10)));
450*67e74705SXin Li EXPECT_TRUE(Range(0, 10).contains(Range(2, 6)));
451*67e74705SXin Li EXPECT_FALSE(Range(2, 6).contains(Range(0, 10)));
452*67e74705SXin Li EXPECT_FALSE(Range(0, 10).contains(Range(0, 11)));
453*67e74705SXin Li }
454*67e74705SXin Li
TEST(Range,CalculateRangesOfReplacements)455*67e74705SXin Li TEST(Range, CalculateRangesOfReplacements) {
456*67e74705SXin Li // Before: aaaabbbbbbz
457*67e74705SXin Li // After : bbbbbbzzzzzzoooooooooooooooo
458*67e74705SXin Li Replacements Replaces;
459*67e74705SXin Li Replaces.insert(Replacement("foo", 0, 4, ""));
460*67e74705SXin Li Replaces.insert(Replacement("foo", 10, 1, "zzzzzz"));
461*67e74705SXin Li Replaces.insert(Replacement("foo", 11, 0, "oooooooooooooooo"));
462*67e74705SXin Li
463*67e74705SXin Li std::vector<Range> Ranges = calculateChangedRanges(Replaces);
464*67e74705SXin Li
465*67e74705SXin Li EXPECT_EQ(2ul, Ranges.size());
466*67e74705SXin Li EXPECT_TRUE(Ranges[0].getOffset() == 0);
467*67e74705SXin Li EXPECT_TRUE(Ranges[0].getLength() == 0);
468*67e74705SXin Li EXPECT_TRUE(Ranges[1].getOffset() == 6);
469*67e74705SXin Li EXPECT_TRUE(Ranges[1].getLength() == 22);
470*67e74705SXin Li }
471*67e74705SXin Li
TEST(Range,RangesAfterReplacements)472*67e74705SXin Li TEST(Range, RangesAfterReplacements) {
473*67e74705SXin Li std::vector<Range> Ranges = {Range(5, 2), Range(10, 5)};
474*67e74705SXin Li Replacements Replaces = {Replacement("foo", 0, 2, "1234")};
475*67e74705SXin Li std::vector<Range> Expected = {Range(0, 4), Range(7, 2), Range(12, 5)};
476*67e74705SXin Li EXPECT_EQ(Expected, calculateRangesAfterReplacements(Replaces, Ranges));
477*67e74705SXin Li }
478*67e74705SXin Li
TEST(Range,RangesBeforeReplacements)479*67e74705SXin Li TEST(Range, RangesBeforeReplacements) {
480*67e74705SXin Li std::vector<Range> Ranges = {Range(5, 2), Range(10, 5)};
481*67e74705SXin Li Replacements Replaces = {Replacement("foo", 20, 2, "1234")};
482*67e74705SXin Li std::vector<Range> Expected = {Range(5, 2), Range(10, 5), Range(20, 4)};
483*67e74705SXin Li EXPECT_EQ(Expected, calculateRangesAfterReplacements(Replaces, Ranges));
484*67e74705SXin Li }
485*67e74705SXin Li
TEST(Range,NotAffectedByReplacements)486*67e74705SXin Li TEST(Range, NotAffectedByReplacements) {
487*67e74705SXin Li std::vector<Range> Ranges = {Range(0, 2), Range(5, 2), Range(10, 5)};
488*67e74705SXin Li Replacements Replaces = {Replacement("foo", 3, 2, "12"),
489*67e74705SXin Li Replacement("foo", 12, 2, "12"),
490*67e74705SXin Li Replacement("foo", 20, 5, "")};
491*67e74705SXin Li std::vector<Range> Expected = {Range(0, 2), Range(3, 4), Range(10, 5),
492*67e74705SXin Li Range(20, 0)};
493*67e74705SXin Li EXPECT_EQ(Expected, calculateRangesAfterReplacements(Replaces, Ranges));
494*67e74705SXin Li }
495*67e74705SXin Li
TEST(Range,RangesWithNonOverlappingReplacements)496*67e74705SXin Li TEST(Range, RangesWithNonOverlappingReplacements) {
497*67e74705SXin Li std::vector<Range> Ranges = {Range(0, 2), Range(5, 2), Range(10, 5)};
498*67e74705SXin Li Replacements Replaces = {Replacement("foo", 3, 1, ""),
499*67e74705SXin Li Replacement("foo", 6, 1, "123"),
500*67e74705SXin Li Replacement("foo", 20, 2, "12345")};
501*67e74705SXin Li std::vector<Range> Expected = {Range(0, 2), Range(3, 0), Range(4, 4),
502*67e74705SXin Li Range(11, 5), Range(21, 5)};
503*67e74705SXin Li EXPECT_EQ(Expected, calculateRangesAfterReplacements(Replaces, Ranges));
504*67e74705SXin Li }
505*67e74705SXin Li
TEST(Range,RangesWithOverlappingReplacements)506*67e74705SXin Li TEST(Range, RangesWithOverlappingReplacements) {
507*67e74705SXin Li std::vector<Range> Ranges = {Range(0, 2), Range(5, 2), Range(15, 5),
508*67e74705SXin Li Range(30, 5)};
509*67e74705SXin Li Replacements Replaces = {
510*67e74705SXin Li Replacement("foo", 1, 3, ""), Replacement("foo", 6, 1, "123"),
511*67e74705SXin Li Replacement("foo", 13, 3, "1"), Replacement("foo", 25, 15, "")};
512*67e74705SXin Li std::vector<Range> Expected = {Range(0, 1), Range(2, 4), Range(12, 5),
513*67e74705SXin Li Range(22, 0)};
514*67e74705SXin Li EXPECT_EQ(Expected, calculateRangesAfterReplacements(Replaces, Ranges));
515*67e74705SXin Li }
516*67e74705SXin Li
TEST(Range,MergeIntoOneRange)517*67e74705SXin Li TEST(Range, MergeIntoOneRange) {
518*67e74705SXin Li std::vector<Range> Ranges = {Range(0, 2), Range(5, 2), Range(15, 5)};
519*67e74705SXin Li Replacements Replaces = {Replacement("foo", 1, 15, "1234567890")};
520*67e74705SXin Li std::vector<Range> Expected = {Range(0, 15)};
521*67e74705SXin Li EXPECT_EQ(Expected, calculateRangesAfterReplacements(Replaces, Ranges));
522*67e74705SXin Li }
523*67e74705SXin Li
TEST(Range,ReplacementsStartingAtRangeOffsets)524*67e74705SXin Li TEST(Range, ReplacementsStartingAtRangeOffsets) {
525*67e74705SXin Li std::vector<Range> Ranges = {Range(0, 2), Range(5, 5), Range(15, 5)};
526*67e74705SXin Li Replacements Replaces = {
527*67e74705SXin Li Replacement("foo", 0, 2, "12"), Replacement("foo", 5, 1, "123"),
528*67e74705SXin Li Replacement("foo", 7, 4, "12345"), Replacement("foo", 15, 10, "12")};
529*67e74705SXin Li std::vector<Range> Expected = {Range(0, 2), Range(5, 9), Range(18, 2)};
530*67e74705SXin Li EXPECT_EQ(Expected, calculateRangesAfterReplacements(Replaces, Ranges));
531*67e74705SXin Li }
532*67e74705SXin Li
TEST(Range,ReplacementsEndingAtRangeEnds)533*67e74705SXin Li TEST(Range, ReplacementsEndingAtRangeEnds) {
534*67e74705SXin Li std::vector<Range> Ranges = {Range(0, 2), Range(5, 2), Range(15, 5)};
535*67e74705SXin Li Replacements Replaces = {Replacement("foo", 6, 1, "123"),
536*67e74705SXin Li Replacement("foo", 17, 3, "12")};
537*67e74705SXin Li std::vector<Range> Expected = {Range(0, 2), Range(5, 4), Range(17, 4)};
538*67e74705SXin Li EXPECT_EQ(Expected, calculateRangesAfterReplacements(Replaces, Ranges));
539*67e74705SXin Li }
540*67e74705SXin Li
TEST(Range,AjacentReplacements)541*67e74705SXin Li TEST(Range, AjacentReplacements) {
542*67e74705SXin Li std::vector<Range> Ranges = {Range(0, 0), Range(15, 5)};
543*67e74705SXin Li Replacements Replaces = {Replacement("foo", 1, 2, "123"),
544*67e74705SXin Li Replacement("foo", 12, 3, "1234")};
545*67e74705SXin Li std::vector<Range> Expected = {Range(0, 0), Range(1, 3), Range(13, 9)};
546*67e74705SXin Li EXPECT_EQ(Expected, calculateRangesAfterReplacements(Replaces, Ranges));
547*67e74705SXin Li }
548*67e74705SXin Li
TEST(Range,MergeRangesAfterReplacements)549*67e74705SXin Li TEST(Range, MergeRangesAfterReplacements) {
550*67e74705SXin Li std::vector<Range> Ranges = {Range(8, 0), Range(5, 2), Range(9, 0), Range(0, 1)};
551*67e74705SXin Li Replacements Replaces = {Replacement("foo", 1, 3, ""),
552*67e74705SXin Li Replacement("foo", 7, 0, "12"), Replacement("foo", 9, 2, "")};
553*67e74705SXin Li std::vector<Range> Expected = {Range(0, 1), Range(2, 4), Range(7, 0), Range(8, 0)};
554*67e74705SXin Li EXPECT_EQ(Expected, calculateRangesAfterReplacements(Replaces, Ranges));
555*67e74705SXin Li }
556*67e74705SXin Li
TEST(DeduplicateTest,removesDuplicates)557*67e74705SXin Li TEST(DeduplicateTest, removesDuplicates) {
558*67e74705SXin Li std::vector<Replacement> Input;
559*67e74705SXin Li Input.push_back(Replacement("fileA", 50, 0, " foo "));
560*67e74705SXin Li Input.push_back(Replacement("fileA", 10, 3, " bar "));
561*67e74705SXin Li Input.push_back(Replacement("fileA", 10, 2, " bar ")); // Length differs
562*67e74705SXin Li Input.push_back(Replacement("fileA", 9, 3, " bar ")); // Offset differs
563*67e74705SXin Li Input.push_back(Replacement("fileA", 50, 0, " foo ")); // Duplicate
564*67e74705SXin Li Input.push_back(Replacement("fileA", 51, 3, " bar "));
565*67e74705SXin Li Input.push_back(Replacement("fileB", 51, 3, " bar ")); // Filename differs!
566*67e74705SXin Li Input.push_back(Replacement("fileB", 60, 1, " bar "));
567*67e74705SXin Li Input.push_back(Replacement("fileA", 60, 2, " bar "));
568*67e74705SXin Li Input.push_back(Replacement("fileA", 51, 3, " moo ")); // Replacement text
569*67e74705SXin Li // differs!
570*67e74705SXin Li
571*67e74705SXin Li std::vector<Replacement> Expected;
572*67e74705SXin Li Expected.push_back(Replacement("fileA", 9, 3, " bar "));
573*67e74705SXin Li Expected.push_back(Replacement("fileA", 10, 2, " bar "));
574*67e74705SXin Li Expected.push_back(Replacement("fileA", 10, 3, " bar "));
575*67e74705SXin Li Expected.push_back(Replacement("fileA", 50, 0, " foo "));
576*67e74705SXin Li Expected.push_back(Replacement("fileA", 51, 3, " bar "));
577*67e74705SXin Li Expected.push_back(Replacement("fileA", 51, 3, " moo "));
578*67e74705SXin Li Expected.push_back(Replacement("fileB", 60, 1, " bar "));
579*67e74705SXin Li Expected.push_back(Replacement("fileA", 60, 2, " bar "));
580*67e74705SXin Li
581*67e74705SXin Li std::vector<Range> Conflicts; // Ignored for this test
582*67e74705SXin Li deduplicate(Input, Conflicts);
583*67e74705SXin Li
584*67e74705SXin Li EXPECT_EQ(3U, Conflicts.size());
585*67e74705SXin Li EXPECT_EQ(Expected, Input);
586*67e74705SXin Li }
587*67e74705SXin Li
TEST(DeduplicateTest,detectsConflicts)588*67e74705SXin Li TEST(DeduplicateTest, detectsConflicts) {
589*67e74705SXin Li {
590*67e74705SXin Li std::vector<Replacement> Input;
591*67e74705SXin Li Input.push_back(Replacement("fileA", 0, 5, " foo "));
592*67e74705SXin Li Input.push_back(Replacement("fileA", 0, 5, " foo ")); // Duplicate not a
593*67e74705SXin Li // conflict.
594*67e74705SXin Li Input.push_back(Replacement("fileA", 2, 6, " bar "));
595*67e74705SXin Li Input.push_back(Replacement("fileA", 7, 3, " moo "));
596*67e74705SXin Li
597*67e74705SXin Li std::vector<Range> Conflicts;
598*67e74705SXin Li deduplicate(Input, Conflicts);
599*67e74705SXin Li
600*67e74705SXin Li // One duplicate is removed and the remaining three items form one
601*67e74705SXin Li // conflicted range.
602*67e74705SXin Li ASSERT_EQ(3u, Input.size());
603*67e74705SXin Li ASSERT_EQ(1u, Conflicts.size());
604*67e74705SXin Li ASSERT_EQ(0u, Conflicts.front().getOffset());
605*67e74705SXin Li ASSERT_EQ(3u, Conflicts.front().getLength());
606*67e74705SXin Li }
607*67e74705SXin Li {
608*67e74705SXin Li std::vector<Replacement> Input;
609*67e74705SXin Li
610*67e74705SXin Li // Expected sorted order is shown. It is the sorted order to which the
611*67e74705SXin Li // returned conflict info refers to.
612*67e74705SXin Li Input.push_back(Replacement("fileA", 0, 5, " foo ")); // 0
613*67e74705SXin Li Input.push_back(Replacement("fileA", 5, 5, " bar ")); // 1
614*67e74705SXin Li Input.push_back(Replacement("fileA", 6, 0, " bar ")); // 3
615*67e74705SXin Li Input.push_back(Replacement("fileA", 5, 5, " moo ")); // 2
616*67e74705SXin Li Input.push_back(Replacement("fileA", 7, 2, " bar ")); // 4
617*67e74705SXin Li Input.push_back(Replacement("fileA", 15, 5, " golf ")); // 5
618*67e74705SXin Li Input.push_back(Replacement("fileA", 16, 5, " bag ")); // 6
619*67e74705SXin Li Input.push_back(Replacement("fileA", 10, 3, " club ")); // 7
620*67e74705SXin Li
621*67e74705SXin Li // #3 is special in that it is completely contained by another conflicting
622*67e74705SXin Li // Replacement. #4 ensures #3 hasn't messed up the conflicting range size.
623*67e74705SXin Li
624*67e74705SXin Li std::vector<Range> Conflicts;
625*67e74705SXin Li deduplicate(Input, Conflicts);
626*67e74705SXin Li
627*67e74705SXin Li // No duplicates
628*67e74705SXin Li ASSERT_EQ(8u, Input.size());
629*67e74705SXin Li ASSERT_EQ(2u, Conflicts.size());
630*67e74705SXin Li ASSERT_EQ(1u, Conflicts[0].getOffset());
631*67e74705SXin Li ASSERT_EQ(4u, Conflicts[0].getLength());
632*67e74705SXin Li ASSERT_EQ(6u, Conflicts[1].getOffset());
633*67e74705SXin Li ASSERT_EQ(2u, Conflicts[1].getLength());
634*67e74705SXin Li }
635*67e74705SXin Li }
636*67e74705SXin Li
637*67e74705SXin Li class MergeReplacementsTest : public ::testing::Test {
638*67e74705SXin Li protected:
mergeAndTestRewrite(StringRef Code,StringRef Intermediate,StringRef Result,const Replacements & First,const Replacements & Second)639*67e74705SXin Li void mergeAndTestRewrite(StringRef Code, StringRef Intermediate,
640*67e74705SXin Li StringRef Result, const Replacements &First,
641*67e74705SXin Li const Replacements &Second) {
642*67e74705SXin Li // These are mainly to verify the test itself and make it easier to read.
643*67e74705SXin Li auto AfterFirst = applyAllReplacements(Code, First);
644*67e74705SXin Li EXPECT_TRUE(static_cast<bool>(AfterFirst));
645*67e74705SXin Li auto InSequenceRewrite = applyAllReplacements(*AfterFirst, Second);
646*67e74705SXin Li EXPECT_TRUE(static_cast<bool>(InSequenceRewrite));
647*67e74705SXin Li EXPECT_EQ(Intermediate, *AfterFirst);
648*67e74705SXin Li EXPECT_EQ(Result, *InSequenceRewrite);
649*67e74705SXin Li
650*67e74705SXin Li tooling::Replacements Merged = mergeReplacements(First, Second);
651*67e74705SXin Li auto MergedRewrite = applyAllReplacements(Code, Merged);
652*67e74705SXin Li EXPECT_TRUE(static_cast<bool>(MergedRewrite));
653*67e74705SXin Li EXPECT_EQ(*InSequenceRewrite, *MergedRewrite);
654*67e74705SXin Li if (*InSequenceRewrite != *MergedRewrite)
655*67e74705SXin Li for (tooling::Replacement M : Merged)
656*67e74705SXin Li llvm::errs() << M.getOffset() << " " << M.getLength() << " "
657*67e74705SXin Li << M.getReplacementText() << "\n";
658*67e74705SXin Li }
mergeAndTestRewrite(StringRef Code,const Replacements & First,const Replacements & Second)659*67e74705SXin Li void mergeAndTestRewrite(StringRef Code, const Replacements &First,
660*67e74705SXin Li const Replacements &Second) {
661*67e74705SXin Li auto AfterFirst = applyAllReplacements(Code, First);
662*67e74705SXin Li EXPECT_TRUE(static_cast<bool>(AfterFirst));
663*67e74705SXin Li auto InSequenceRewrite = applyAllReplacements(*AfterFirst, Second);
664*67e74705SXin Li tooling::Replacements Merged = mergeReplacements(First, Second);
665*67e74705SXin Li auto MergedRewrite = applyAllReplacements(Code, Merged);
666*67e74705SXin Li EXPECT_TRUE(static_cast<bool>(MergedRewrite));
667*67e74705SXin Li EXPECT_EQ(*InSequenceRewrite, *MergedRewrite);
668*67e74705SXin Li if (*InSequenceRewrite != *MergedRewrite)
669*67e74705SXin Li for (tooling::Replacement M : Merged)
670*67e74705SXin Li llvm::errs() << M.getOffset() << " " << M.getLength() << " "
671*67e74705SXin Li << M.getReplacementText() << "\n";
672*67e74705SXin Li }
673*67e74705SXin Li };
674*67e74705SXin Li
TEST_F(MergeReplacementsTest,Offsets)675*67e74705SXin Li TEST_F(MergeReplacementsTest, Offsets) {
676*67e74705SXin Li mergeAndTestRewrite("aaa", "aabab", "cacabab",
677*67e74705SXin Li {{"", 2, 0, "b"}, {"", 3, 0, "b"}},
678*67e74705SXin Li {{"", 0, 0, "c"}, {"", 1, 0, "c"}});
679*67e74705SXin Li mergeAndTestRewrite("aaa", "babaa", "babacac",
680*67e74705SXin Li {{"", 0, 0, "b"}, {"", 1, 0, "b"}},
681*67e74705SXin Li {{"", 4, 0, "c"}, {"", 5, 0, "c"}});
682*67e74705SXin Li mergeAndTestRewrite("aaaa", "aaa", "aac", {{"", 1, 1, ""}},
683*67e74705SXin Li {{"", 2, 1, "c"}});
684*67e74705SXin Li
685*67e74705SXin Li mergeAndTestRewrite("aa", "bbabba", "bbabcba",
686*67e74705SXin Li {{"", 0, 0, "bb"}, {"", 1, 0, "bb"}}, {{"", 4, 0, "c"}});
687*67e74705SXin Li }
688*67e74705SXin Li
TEST_F(MergeReplacementsTest,Concatenations)689*67e74705SXin Li TEST_F(MergeReplacementsTest, Concatenations) {
690*67e74705SXin Li // Basic concatenations. It is important to merge these into a single
691*67e74705SXin Li // replacement to ensure the correct order.
692*67e74705SXin Li EXPECT_EQ((Replacements{{"", 0, 0, "ab"}}),
693*67e74705SXin Li mergeReplacements({{"", 0, 0, "a"}}, {{"", 1, 0, "b"}}));
694*67e74705SXin Li EXPECT_EQ((Replacements{{"", 0, 0, "ba"}}),
695*67e74705SXin Li mergeReplacements({{"", 0, 0, "a"}}, {{"", 0, 0, "b"}}));
696*67e74705SXin Li mergeAndTestRewrite("", "a", "ab", {{"", 0, 0, "a"}}, {{"", 1, 0, "b"}});
697*67e74705SXin Li mergeAndTestRewrite("", "a", "ba", {{"", 0, 0, "a"}}, {{"", 0, 0, "b"}});
698*67e74705SXin Li }
699*67e74705SXin Li
TEST_F(MergeReplacementsTest,NotChangingLengths)700*67e74705SXin Li TEST_F(MergeReplacementsTest, NotChangingLengths) {
701*67e74705SXin Li mergeAndTestRewrite("aaaa", "abba", "acca", {{"", 1, 2, "bb"}},
702*67e74705SXin Li {{"", 1, 2, "cc"}});
703*67e74705SXin Li mergeAndTestRewrite("aaaa", "abba", "abcc", {{"", 1, 2, "bb"}},
704*67e74705SXin Li {{"", 2, 2, "cc"}});
705*67e74705SXin Li mergeAndTestRewrite("aaaa", "abba", "ccba", {{"", 1, 2, "bb"}},
706*67e74705SXin Li {{"", 0, 2, "cc"}});
707*67e74705SXin Li mergeAndTestRewrite("aaaaaa", "abbdda", "abccda",
708*67e74705SXin Li {{"", 1, 2, "bb"}, {"", 3, 2, "dd"}}, {{"", 2, 2, "cc"}});
709*67e74705SXin Li }
710*67e74705SXin Li
TEST_F(MergeReplacementsTest,OverlappingRanges)711*67e74705SXin Li TEST_F(MergeReplacementsTest, OverlappingRanges) {
712*67e74705SXin Li mergeAndTestRewrite("aaa", "bbd", "bcbcd",
713*67e74705SXin Li {{"", 0, 1, "bb"}, {"", 1, 2, "d"}},
714*67e74705SXin Li {{"", 1, 0, "c"}, {"", 2, 0, "c"}});
715*67e74705SXin Li
716*67e74705SXin Li mergeAndTestRewrite("aaaa", "aabbaa", "acccca", {{"", 2, 0, "bb"}},
717*67e74705SXin Li {{"", 1, 4, "cccc"}});
718*67e74705SXin Li mergeAndTestRewrite("aaaa", "aababa", "acccca",
719*67e74705SXin Li {{"", 2, 0, "b"}, {"", 3, 0, "b"}}, {{"", 1, 4, "cccc"}});
720*67e74705SXin Li mergeAndTestRewrite("aaaaaa", "abbbba", "abba", {{"", 1, 4, "bbbb"}},
721*67e74705SXin Li {{"", 2, 2, ""}});
722*67e74705SXin Li mergeAndTestRewrite("aaaa", "aa", "cc", {{"", 1, 1, ""}, {"", 2, 1, ""}},
723*67e74705SXin Li {{"", 0, 2, "cc"}});
724*67e74705SXin Li mergeAndTestRewrite("aa", "abbba", "abcbcba", {{"", 1, 0, "bbb"}},
725*67e74705SXin Li {{"", 2, 0, "c"}, {"", 3, 0, "c"}});
726*67e74705SXin Li
727*67e74705SXin Li mergeAndTestRewrite("aaa", "abbab", "ccdd",
728*67e74705SXin Li {{"", 0, 1, ""}, {"", 2, 0, "bb"}, {"", 3, 0, "b"}},
729*67e74705SXin Li {{"", 0, 2, "cc"}, {"", 2, 3, "dd"}});
730*67e74705SXin Li mergeAndTestRewrite("aa", "babbab", "ccdd",
731*67e74705SXin Li {{"", 0, 0, "b"}, {"", 1, 0, "bb"}, {"", 2, 0, "b"}},
732*67e74705SXin Li {{"", 0, 3, "cc"}, {"", 3, 3, "dd"}});
733*67e74705SXin Li }
734*67e74705SXin Li
735*67e74705SXin Li } // end namespace tooling
736*67e74705SXin Li } // end namespace clang
737