On Tue, 14 Sep 2021 00:10:26 GMT, lawrence.andrews <[email protected]> wrote:
>> Problem : >> 1) Test case was failing with following exception >> test result: Error. Parse Exception: Arguments to `manual' option not >> supported: yes >> 2) After removing =yes/no Parser exception is not seen but test UI will show >> and closes immediately where the user cannot see what is on the frame. >> >> Fix : >> 1) Add frame to show the test instructions. >> 2) Added timeout for the test case. >> 3) Show the test UI until either test case gets timeout or user press pass >> or fail button. >> >> @shurymury > > lawrence.andrews has updated the pull request incrementally with one > additional commit since the last revision: > > Fixed minor issue test/jdk/java/awt/Graphics/LCDTextAndGraphicsState.java line 126: > 124: instructionTextArea.setText(INSTRUCTIONS); > 125: instructionTextArea.setEditable(false); > 126: instructionTextArea.setBackground(Color.white); Is there a particular reason to set the background as white? It is better if the background and foreground colors are set by the L&F itself. I have fixed couple of bugs for GTKL&F, where the panel background color was set manually and then the L&F chosen color for foreground/Text was also same, so the text was not visible at all. ------------- PR: https://git.openjdk.java.net/jdk/pull/5503
