Skip to content

Commit

Permalink
8280948: Write a regression test for JDK-4659800
Browse files Browse the repository at this point in the history
Backport-of: 66b2c3b66e253ac3d8718c0c6d7c7551dbe04001
  • Loading branch information
Victor Rudometov authored and mrserb committed Oct 29, 2022
1 parent aa98d09 commit 6a45f75
Showing 1 changed file with 124 additions and 0 deletions.
124 changes: 124 additions & 0 deletions test/jdk/javax/swing/JButton/4659800/EnterKeyActivatesButton.java
@@ -0,0 +1,124 @@
/*
* Copyright (c) 2022, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
* under the terms of the GNU General Public License version 2 only, as
* published by the Free Software Foundation.
*
* This code is distributed in the hope that it will be useful, but WITHOUT
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
* version 2 for more details (a copy is included in the LICENSE file that
* accompanied this code).
*
* You should have received a copy of the GNU General Public License version
* 2 along with this work; if not, write to the Free Software Foundation,
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
*
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
* or visit www.oracle.com if you need additional information or have any
* questions.
*/

import java.awt.event.KeyEvent;
import java.awt.Robot;
import java.util.Arrays;
import java.util.List;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;
import javax.swing.UIManager;
import javax.swing.UnsupportedLookAndFeelException;

import static java.util.stream.Collectors.toList;

/*
* @test
* @key headful
* @requires (os.family == "windows")
* @bug 4659800
* @summary Check whether pressing <Enter> key generates
* ActionEvent on focused Button or not. This is applicable only for
* WindowsLookAndFeel and WindowsClassicLookAndFeel.
* @run main EnterKeyActivatesButton
*/
public class EnterKeyActivatesButton {
private static volatile boolean buttonPressed;
private static JFrame frame;

public static void main(String[] s) throws Exception {
runTest();
}

private static void setLookAndFeel(String lafName) {
try {
UIManager.setLookAndFeel(lafName);
} catch (UnsupportedLookAndFeelException ignored) {
System.out.println("Ignoring Unsupported L&F: " + lafName);
} catch (ClassNotFoundException | InstantiationException
| IllegalAccessException e) {
throw new RuntimeException(e);
}
}

private static void disposeFrame() {
if (frame != null) {
frame.dispose();
frame = null;
}
}

private static void createUI() {
frame = new JFrame();
JPanel panel = new JPanel();
JButton focusedButton = new JButton("Button1");
focusedButton.addActionListener(e -> buttonPressed = true);
panel.add(focusedButton);
panel.add(new JButton("Button2"));
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.add(panel);
frame.pack();
frame.setLocationRelativeTo(null);
frame.setVisible(true);
focusedButton.requestFocusInWindow();
}

public static void runTest() throws Exception {
Robot robot = new Robot();
robot.setAutoDelay(100);
// Consider only Windows and Windows Classic LnFs.
List<String> winlafs = Arrays.stream(UIManager.getInstalledLookAndFeels())
.filter(laf -> laf.getName().startsWith("Windows"))
.map(laf -> laf.getClassName())
.collect(toList());

for (String laf : winlafs) {
try {
buttonPressed = false;
System.out.println("Testing L&F: " + laf);
SwingUtilities.invokeAndWait(() -> {
setLookAndFeel(laf);
createUI();
});

robot.waitForIdle();
robot.keyPress(KeyEvent.VK_ENTER);
robot.keyRelease(KeyEvent.VK_ENTER);
robot.waitForIdle();

if (buttonPressed) {
System.out.println("Test Passed for L&F: " + laf);
} else {
throw new RuntimeException("Test Failed, button not pressed for L&F: " + laf);
}

} finally {
SwingUtilities.invokeAndWait(EnterKeyActivatesButton::disposeFrame);
}
}

}
}

1 comment on commit 6a45f75

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.