RFR: 8359687: Use PassFailJFrame for java/awt/print/Dialog/DialogType.java [v2]

Tejesh R tr at openjdk.org
Wed Jun 18 06:01:35 UTC 2025


On Tue, 17 Jun 2025 09:09:42 GMT, Srinivas Mandalika <smandalika at openjdk.org> wrote:

>> Use PassFailJFrame to streamline the test java/awt/print/Dialog/DialogType.java, as is for several of the manual tests.
>> 
>> Modified Manual Test Passed when run using jtreg.
>
> Srinivas Mandalika has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Added SkippedException when Printer is unavailable.

test/jdk/java/awt/print/Dialog/DialogType.java line 52:

> 50:         Two dialogs are shown in succession.
> 51:         The test passes as long as no exceptions are thrown, *AND*
> 52:         if running on Windows only, the first dialog is a native windows

Windows specific can be as note or even you can append it to instructions on Windows OS check condition.

test/jdk/java/awt/print/Dialog/DialogType.java line 62:

> 60:         }
> 61:         PassFailJFrame passFailJFrame = PassFailJFrame.builder()
> 62:             .title("DialogType Test Instructions")

Not required.

test/jdk/java/awt/print/Dialog/DialogType.java line 64:

> 62:             .title("DialogType Test Instructions")
> 63:             .instructions(INSTRUCTIONS)
> 64:             .rows((int)INSTRUCTIONS.lines().count() + 2)

Not required.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25842#discussion_r2153708020
PR Review Comment: https://git.openjdk.org/jdk/pull/25842#discussion_r2153703117
PR Review Comment: https://git.openjdk.org/jdk/pull/25842#discussion_r2153703273


More information about the client-libs-dev mailing list