On Mon, 28 Mar 2022 17:13:44 GMT, Lance Andersen <[email protected]> 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
