RFR: 8294156: Allow PassFailJFrame.Builder to create test UI [v2]
Alexey Ivanov
aivanov at openjdk.org
Wed Sep 13 14:52:39 UTC 2023
On Wed, 13 Sep 2023 14:25:19 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> This enhances the `Builder` pattern added in [JDK-8294535](https://bugs.openjdk.org/browse/JDK-8294535) with a new method `testUI` which allows passing a lambda expression or a method reference to create *the test UI window*.
>>
>> The `PassFailJFrame` will automatically call the method on the EDT to create the UI, add it to the internal list of windows, install the window closing listener and finally position and show both the instructions and test UI.
>>
>> Alternatively, you can pass an already created window.
>>
>> The `main` method of a manual test could look as simple as a sequence of calls:
>>
>>
>> public static void main(String[] args) throws Exception {
>> PassFailJFrame.builder()
>> .instructions(INSTRUCTIONS)
>> .testUI(() -> createTestUI())
>> .build()
>> .awaitAndCheck();
>> }
>>
>> where `createTestUI` returns a test UI window.
>
> Alexey Ivanov has updated the pull request incrementally with one additional commit since the last revision:
>
> 8294156: Allow creating several test windows
>
> The windows can be positioned in advance, or
> a call back PositionWindows interface can be used
> to define their positions after the instruction UI
> frame is positioned on the screen.
I've implemented the feature for adding several test UI windows.
@azvegint Could you re-review please?
The test in #15666 needed a small update as the result.
The demos for multiple windows are available in #15721. I couldn't resist to play around with a couple of layouts.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/15665#issuecomment-1717786091
More information about the client-libs-dev
mailing list