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