ioeric marked an inline comment as done. ================ Comment at: unittests/Tooling/RefactoringTest.cpp:112 @@ -111,4 +111,3 @@ -TEST_F(ReplacementTest, CanApplyReplacements) { - FileID ID = Context.createInMemoryFile("input.cpp", - "line1\nline2\nline3\nline4"); +static Replacements toReplacements(const std::set<Replacement> &Replaces) { + Replacements Result; ---------------- klimek wrote: > Looks like we should put this function into some header, so we don't need to > re-implement it in every test? Done.
Added unittests/Tooling/ReplacementTest.h that contains `ReplacementTest` class and `toReplacements`, both of which are used in multiple test files. https://reviews.llvm.org/D21748 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits