[jdk11u-dev] Integrated: 8328957: Update PKCS11Test.java to not use hardcoded path

[Francisco Ferrari Bihurriet]

On Wed, 26 Feb 2025 21:07:06 GMT, Francisco Ferrari Bihurriet <fferrari at openjdk.org> wrote:

> Hi, I would like to propose an openjdk/jdk at 16576b87b7267aaa99c41f77993287e3479577aa backport.
> 
> I do this backport for parity with 11.0.26-oracle. The backport is clean.
> 
> Also, this allows running _SunPKCS11_ tests more easily, without the need to make symlink workarounds to locate the NSS library inside a `<base>/nss/lib` path.
> 
> ### Testing
> 
> * `jdk:tier1` (see [GitHub Actions run](https://github.com/franferrax/jdk11u-dev/actions/runs/13551011718))
>     * The failure in macos-x64 / test (hs/tier1 serviceability) / `serviceability/sa/ClhsdbFindPC.java#id1` is unrelated with PKCS11 test changes
> * `test/jdk/sun/security/pkcs11`
>     * No regressions found against `jdk11u-dev/master` (currently: 659a4669208645420e151e78ab5fd3ac3808b310)
>       ```
>       ==============================
>       Test summary
>       ==============================
>          TEST                                              TOTAL  PASS  FAIL ERROR   
>       >> jtreg:test/jdk/sun/security/pkcs11                   95    84    10     1 <<
>       ==============================
>       TEST FAILURE
>       ```
>     * <details>
>       <summary>jtreg <code>summary.txt</code> comparison</summary>
>       
>       ```diff
>       diff --git a/jdk11u-dev/pkcs11_regression/report/text/summary.txt b/jdk11u-dev-backport-8328957/pkcs11_regression/report/text/summary.txt
>       index ec3301f..552b9d0 100644
>       --- a/jdk11u-dev/pkcs11_regression/report/text/summary.txt
>       +++ b/jdk11u-dev-backport-8328957/pkcs11_regression/report/text/summary.txt
>       @@ -1,98 +1,98 @@
>        sun/security/pkcs11/Cipher/CancelMultipart.java              Passed. Execution successful
>        sun/security/pkcs11/Cipher/EncryptionPadding.java            Passed. Execution successful
>        sun/security/pkcs11/Cipher/JNICheck.java                     Passed. Execution successful
>        sun/security/pkcs11/Cipher/ReinitCipher.java                 Passed. Execution successful
>        sun/security/pkcs11/Cipher/Test4512704.java                  Passed. Execution successful
>        sun/security/pkcs11/Cipher/TestCICOWithGCM.java              Passed. Execution successful
>        sun/security/pkcs11/Cipher/TestCICOWithGCMAndAAD.java        Passed. Execution successful
>        sun/security/pkcs11/Cipher/TestChaChaPoly.java               Passed. Execution successful
>        sun/security/pkcs11/Cipher/TestChaChaPolyKAT.java            Passed. Execution successful
>        sun/security/pkcs11/Cipher/TestChaChaPolyNoReuse.java        Passed. Execution succe...

This pull request has now been integrated.

Changeset: ea3a0e45
Author:    Francisco Ferrari Bihurriet <fferrari at openjdk.org>
Committer: Severin Gehwolf <sgehwolf at openjdk.org>
URL:       https://git.openjdk.org/jdk11u-dev/commit/ea3a0e4504c9972d666648fd793f85ed4457056b
Stats:     37 lines in 1 file changed: 11 ins; 9 del; 17 mod

8328957: Update PKCS11Test.java to not use hardcoded path

Backport-of: 16576b87b7267aaa99c41f77993287e3479577aa

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

PR: https://git.openjdk.org/jdk11u-dev/pull/3004