On Thu, 24 Feb 2022 11:46:58 GMT, Manukumar V S <m...@openjdk.org> wrote:

> Create a regression test for 
> [JDK-4518432](https://bugs.openjdk.java.net/browse/JDK-4518432).
> 
> Issue: Copying from JTree and then changing the data in the component that 
> was copied from, can cause pastes to use 
>            the new edited data.
> 
> This test verifies that copying from JTree node and then changing the data in 
> the component that was copied from,
>  is not causing the pastes to use the new edited data instead of the original 
> copied data.
> 
> Testing:
> Java 1.4.0 -> Test Failed.
> $ ./j2sdk1.4.0/bin/java JTreeWin
> Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel
> Test Failed in javax.swing.plaf.metal.MetalLookAndFeel, Expected : 
> "colorsblue", but actual : "ctextolorsblue"
> Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel
> Test Failed in com.sun.java.swing.plaf.windows.WindowsLookAndFeel, Expected : 
> "colorsblue", but actual : "ctextolorsblue"
> 
> Java 1.4.1 -> Test Passed.
> $ ./j2sdk1.4.1/bin/java JTreeWin
> Testing L&F: javax.swing.plaf.metal.MetalLookAndFeel
> Test Passed in javax.swing.plaf.metal.MetalLookAndFeel
> Testing L&F: com.sun.java.swing.plaf.windows.WindowsLookAndFeel
> Test Passed in com.sun.java.swing.plaf.windows.WindowsLookAndFeel

Marked as reviewed by aivanov (Reviewer).

test/jdk/javax/swing/JTree/4518432/JTreeNodeCopyPasteTest.java line 100:

> 98:                     throw new RuntimeException("Test Failed, waited too 
> long, " +
> 99:                             "but the JTree can't gain focus for L&F: " + 
> laf);
> 100:                 }

The tree is the only component on the frame. By default, it will get focus. The 
call to `waitForIdle` guarantees all the events are processed. Even if tree 
doesn't get focus, the next thing you do is click a node in the tree, which 
brings the focus.

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

PR: https://git.openjdk.java.net/jdk/pull/7605

Reply via email to