On Mon, 29 Jan 2024 09:33:35 GMT, Jose Pereda <jper...@openjdk.org> wrote:

>> Nir Lisker has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Update tests
>
> tests/system/src/test/java/test/com/sun/javafx/animation/SynchronisityTest.java
>  line 20:
> 
>> 18: 
>> 19: // Based on https://bugs.openjdk.org/browse/JDK-8159048
>> 20: public class SynchronisityTest extends Application {
> 
> shouldn't it be `SynchronicityTest`?

Yes, or maybe `SynchronizationTest`.

More importantly, the main test class should not extend `Application`. See 
[JDK-8234876](https://bugs.openjdk.org/browse/JDK-8234876).

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1352#discussion_r1469546192

Reply via email to