On Wed, 10 Sep 2025 16:58:14 GMT, Kevin Rushforth <k...@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

Done

> 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.

I updated it and removed the `ScreenCaptureTestWatcher`

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1893#discussion_r2337907764
PR Review Comment: https://git.openjdk.org/jfx/pull/1893#discussion_r2337170383

Reply via email to