RFR: 8340193: Open source several AWT Dialog tests - Batch 1
Prasanta Sadhukhan
psadhukhan at openjdk.org
Fri Sep 27 07:57:36 UTC 2024
On Wed, 25 Sep 2024 07:20:21 GMT, Tejesh R <tr at openjdk.org> wrote:
> Open source these AWT Mouse tests:
>
> java/awt/Dialog/FileDialogIconTest/FileDialogIconTest.java
> java/awt/Dialog/FileDialogWrongNameCrash.java
> java/awt/Dialog/DialogResizeTest.java
> java/awt/Dialog/GetLocationTest_1.java
> java/awt/Dialog/DialogIconTest/DialogIconTest.java
test/jdk/java/awt/Dialog/DialogResizeTest.java line 133:
> 131: public void componentHidden(ComponentEvent e) {
> 132: System.out.println("componentHidden event occurred.");
> 133: }
I guess PassFailJFrame.log would be of more use than System println, if at all these logs are of any help..
test/jdk/java/awt/Dialog/FileDialogWrongNameCrash.java line 58:
> 56:
> 57: private static Frame initialize() {
> 58: Frame frame = new Frame("Frame");
give more meaningful name
test/jdk/java/awt/Dialog/GetLocationTest_1.java line 50:
> 48: up with the top left by the cursor
> 49: 2. If you see this correct behavior press PASS. If you see that
> 50: the yellow window is offset by an inset (5, 24) press FAIL
how will tester knows that the inset is of this dimension, it might vary from platform to platform and also vary with different scaling? maybe just delete this hardcoded dimension?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21174#discussion_r1778177853
PR Review Comment: https://git.openjdk.org/jdk/pull/21174#discussion_r1778188170
PR Review Comment: https://git.openjdk.org/jdk/pull/21174#discussion_r1778189364
More information about the client-libs-dev
mailing list