RFR: 8357470: src/java.base/share/classes/sun/security/util/Debug.java implement the test for args.toLowerCase [v4]

Sean Coffey coffeys at openjdk.org
Fri Aug 1 14:22:59 UTC 2025


On Wed, 25 Jun 2025 14:01:15 GMT, Mikhail Yankelevich <myankelevich at openjdk.org> wrote:

>> * added an automated mixed case option
>> * using multithreading now
>> * added logs for simpler debug
>> * added missing -Djava.security.auth.debug coverage
>
> Mikhail Yankelevich has updated the pull request incrementally with one additional commit since the last revision:
> 
>   reverting to have only the mixed case to keep the test more consise and still cover toLowerCase

test/jdk/sun/security/util/Debug/DebugOptions.java line 114:

> 112:     }
> 113: 
> 114:     /**

approach looks fine - this comment could be removed or corrected

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25391#discussion_r2248114669


More information about the security-dev mailing list