On Wed, 18 Sep 2024 15:31:40 GMT, Andy Goryachev <ango...@openjdk.org> wrote:

>> tests/system/src/test/java/test/com/sun/javafx/application/ConcurrentStartupTest.java
>>  line 46:
>> 
>>> 44:     @Test
>>> 45:     public void testStartupReturnBeforeRunnableComplete() {
>>> 46:         assertTimeout(Duration.ofMillis(15000), () -> {
>> 
>> Why not use an `@Timeout` annotation on the method? That would have been the 
>> natural change.
>
> my mistake.  switched to the Timeout annotation.

fixed

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1569#discussion_r1765652815

Reply via email to