<i18n dev> RFR: 8319678: Several tests from corelibs areas ignore VM flags [v3]

Jaikiran Pai jpai at openjdk.org
Thu Apr 11 09:28:43 UTC 2024


On Thu, 11 Apr 2024 09:07:10 GMT, Mahendra Chhipa <mchhipa at openjdk.org> wrote:

>> Updated following tests to use ProcessTools methods:
>>  test/jdk/java/lang/Thread/UncaughtExceptionsTest.java
>>  test/jdk/java/lang/annotation/LoaderLeakTest.java
>>  test/jdk/java/rmi/reliability/benchmark/bench/rmi/Main.java
>>  test/jdk/java/time/nontestng/java/time/chrono/HijrahConfigTest.java
>>  test/jdk/javax/naming/spi/providers/InitialContextTest.java
>
> Mahendra Chhipa has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Implemented the review comments.

test/jdk/javax/naming/spi/providers/InitialContextTest.java line 301:

> 299:                 .map(Path::toString)
> 300:                 .collect(Collectors.joining(File.pathSeparator));
> 301: //        System.setProperty("test.noclasspath", "true");

This newly introduced commented code can be removed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18602#discussion_r1560711526


More information about the i18n-dev mailing list