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

[Sean Coffey]

On Tue, 24 Jun 2025 13:41:42 GMT, Mikhail Yankelevich <myankelevich at openjdk.org> wrote:

> > As the original contributor of this test case, it's nice to see the speed up via use of ExecutorService.
> > The upper case and lower case property tests are somewhat questionable IMO. One or two lines of test code would suffice but now we have a lot of noise distracting from the original intent of the test code. One could argue that every system property in the JDK could be subject to the same uppercase/lower case testing but such tests don't exist. A few lines of test code can easily verify behavior. Anything else enters the fuzzy logic testing approach. Hope it's not a pattern that new tests have to adopt.
> 
> Do you think that keeping it just mixed case would be sufficient? I believe this approach might be too much for other tests, it seems to me like a lot for this one as well

I'd welcome anything that helps reduce the extra code that's being added to test multiple case combinations for a system property. It strikes me as overkill.

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

PR Comment: https://git.openjdk.org/jdk/pull/25391#issuecomment-3000692391