On Mon, 7 Feb 2022 18:29:47 GMT, lawrence.andrews <[email protected]> wrote:
>> 1) Removed =yesno that was causing the test to fail with following exception
>> test result: Error. Parse Exception: Arguments to `manual' option not
>> supported: yesno
>> After removing =yesno, test was just passing without user interaction so
>> fixed the following
>>
>> Add the following so that user can user interact with the test
>> a) Added pass/fail button and instruction for the user to know what he/she
>> is going to test and what is expected.
>> b) Added Dialog with textarea to enter the reason for the testcase failure.
>> This will help the user to understanding the reason why the test was failed
>> while analyzing the results.
>>
>> @shurymury
>
> lawrence.andrews has updated the pull request incrementally with one
> additional commit since the last revision:
>
> Dispose the frame via EDT
test/jdk/java/awt/Graphics/TextAAHintsTest.java line 228:
> 226: mainThread = Thread.currentThread();
> 227: try {
> 228: mainThread.sleep(testTimeOut);
Suggestion:
Thread.sleep(testTimeOut);
`sleep` is a static method, it's better to call it via the class name rather
than an instance variable. This will resolve the IDE warning.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7275
