On Mon, 16 Jun 2025 21:32:16 GMT, Valerie Peng <valer...@openjdk.org> wrote:
>> src/java.base/share/classes/javax/crypto/Cipher.java line 348:
>>
>>> 346: transformation.substring(0, endIdx).trim());
>>> 347: if (algo.isEmpty()) {
>>> 348: throw new NoSuchAlgorithmException("Invalid
>>> transformation: " +
>>
>> I don't think this exception is tested. It could be by adding `test(" ",
>> provider);` in
>> [test/jdk/javax/crypto/Cipher/TestEmptyModePadding.java](https://github.com/openjdk/jdk/pull/25808/files#diff-5442ba4fbe85af64a99ca3c2d2b5795026be4c40e23af8337d52a64cefd6af35)
>> on line 68
>
> Sure, I can add it.
Thank you! This is covered now
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25808#discussion_r2151921175
