RFR: 8158801: [TEST_BUG] Mixing tests fail because of focus workaround trick [v37]
Khalid Boulanouare
duke at openjdk.org
Thu Sep 4 15:49:47 UTC 2025
On Thu, 4 Sep 2025 14:12:50 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> Khalid Boulanouare has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Organizes imports
>
> test/jdk/java/awt/Mixing/AWT_Mixing/GlassPaneOverlappingTestBase.java line 28:
>
>> 26: import java.awt.event.MouseAdapter;
>> 27: import java.awt.event.MouseEvent;
>> 28: import java.lang.Override;
>
> All the classes from `java.lang.*` are imported implicitly.
>
> That is, remove `import java.lang.Override`.
lang package imports removed.
> test/jdk/java/awt/Mixing/AWT_Mixing/GlassPaneOverlappingTestBase.java line 100:
>
>> 98: protected boolean isMultiFramesTest(){
>> 99: return false;
>> 100: }
>
> There should be no blank line before the closing brace of a method or constructor. That is the snippet should like this:
>
>
> public GlassPaneOverlappingTestBase(boolean defaultClickValidation) {
> super(defaultClickValidation);
> }
>
> @Override
> protected boolean isMultiFramesTest() {
> return false;
> }
Done.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25971#discussion_r2322618892
PR Review Comment: https://git.openjdk.org/jdk/pull/25971#discussion_r2322627072
More information about the client-libs-dev
mailing list