<div dir="ltr">I can't check this until Monday.  But drive by comment: Why do we need the MAPMODE?  Can't the JUnitFormatter just treat xpasses as whatever it wants without the help of this option?<div><br></div><div>Also, I'm not really familiar with JUnit as a general tool.  Is there some reason a one-size-fits-all JSON formatter is insufficient?</div><div><br></div><div>Last question: Is the current stdout/stderr formatter re-written as a second formatter plugin, or not?</div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Sep 12, 2015 at 3:26 PM Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tfiala added a comment.<br>
<br>
A couple notes on this:<br>
<br>
- Failures don't currently capture backtraces.  I can add this later, but I am leaving it for another pass.  This change is big enough as is.  Right now I'm focusing on counts and green/red CI solutions.<br>
- Tests don't capture testcase-specific stdout/stderr.  This is too big a change for right now.  We can get there but not until we stop depending on the stdout/stderr going to other places for the whole test run.<br>
<br>
Skip reason is captured, as are bug numbers, exception types and exception messages, stuffed into relevant JUnit xml attributes.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D12831" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12831</a><br>
<br>
<br>
<br>
</blockquote></div>