RFR: 8354554: Open source several clipboard tests batch1 [v2]
Alexander Zvegintsev
azvegint at openjdk.org
Wed Apr 23 21:08:55 UTC 2025
On Wed, 23 Apr 2025 03:13:35 GMT, Sergey Bylokhov <serb at openjdk.org> 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
More information about the client-libs-dev
mailing list