On Mon, 28 Mar 2022 17:13:44 GMT, Lance Andersen <lan...@openjdk.org> wrote:
>> Naoto Sato has updated the pull request incrementally with one additional >> commit since the last revision: >> >> New unit test. IllegalArgumentException. > > test/jdk/java/util/Locale/TestOf.java line 79: > >> 77: @Test (expectedExceptions = IllegalArgumentException.class) >> 78: public void test_IAE() { >> 79: Locale.of("en", "", "", "", ""); > > I would consider using `assertThrows(IllegalArgumentException.class, () -> > Locale.of("en", "", "", "", "")); ` instead of the expectedExceptions > annotation element as it is the preferred way forward Thanks. Modified as suggested. ------------- PR: https://git.openjdk.java.net/jdk/pull/7947