[PATCH] D17852: Added formatAndApplyAllReplacements that works on multiple files in libTooling.

[Eric Liu via cfe-commits]

Just wanted to check if there are existing test cases for format::getStyle.
If there is no test for it, I'll add test cases for sure :=)

On Mon, Mar 21, 2016, 21:20 Manuel Klimek <klimek at google.com> wrote:

> On Mon, Mar 21, 2016 at 6:36 PM Eric Liu <ioeric at google.com> wrote:
>
>> ioeric added inline comments.
>>
>> ================
>> Comment at: unittests/Tooling/RefactoringTest.cpp:206
>> @@ +205,3 @@
>> +TEST_F(ReplacementTest, ReplaceAndFormatNoStyle) {
>> +  std::string Code = "MyType012345678901234567890123456789 *a =\n"
>> +                     "    new MyType012345678901234567890123456789();\n"
>> ----------------
>> klimek wrote:
>> > I'd rely on the general case to be covered by the other test then, and
>> just test that the style is picked up correctly (or falls back correctly to
>> LLVM style).
>> You are right. Shall I remove this test case for now and add general test
>> cases(I mean non-unit test) for format::getStyle in the future?
>>
>
> Any reason to not add this now?
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160321/d7701995/attachment-0001.html>