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

Koushik Muthukrishnan Thirupattur duke at openjdk.org
Sun Jun 22 17:23:32 UTC 2025


On Mon, 16 Jun 2025 10:38:08 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 with a new target base due to a merge or a rebase. The pull request now contains two commits:
> 
>  - Merge branch 'master' into JDK-8357470
>  - JDK-8357470: src/java.base/share/classes/sun/security/util/Debug.java implement the test for args.toLowerCase
>    
>    * added an automated mixed case option
>    * using multithreading now
>    * added logs for simpler debug
>    * added missing -Djava.security.auth.debug coverage

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

> 150:      * @return resulting string
> 151:      */
> 152:     private String makeFirstAndLastLetterUppercase(final String paramString) {

If the goal is to test case-insensitive parsing comprehensively, this method should be expanded to test various case combinations (e.g., all-uppercase, alternating case, etc). Otherwise, document this limitation.

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

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


More information about the security-dev mailing list