RFR: 8158801: [TEST_BUG] Mixing tests fail because of focus workaround trick [v38]
Khalid Boulanouare
duke at openjdk.org
Mon Sep 15 08:30:08 UTC 2025
On Thu, 4 Sep 2025 18:26:59 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> Khalid Boulanouare has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Removes MULTI_FRAME_TEST instance variable and cleans up code
>
> test/jdk/java/awt/Mixing/AWT_Mixing/SimpleOverlappingTestBase.java line 73:
>
>> 71: return true;
>> 72: }
>> 73: public SimpleOverlappingTestBase() {
>
> Suggestion:
>
> }
>
> public SimpleOverlappingTestBase() {
>
>
> Ensure there's a blank line around method declarations. (For modified / added methods.)
Done.
> test/jdk/java/awt/Mixing/AWT_Mixing/SimpleOverlappingTestBase.java line 173:
>
>> 171: if (!await) {
>> 172: throw new RuntimeException("Ancestor frame didn't receive " +
>> 173: "focus");
>
> Suggestion:
>
> throw new RuntimeException("Ancestor frame didn't receive focus");
>
>
> I'm for breaking the rule of fitting into 80-column limit, wrapping this line doesn't improve readability.
Done.
> test/jdk/java/awt/Mixing/AWT_Mixing/SimpleOverlappingTestBase.java line 184:
>
>> 182: if (ancestor != null && isMultiFramesTest()) {
>> 183: ancestor.dispose();
>> 184: }
>
> These four lines are indented inconsistently, the indentation should be reduced by four spaces.
Done.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25971#discussion_r2348215286
PR Review Comment: https://git.openjdk.org/jdk/pull/25971#discussion_r2348215908
PR Review Comment: https://git.openjdk.org/jdk/pull/25971#discussion_r2348216383
More information about the client-libs-dev
mailing list