On Mon, 13 Apr 2026 18:38:04 GMT, Marcono1234 <[email protected]> wrote:

>> Kept it as is as this is actually an old test with other cases removed. I 
>> only edited and renamed it. 
>> The other file has older name and refactored other tests in addition to the 
>> new ones
>
> Out of curiosity: What is the reason why this separate test is needed though? 
> It seems some of the JUnit tests in `DKSTest` even call the `checkEntries` 
> method here.

This test uses system properties which could only set as a script or a 
subprocess. Either way this is cleaner than starting 2 versions of the same test

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30712#discussion_r3078479665

Reply via email to