RFR: 7903324: Improve per-class reporting of JUnit tests, in .jtr file
Christian Stein
cstein at openjdk.org
Thu Oct 27 18:22:13 UTC 2022
On Mon, 26 Sep 2022 15:31:04 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:
>> src/share/classes/com/sun/javatest/regtest/agent/JUnitRunner.java line 182:
>>
>>> 180: @Override
>>> 181: public void executionFinished(TestIdentifier identifier, TestExecutionResult result) {
>>> 182: System.out.println(identifier.getDisplayName() + ": " + result.getStatus());
>>
>> Which `System` stream is better? `out`, or `err` ?
>
> Maybe `PrintingListener` should have a constructor which takes a `PrintStream` or `PrintWriter`, so that we only get to reference the `System` stream once, when the listener is created...
Using `System.err` via a constructor parameter since https://github.com/openjdk/jtreg/pull/127/commits/7adedc204ed22192cd25a9ed42dad4f0af20249c
-------------
PR: https://git.openjdk.org/jtreg/pull/127
More information about the jtreg-dev
mailing list