<div dir="ltr">Done in r194494.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 8, 2013 at 2:17 AM, Peter Collingbourne <span dir="ltr"><<a href="mailto:peter@pcc.me.uk" target="_blank">peter@pcc.me.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thu, Nov 07, 2013 at 11:18:05PM -0000, Manuel Klimek wrote:<br>
> +TEST(ClangToolTest, InjectDiagnosticConsumer) {<br>
> +  FixedCompilationDatabase Compilations("/", std::vector<std::string>());<br>
> +  ClangTool Tool(Compilations, std::vector<std::string>(1, "/a.cc"));<br>
> +  Tool.mapVirtualFile("/a.cc", "int x = undeclared;");<br>
> +  TestDiagnosticConsumer Consumer;<br>
> +  Tool.setDiagnosticConsumer(&Consumer);<br>
> +  Tool.run(newFrontendActionFactory<SyntaxOnlyAction>());<br>
> +  EXPECT_EQ(1u, Consumer.NumDiagnosticsSeen);<br>
> +}<br>
<br>
</div>Should this also be testing buildASTs?  (I guess you could use a #warning<br>
directive in order to ensure ASTUnit returns an AST).<br>
<br>
Thanks,<br>
<span class="HOEnZb"><font color="#888888">--<br>
Peter<br>
</font></span></blockquote></div><br></div>