Re: RFR: 8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java [v3]

2025-05-01 Thread Rajan Halade
On Thu, 1 May 2025 18:08:40 GMT, Mikhail Yankelevich wrote: >> * Changed the test to use scratch directory >> * Cleaned up the imports > > Mikhail Yankelevich has updated the pull request incrementally with one > additional commit since the last revision: > > changed file format and throwing

Re: RFR: 8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java [v2]

2025-05-01 Thread Mikhail Yankelevich
On Thu, 1 May 2025 10:31:20 GMT, Mikhail Yankelevich wrote: >> * Changed the test to use scratch directory >> * Cleaned up the imports > > Mikhail Yankelevich has updated the pull request incrementally with one > additional commit since the last revision: > > changed file creation to use Uti

Re: RFR: 8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java [v2]

2025-05-01 Thread Mikhail Yankelevich
On Thu, 1 May 2025 15:34:11 GMT, Rajan Halade wrote: >> Mikhail Yankelevich has updated the pull request incrementally with one >> additional commit since the last revision: >> >> changed file creation to use Utils.createTempFile > > test/jdk/sun/security/pkcs12/P12SecretKey.java line 88: >

Re: RFR: 8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java [v3]

2025-05-01 Thread duke
On Thu, 1 May 2025 18:08:40 GMT, Mikhail Yankelevich wrote: >> * Changed the test to use scratch directory >> * Cleaned up the imports > > Mikhail Yankelevich has updated the pull request incrementally with one > additional commit since the last revision: > > changed file format and throwing

Re: RFR: 8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java [v3]

2025-05-01 Thread Mikhail Yankelevich
> * Changed the test to use scratch directory > * Cleaned up the imports Mikhail Yankelevich has updated the pull request incrementally with one additional commit since the last revision: changed file format and throwing an exception in case of the failure - Changes: - all: htt

Re: RFR: 8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java [v2]

2025-05-01 Thread Rajan Halade
On Thu, 1 May 2025 10:31:20 GMT, Mikhail Yankelevich wrote: >> * Changed the test to use scratch directory >> * Cleaned up the imports > > Mikhail Yankelevich has updated the pull request incrementally with one > additional commit since the last revision: > > changed file creation to use Uti

Re: RFR: 8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java [v2]

2025-05-01 Thread Rajan Halade
On Thu, 1 May 2025 15:25:46 GMT, Weijun Wang wrote: > LGTM. Or, you can just create files named `algName-keySize.ks`. There is no > need to be random here. +1. This would be a good enhancement to include. - PR Comment: https://git.openjdk.org/jdk/pull/24718#issuecomment-2845087042

Re: RFR: 8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java [v2]

2025-05-01 Thread Weijun Wang
On Thu, 1 May 2025 10:31:20 GMT, Mikhail Yankelevich wrote: >> * Changed the test to use scratch directory >> * Cleaned up the imports > > Mikhail Yankelevich has updated the pull request incrementally with one > additional commit since the last revision: > > changed file creation to use Uti

Re: RFR: 8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java [v2]

2025-05-01 Thread Mikhail Yankelevich
> * Changed the test to use scratch directory > * Cleaned up the imports Mikhail Yankelevich has updated the pull request incrementally with one additional commit since the last revision: changed file creation to use Utils.createTempFile - Changes: - all: https://git.openjdk.or

Re: RFR: 8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java [v2]

2025-05-01 Thread Mikhail Yankelevich
On Thu, 1 May 2025 05:00:13 GMT, Rajan Halade wrote: >> Mikhail Yankelevich has updated the pull request incrementally with one >> additional commit since the last revision: >> >> changed file creation to use Utils.createTempFile > > test/jdk/sun/security/pkcs12/P12SecretKey.java line 68: >

Re: RFR: 8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java

2025-04-30 Thread Rajan Halade
On Thu, 17 Apr 2025 09:51:40 GMT, Mikhail Yankelevich wrote: > * Changed the test to use scratch directory > * Cleaned up the imports test/jdk/sun/security/pkcs12/P12SecretKey.java line 68: > 66: > 67: // temporary files are created in scratch directory > 68: final File ksFile

RFR: 8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java

2025-04-17 Thread Mikhail Yankelevich
* Changed the test to use scratch directory * Cleaned up the imports - Commit messages: - JDK-8183348: Better cleanup for jdk/test/sun/security/pkcs12/P12SecretKey.java Changes: https://git.openjdk.org/jdk/pull/24718/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=24718&ra