RFR: 8349535: Refactor ./pkcs11/Provider/MultipleLogins.sh to java test [v6]

Weijun Wang weijun at openjdk.org
Wed Apr 9 14:27:34 UTC 2025


On Wed, 19 Mar 2025 18:08:40 GMT, Mikhail Yankelevich <myankelevich at openjdk.org> wrote:

>> Moved the sh file logic to jtreg java test.
>
> Mikhail Yankelevich has updated the pull request incrementally with one additional commit since the last revision:
> 
>   minor: space at the end

test/jdk/sun/security/pkcs11/Provider/MultipleLogins.java line 75:

> 73:         for (int i = 0; i < NUM_PROVIDERS; i++) {
> 74:             // loop to set up test without security manger
> 75:             providers[i] = (SunPKCS11)newPKCS11Provider();

Not sure if it's worth updating, but since `configure` always returns a new provider, there is no need to call `newPKCS11Provider` here and we can just call `configure` on the same `Security.getProvider("SunPKCS11")`. Also, you can use the public `AuthProvider` instead of internal class `SunPKCS11` everywhere.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23590#discussion_r2035495685


More information about the security-dev mailing list