RFR: 8317116: Provide layouts for multiple test UI in PassFailJFrame [v2]
Alexey Ivanov
aivanov at openjdk.org
Thu Sep 26 13:57:09 UTC 2024
> Provides a set of default layouts for manual test using multiple test UI windows.
>
> The initial implementation in the demo in #15721 was too complicated. After looking at the simplified version used in #21180, I re-worked the layouts into separate methods.
>
> The layout methods are in `WindowLayouts` class next to `PassFailJFrame`. It is not necessary to mention it explicitly in the `@build` jtreg tag.
>
> The builder now provides new methods:
>
> - `positionTestUIRightRow` positions the test UI windows in a row to the right of the instruction frame. The top of the windows is aligned to that of the instruction frame.
> - `positionTestUIRightColumn` positions the test UI windows in a column to the right of the instruction frame. The top of the first window is aligned to that of the instruction frame.
> - `positionTestUIRightColumnCentered` positions the test UI windows in a column to the right of the instruction frame centering the stack of the windows.
> - `positionTestUIBottomRow` positions the test UI windows in a row to the bottom of the instruction frame. The left of the first window is aligned to that of the instruction frame.
> - `positionTestUIBottomRowCentered` positions the test UI windows in a row to the bottom of the instruction frame centering the row of the windows.
> - `positionTestUIBottomColumn` positions the test UI windows in a column to the bottom of the instruction frame. The left of the first window is aligned to that of the instruction frame.
>
> If required, the test developer can provide a custom implementation.
Alexey Ivanov has updated the pull request incrementally with one additional commit since the last revision:
Add missing parentheses for vertical positioning
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/21207/files
- new: https://git.openjdk.org/jdk/pull/21207/files/53d70888..10c9b563
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=21207&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=21207&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/21207.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/21207/head:pull/21207
PR: https://git.openjdk.org/jdk/pull/21207
More information about the client-libs-dev
mailing list