On Mon, 2 Dec 2024 19:03:55 GMT, Harshitha Onkar <[email protected]> wrote:

>> The following two tests failed intermittently due to stability issues after 
>> JEP486 (Removal of Security Manager). Tests have been updated for stability. 
>> This test is applicable for linux platform only, hence `@requires (os.family 
>> == "linux")`  jtreg tag is added and the redundant OS check within the test 
>> is removed.
>> 
>> - java/awt/datatransfer/Independence/IndependenceAWTTest.java
>> - java/awt/datatransfer/Independence/IndependenceSwingTest.java
>> 
>> Multiple runs of the test on CI looks good and the tests pass.
>
> Harshitha Onkar has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   removed redundant null check

test/jdk/java/awt/datatransfer/Independence/IndependenceAWTTest.java line 76:

> 74: 
> 75:     private static void createAndShowUI() {
> 76:         frame = new Frame("IndependenceSwingTest");

Should that be ..AWTTest ?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22490#discussion_r1866721693

Reply via email to