On Tue, 26 Mar 2024 20:57:52 GMT, Alisen Chung <ach...@openjdk.org> wrote:
>> Removing applet usage from manual JColorChooser tests > > Alisen Chung has updated the pull request incrementally with one additional > commit since the last revision: > > instruction string change are you going to reduce the timeout as Alexander suggested ? test/jdk/javax/swing/JColorChooser/Test6977726.java line 42: > 40: String instructions = """ > 41: Check that there is a panel with \"Text Preview Panel\" > text > 42: and with title \"Preview\" in the JColorChooser."""; (1) You have a whitespace error that needs fixing. (2) It is probably worth adding a sentence "Test passes if the panel is as described, test fails otherwise". ------------- PR Review: https://git.openjdk.org/jdk/pull/18369#pullrequestreview-1961753498 PR Review Comment: https://git.openjdk.org/jdk/pull/18369#discussion_r1540122335
