Skip to content

Fixes #3179 : Add module for Java 21 tests.#3180

Merged
TimvdLippe merged 1 commit intomockito:mainfrom
LeMikaelF:add-java21-module
Nov 27, 2023
Merged

Fixes #3179 : Add module for Java 21 tests.#3180
TimvdLippe merged 1 commit intomockito:mainfrom
LeMikaelF:add-java21-module

Conversation

@LeMikaelF
Copy link
Copy Markdown
Contributor

This PR adds a Java 21 module to be used for testing.

The current version of Gradle (8.4) doesn't yet support running on Java 21, but it does support compiling and testing using Java 21 through the use of toolchains (source). 8.5 will support running Java 21 (source), so the toolchain block can be removed then.

The test source directory is currently empty, apart from a .gitkeep file, but as soon as this PR is merged, it can be used for the tests from #3173 and #3167.

There are no tests yet, but I did test the module by adding my test from #3173 (see top of that PR's description). The tests are ignored when running using Java 11, and correctly fail/pass when using Java 21.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 27, 2023

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.50%. Comparing base (f522f49) to head (44e2040).
⚠️ Report is 309 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #3180   +/-   ##
=========================================
  Coverage     85.50%   85.50%           
  Complexity     2914     2914           
=========================================
  Files           334      334           
  Lines          8864     8864           
  Branches       1097     1097           
=========================================
  Hits           7579     7579           
  Misses          995      995           
  Partials        290      290           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@LeMikaelF
Copy link
Copy Markdown
Contributor Author

Looks like the CI is flaky...

@TimvdLippe TimvdLippe merged commit a0cc22d into mockito:main Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants