On Thu, 5 Oct 2023 20:57:07 GMT, Justin Lu <j...@openjdk.org> wrote:

> Please review this PR which refactors the ChoiceFormat tests to use JUnit.

test/jdk/java/text/Format/ChoiceFormat/Bug8001209.java line 78:

> 76:                 "Mutating array returned from getter changed internals of 
> ChoiceFormat");
> 77: 
> 78:         // Modify the array passed to setFormats() -> dayOfWeekNames

I think we should keep the `err` variable, and not throw an exception (and end 
the test) here. Otherwise, the test after this will not run.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16065#discussion_r1348978190

Reply via email to