On Sat, 25 Sep 2021 13:13:03 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

> The gradle verification task failed (see the test results from GitHub 
> Actiona). It looks like it still needs the hamcrest jars. Go ahead and add 
> them back in (and remove the exclusion for them).

Yeah, I noticed, I couldn't get rid of them.  Apparently JUnit 4 itself is 
depending on them now (starting from 4.10 orso) and the Vintage engine requires 
JUnit 4.12+, so it is actually overriding JUnit 4.8.2 to 4.13.2 and pulls in 
the Hamcrest jars again.

I've tried forcing JUnit to 4.8.2 and even though that works, it leads to a 
spam stacktrace in the logs for each test executed because it can't find 
something via reflection.  I then tried to hide this log message, but have been 
unable to find any way of getting that message blocked by configuring 
java.util.logging that works in combination with Gradle and its daemon.

So... I've sort of think this is not a fight we can win easily and have 
reincluded the Hamcrest jars.  I also removed the JUnit 4 dependency as the 
Vintage engine will pull a newer one in any way.

I think I should upgrade these lines as well: `testImplementation group: 
"junit", name: "junit", version: "4.8.2"` in each of the modules to make it 
clear 4.13.2 is used and not 4.8.2.

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

PR: https://git.openjdk.java.net/jfx/pull/633

Reply via email to