On Wed, 12 Nov 2025 17:58:51 GMT, Matthew Donovan <[email protected]> wrote:

> In this PR, I updated a few crypto tests to remove the hard-coded provider 
> name and use the `test.provider.name` property instead. These tests were 
> missed while working on 
> [JDK-8341927](https://bugs.openjdk.org/browse/JDK-8341927).

test/jdk/com/sun/crypto/provider/KeyAgreement/DHGenSharedSecret.java line 61:

> 59:         KeyPairGenerator keyGen =
> 60:             KeyPairGenerator.getInstance("DH",
> 61:                     System.getProperty("test.provider.name", "SunJCE"));

You may consider defining a final string `PROVIDER_NAME` used by this class 
like the other changed files.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28273#discussion_r2528462927

Reply via email to