On Wed, 10 Sep 2025 14:43:11 GMT, Ziad El Midaoui <zelmida...@openjdk.org> 
wrote:

> The test seems to be working in Linux currently, so I enabled it for Linux 
> systems.

tests/system/src/test/java/test/robot/javafx/scene/ContextMenuNPETest.java line 
48:

> 46: import org.junit.jupiter.api.Test;
> 47: import org.junit.jupiter.api.extension.ExtendWith;
> 48: import test.util.ScreenCaptureTestWatcher;

You can remove these now-unused imports

tests/system/src/test/java/test/robot/javafx/scene/ContextMenuNPETest.java line 
64:

> 62:  */
> 63: 
> 64: @ExtendWith(ScreenCaptureTestWatcher.class)

`ScreenCaptureTestWatcher` is only intended to be used with developer builds. 
We must not integrate a test into the repo with it enabled.

-------------

PR Review Comment: https://git.openjdk.org/jfx/pull/1893#discussion_r2337353860
PR Review Comment: https://git.openjdk.org/jfx/pull/1893#discussion_r2337076924

Reply via email to