On Mon, 13 Apr 2026 17:54:01 GMT, Mikhail Yankelevich 
<[email protected]> wrote:

>> test/jdk/sun/security/provider/KeyStore/DKSSystemPropertiesTest.java line 2:
>> 
>>> 1: /*
>>> 2:  * Copyright (c) 2013, 2026, Oracle and/or its affiliates. All rights 
>>> reserved.
>> 
>> this looks like a completely new file; is the copyright correct?
>
> 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.

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

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

Reply via email to