On Wed, 23 Apr 2025 03:13:35 GMT, Sergey Bylokhov <[email protected]> wrote:
>> Alexander Zvegintsev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> use ProcessTools
>
> 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.
@mrserb Thanks for the tip, it allows to remove a lot of duplicated code.
@prrace Changing it to use ProcessTools is quite easy, so updated.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24806#discussion_r2056872931
