On Wed, 10 Sep 2025 15:37:32 GMT, Ziad El Midaoui <zelmida...@openjdk.org> wrote:
>> 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` Perhaps we should keep it for a while, to capture the screenshot if it fails (there is almost zero overhead in keeping it if the tests don't fail). We could create a followup ticket to remove it in the next test sprint. What do you think? ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1893#discussion_r2337933257
