Fixes #3709: Add GraalVM nativeTest test suite#3714
Fixes #3709: Add GraalVM nativeTest test suite#3714phipag wants to merge 10 commits intomockito:mainfrom
Conversation
|
This PR is ready for review now. I tested the GraalVM job in my fork: https://github.com/phipag/mockito/actions/runs/17128459949. It fails as expected and should succeed after merging #3710 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3714 +/- ##
=========================================
Coverage 86.43% 86.43%
Complexity 2969 2969
=========================================
Files 341 341
Lines 9011 9011
Branches 1110 1110
=========================================
Hits 7789 7789
Misses 941 941
Partials 281 281 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I see the JVM tests fail now with: I should fix that the tests still work when using JVM and not GraalVM in the environment. |
|
I added a potential fix in a347ff3. Let's see if this is enough to conditionally enable the tracing agent. |
|
The CI shows the expected result now: After merging and rebasing on #3710 we should see the graalvm workflow succeeding as well. |
|
This landed with #3710 |
Description
Adds a test suite using the subclass MockMaker running a simple Mockito test using GraalVM native image. Tested using
21.0.8-graal.Run:
After merging #3710 the
reflect-config.jsonshould include a line similar to this:Checklist
including project members to get a better picture of the change
commit is meaningful and help the people that will explore a change in 2 years
./gradlew spotlessApplyfor auto-formatting)Fixes #<issue number>in the description if relevantFixes #<issue number>if relevant