On Wed, 16 Nov 2022 17:49:22 GMT, Andy Goryachev <ango...@openjdk.org> wrote:

>> tests/system/src/test/java/test/util/Util.java line 317:
>> 
>>> 315:      * @param args - command line arguments
>>> 316:      */
>>> 317:     public static <T extends Application> void launch (
>> 
>> This looks like a useful utility. I think it would be helpful to either 
>> remove the timeout parameter entirely, or else have a variant of this that 
>> doesn't take the timeout value, and use a default timeout value. Most tests 
>> use 10 or 15 seconds so standardizing on 15 seems reasonable.
>> 
>> Also, many of the tests use `Platform.startup` since they don't need to 
>> launch an application. Have you looked at providing a similar utility method 
>> for those cases?
>
> 1. added a method with 15 second default.  There are only 4 remaining that 
> use custom timeout, the first two use 5, the other two 50:
> test.javafx.embed.swing.JFXPanelTest.doSetupOnce()
> test.javafx.embed.swing.SwingFXUtilsTest.doSetupOnce()
> test.javafx.embed.swing.SwingNodeMemoryLeakTest.setupOnce()
> test.javafx.embed.swing.SwingNodeScaleTest.setupOnce()

2. will add Util.startup(CountDownLatch, Runnable) for some of the tests

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

PR: https://git.openjdk.org/jfx/pull/950

Reply via email to