RFR: 8338016: SplitMenuButton constructors should match MenuButton [v2]

Michael Strauß mstrauss at openjdk.org
Thu Aug 15 17:43:56 UTC 2024


On Thu, 15 Aug 2024 17:07:40 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> 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

Well, you _do_ modify this file significantly, so it's not entirely out of scope. I've been converting files over to JUnit5 in the past as part of significant modifications, I think it's a valid path to get us to eventually move completely to JUnit5 (because as you say, bulk conversion isn't probably happening any time soon).

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

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


More information about the openjfx-dev mailing list