RFR: 8315834: Open source several Swing JSpinner related tests

Prasanta Sadhukhan psadhukhan at openjdk.org
Wed Sep 13 06:38:39 UTC 2023


On Tue, 12 Sep 2023 05:46:23 GMT, Tejesh R <tr at openjdk.org> wrote:

> Open source these Swing JSpinner related tests:
> 
> javax/swing/JSpinner/4522737/bug4522737.java
> javax/swing/JSpinner/4656590/bug4656590.java
> javax/swing/JSpinner/4680204/bug4680204.java
> javax/swing/JSpinner/4862257/bug4862257.java
> javax/swing/JSpinner/5104421/bug5104421.java

Marked as reviewed by psadhukhan (Reviewer).

test/jdk/javax/swing/JSpinner/bug4522737.java line 51:

> 49:                 objectOutputStream = new ObjectOutputStream(byteArrayOutputStream);
> 50:                 objectOutputStream.writeObject(originalComponent);
> 51:             } catch (Throwable e) {

You can remove this try-catch block and let original exception propagate

test/jdk/javax/swing/JSpinner/bug4680204.java line 65:

> 63:                 frame.getContentPane().add(sp2);
> 64:                 sp2.setModel(new SpinnerNumberModel(1, 1, 100, 1));
> 65:                 frame.pack();

Probably setLocationRelativeTo call would be good

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

PR Review: https://git.openjdk.org/jdk/pull/15678#pullrequestreview-1623787651
PR Review Comment: https://git.openjdk.org/jdk/pull/15678#discussion_r1324033900
PR Review Comment: https://git.openjdk.org/jdk/pull/15678#discussion_r1324036078


More information about the client-libs-dev mailing list