RFR: 8328005: Convert java/awt/im/JTextFieldTest.java applet test to main [v2]

Phil Race prr at openjdk.org
Wed Mar 13 18:18:14 UTC 2024


On Wed, 13 Mar 2024 00:27:32 GMT, Damon Nguyen <dnguyen at openjdk.org> wrote:

>> Convert java/awt/im/JTextFieldTest.java applet test to main using PassFailJFrame.
>
> Damon Nguyen has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix suggested redudant changes

test/jdk/java/awt/im/JTextFieldTest.java line 47:

> 45:             """
> 46:              Please run this test case under Solaris cjk locale with inputmethod
> 47:              support, if you could input Chinese/Japanese/Korean in the swing

The scenario here should not be specific to Solaris.
I think you should rephrase the first sentence to
"Please run this test in a CJK (Chinese/Japanese/Korean) locale with input method support".

I some times wonder how well these tests that say "please change your environment" actually get properly tested.
It is perhaps possible to do change desktop locale support on the fly but by the time
you read the instructions the test is already running.
That introduces another wrinkle. Maybe you need to restart at least this test.
And I am not 100% sure that jtreg passes through locale vars. I hope it does.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/18247#discussion_r1523718224


More information about the client-libs-dev mailing list