Add support for JUnit compatible XML output in the new test runner.

Add support for JUnit compatible XML output in the new test runner.

Implement a new output method in run-tests.py which
outputs JUnit compatible XML status information.

BUG=

Committed: https://code.google.com/p/v8/source/detail?r=14217

[Jakob Kummerow, 2013-04-09 09:41:35]

LGTM with a few minor comments.

https://codereview.chromium.org/13813003/diff/1/tools/testrunner/local/junit_...
File tools/testrunner/local/junit_output.py (right):

https://codereview.chromium.org/13813003/diff/1/tools/testrunner/local/junit_...
tools/testrunner/local/junit_output.py:32: class JUnitTestOutput:
nit: This class is small enough that you could just fold its functionality into
JUnitTestProgressIndicator if you wanted. I don't feel strongly about it either
way.

https://codereview.chromium.org/13813003/diff/1/tools/testrunner/local/progre...
File tools/testrunner/local/progress.py (right):

https://codereview.chromium.org/13813003/diff/1/tools/testrunner/local/progre...
tools/testrunner/local/progress.py:32: from testrunner.local import junit_output
nit: please use "from . import junit_output"

https://codereview.chromium.org/13813003/diff/1/tools/testrunner/local/progre...
tools/testrunner/local/progress.py:273: self.outputter.HasRunTest(
nit: suggestion for formatting that better reflects the semantic structure:

    self.outputter.HasRunTest(
        [test.GetLabel()] + self.runner.context.mode_flags + test.flags,
        test.duration,
        fail_text)

[palfia, 2013-04-10 17:23:32]

Thanks for the comments, I fixed the nits.

https://codereview.chromium.org/13813003/diff/1/tools/testrunner/local/junit_...
File tools/testrunner/local/junit_output.py (right):

https://codereview.chromium.org/13813003/diff/1/tools/testrunner/local/junit_...
tools/testrunner/local/junit_output.py:32: class JUnitTestOutput:
On 2013/04/09 09:41:35, Jakob wrote:
> nit: This class is small enough that you could just fold its functionality
into
> JUnitTestProgressIndicator if you wanted. I don't feel strongly about it
either
> way.

In this case I leave it as it is. I like the current form a bit better, because
I think that the separation of the XML writer and the Progress Indicator makes
the code more readable.

https://codereview.chromium.org/13813003/diff/1/tools/testrunner/local/progre...
File tools/testrunner/local/progress.py (right):

https://codereview.chromium.org/13813003/diff/1/tools/testrunner/local/progre...
tools/testrunner/local/progress.py:32: from testrunner.local import junit_output
On 2013/04/09 09:41:35, Jakob wrote:
> nit: please use "from . import junit_output"

Done.

https://codereview.chromium.org/13813003/diff/1/tools/testrunner/local/progre...
tools/testrunner/local/progress.py:273: self.outputter.HasRunTest(
On 2013/04/09 09:41:35, Jakob wrote:
> nit: suggestion for formatting that better reflects the semantic structure:
> 
>     self.outputter.HasRunTest(
>         [test.GetLabel()] + self.runner.context.mode_flags + test.flags,
>         test.duration,
>         fail_text)

Done.

[Jakob Kummerow, 2013-04-10 21:08:39]

lgtm

[palfia, 2013-04-11 00:22:40]

Committed manually as r14217 (presubmit successful).

[Prashanth Srinivasan, 2013-04-16 21:09:51]

On 2013/04/11 00:22:40, palfia wrote:
> Committed manually as r14217 (presubmit successful).

Is there a plan to include skipped tests as a part of JUnit compatible XML
output?