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 :=)<br><br><div class="gmail_quote"><div dir="ltr">On Mon, Mar 21, 2016, 21:20 Manuel Klimek <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Mon, Mar 21, 2016 at 6:36 PM Eric Liu <<a href="mailto:ioeric@google.com" target="_blank">ioeric@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ioeric added inline comments.<br>
<br>
================<br>
Comment at: unittests/Tooling/RefactoringTest.cpp:206<br>
@@ +205,3 @@<br>
+TEST_F(ReplacementTest, ReplaceAndFormatNoStyle) {<br>
+  std::string Code = "MyType012345678901234567890123456789 *a =\n"<br>
+                     "    new MyType012345678901234567890123456789();\n"<br>
----------------<br>
klimek wrote:<br>
> 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).<br>
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?<br></blockquote><div><br></div></div></div><div dir="ltr"><div class="gmail_quote"><div>Any reason to not add this now?</div></div></div>
</blockquote></div>