<div dir="ltr"><div>We use lit tests for diagnostic messages. Currently they use FileCheck, but it probably makes sense to implement an analog of Clang's -verify in clang-tidy.</div><div><br></div>As for the tests for fix-its, I consider unit tests to be more readable for this purpose. They are also more isolated and closer to the check itself. In a lit test, if we verify the full text of the output, the FileCheck tests become quite bulky. Another reason to put these checks in the unit tests, is that each new lit test file has non-trivial run-time overhead.<div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, May 20, 2014 at 7:32 PM, Sean Silva <span dir="ltr"><<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Is there a particular reason that this is being tested with unittests instead of the usual lit tests? This kind of thing seems like it would be a lot more readable as a lit test.<span class="HOEnZb"><font color="#888888"><div>
<br></div><div>-- Sean Silva</div></font></span></div></blockquote></div>
</div></div></div>