On Wed, 23 Apr 2025 03:13:35 GMT, Sergey Bylokhov <[email protected]> wrote:
>> few more clipboard tests are opensourced. Some of them are in problemlist.
>
> test/jdk/java/awt/Clipboard/LostOwnershipChainTest/SystemClipboard2ProcTest.java
> line 112:
>
>> 110: String javaPath = System.getProperty("java.home", "");
>> 111:
>> 112: Process process = new ProcessBuilder(
>
> Don't we already have some libraries, like ProcessTools and OutputAnalyzer,
> to run tests in a separate process?
> Right now, this test, for example, skips all VM parameters.
This is PR opening tests which had this code written a long time ago. Before
those libraries existed.
I guess it was a matter of how much effort should be put in to modernise the
test, just to get it into open.
Alexander just minimally upgraded it to use ProcessBuilder.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24806#discussion_r2056859607
