On Thu, 15 Aug 2024 16:53:47 GMT, Michael Strauß <mstra...@openjdk.org> wrote:

>> Andy Goryachev has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   tests
>
> modules/javafx.controls/src/test/java/test/javafx/scene/control/SplitMenuButtonTest.java
>  line 34:
> 
>> 32: import org.junit.Test;
>> 33: 
>> 34: import static org.junit.Assert.*;
> 
> Since you're modifying this file significantly, you could also use the JUnit5 
> imports.

I think that would be out of scope for this PR.

We are considering updating tests to junit5 in bulk at some point, but I 
suspect it's a fairly low priority, unless we need to address something like 
https://bugs.openjdk.org/browse/JDK-8328629

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1535#discussion_r1718724352

Reply via email to