RFR: 8336332: Rework tests to avoid unrelated stderr output [v8]
Kevin Rushforth
kcr at openjdk.org
Thu Sep 18 17:54:44 UTC 2025
On Thu, 18 Sep 2025 16:33:45 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> This PR removes unrelated `stderr` output in the headless test logs by redirecting it to an in-memory buffer. Exceptions found in the buffer can be checked against the expected list.
>>
>> In the case when any mismatch is detected, whether the type or the number of exceptions of particular type, the accumulated buffer gets dumped to `stderr` and the test fails.
>>
>> ## How To
>>
>> To redirect stderr and later check the exceptions, surround your code with
>>
>> `OutputRedirect.suppressStderr()` and either `OutputRedirect.checkStderr()` or `OutputRedirect.checkAndRestoreStderr()` (ideally, in the `finally` block).
>>
>> To simply undo redirection, without checking, call `OutputRedirect.restoreStderr()`.
>>
>> To add the check to all the tests in the file, one can call the above mentioned methods inside `@BeforeEach` and `@AfterEach`.
>>
>> ## Changes
>>
>> - added `OutputRedirect` facility
>>
>> ## Miscellaneous
>>
>> `ErrorLoggingUtiltity` name will be fixed in a followup https://bugs.openjdk.org/browse/JDK-8367995
>
> Andy Goryachev has updated the pull request incrementally with one additional commit since the last revision:
>
> review comments
modules/javafx.base/src/test/java/test/javafx/util/OutputRedirect.java line 146:
> 144: throw new IllegalArgumentException("must specify Class<? extends Throwable>: " + c);
> 145: }
> 146: } else if(x instanceof String) {
Minor: space after `if`
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1897#discussion_r2360435742
More information about the openjfx-dev
mailing list