[PATCH] D30813: [clang-rename] Introduce an unittest skeleton for clang-rename.

Eric Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 10 02:05:40 PST 2017


ioeric accepted this revision.
ioeric added a comment.
This revision is now accepted and ready to land.

lg



================
Comment at: unittests/clang-rename/ClangRenameTests.cpp:91
+        "#include \"%s\"\n%s", HeaderName.c_str(), Expected.str().c_str());
+    EXPECT_EQ(format(ExpectedCode), format(Actual));
+  }
----------------
I'd put EXPECT_EQ in test cases so that it would be easier to see which case fails. Maybe pull out a function for wrapping `Expected` and EXPECT_EQ on the formatted result and `format(Actual)`?


https://reviews.llvm.org/D30813





More information about the cfe-commits mailing list