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