On Wed, 23 Apr 2025 21:06:24 GMT, Alexander Zvegintsev <azveg...@openjdk.org> 
wrote:

>> 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.

it does seem to cut a lot of repeated boiler plate

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/24806#discussion_r2056914166

Reply via email to