On Tue, 8 Sep 2026 12:08:03 GMT, Lukasz Kostyra <[email protected]> wrote:
> When running tests in DebugNative I caught this testbug, `InitializeJavaFX` > tests used a 5-second startup timeout while in all other tests we assume a 15 > second timeout. > > This change fixes this discrepancy by using `test.util.Util.STARTUP_TIMEOUT` > constant instead of a magic number in `InitializeJavaFXLaunchBase`. This > lines up the startup timeout for these tests with all other tests we have. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). Simple change. I just scanned the project for `.await(` and there are some more that have a 'hardcoded' timeout. Maybe at one point, it makes sense that all use the same method/constant. ------------- Marked as reviewed by mhanl (Reviewer). PR Review: https://git.openjdk.org/jfx/pull/2300#pullrequestreview-5143342344
