Skip to content

build(deps): bump org.junit.jupiter:junit-jupiter-api from 5.11.4 to 5.12.2 in /hiero-dependency-versions #1014

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Apr 14, 2025

Bumps org.junit.jupiter:junit-jupiter-api from 5.11.4 to 5.12.2.

Release notes

Sourced from org.junit.jupiter:junit-jupiter-api's releases.

JUnit 5.12.2 = Platform 1.12.2 + Jupiter 5.12.2 + Vintage 5.12.2

See Release Notes.

Full Changelog: junit-team/junit5@r5.12.1...r5.12.2

JUnit 5.12.1 = Platform 1.12.1 + Jupiter 5.12.1 + Vintage 5.12.1

See Release Notes.

Full Changelog: junit-team/junit5@r5.12.0...r5.12.1

JUnit 5.12.0 = Platform 1.12.0 + Jupiter 5.12.0 + Vintage 5.12.0

See Release Notes.

New Contributors

Full Changelog: junit-team/junit5@r5.11.4...r5.12.0

JUnit 5.12.0-RC2 = Platform 1.12.0-RC2 + Jupiter 5.12.0-RC2 + Vintage 5.12.0-RC2

See Release Notes.

Full Changelog: junit-team/junit5@r5.12.0-RC1...r5.12.0-RC2

JUnit 5.12.0-RC1 = Platform 1.12.0-RC1 + Jupiter 5.12.0-RC1 + Vintage 5.12.0-RC1

See Release Notes.

Full Changelog: junit-team/junit5@r5.12.0-M1...r5.12.0-RC1

JUnit 5.12.0-M1 = Platform 1.12.0-M1 + Jupiter 5.12.0-M1 + Vintage 5.12.0-M1

See Release Notes.

New Contributors

... (truncated)

Commits
  • 0a44659 Release 5.12.2
  • 4c7dfdc Finalize 5.12.2 release notes
  • 561613e Fix handling of CleanupMode.ON_SUCCESS
  • 19d07d2 Add 5.12.2 release notes from template
  • 803cbb6 Add build parameter for enabling dry-run mode for test execution
  • eb43e62 Back to snapshots for further development
  • ba9c9ae Release 5.12.1
  • e28ad4a Finalize 5.12.1 release notes
  • 1044e2c Move entry to 5.12.1 release notes
  • bea821d Fix Javadoc formatting
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [org.junit.jupiter:junit-jupiter-api](https://github.com/junit-team/junit5) from 5.11.4 to 5.12.2.
- [Release notes](https://github.com/junit-team/junit5/releases)
- [Commits](junit-team/junit5@r5.11.4...r5.12.2)

---
updated-dependencies:
- dependency-name: org.junit.jupiter:junit-jupiter-api
  dependency-version: 5.12.2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file java Pull requests that update Java code labels Apr 14, 2025
@dependabot dependabot bot requested review from a team as code owners April 14, 2025 09:29
@dependabot dependabot bot requested a review from AlfredoG87 April 14, 2025 09:29
AlfredoG87
AlfredoG87 previously approved these changes Apr 15, 2025
Copy link
Contributor

@AlfredoG87 AlfredoG87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AlfredoG87 AlfredoG87 added this to the 0.9.0 milestone Apr 15, 2025
@AlfredoG87 AlfredoG87 self-assigned this Apr 15, 2025
@AlfredoG87 AlfredoG87 requested a review from Copilot April 15, 2025 19:42
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • hiero-dependency-versions/build.gradle.kts: Language not supported

Signed-off-by: georgi-l95 <glazarov95@gmail.com>
@AlfredoG87
Copy link
Contributor

@jjohannes, we found this dependency discrepancy and also found a potential Fix to it (@georgi-l95 did), but we are not sure that the fix is a desired solution. wanted to ask your thoughts on this first, before merging and if you don't approve, what would be the recommended fix?

Thank you 🙏

@Nana-EC
Copy link
Contributor

Nana-EC commented Apr 16, 2025

Blocking until after refactor

Copy link
Contributor

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Blocking until after refactor merge

@AlfredoG87 AlfredoG87 modified the milestones: 0.9.0, 0.10.0 Apr 17, 2025
This reverts commit 9cb64af.

Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
Signed-off-by: Jendrik Johannes <jendrik.johannes@gmail.com>
@jjohannes
Copy link
Contributor

Your change fixes the problem indirectly, but we can do it simpler and more consistent with the notations we use.

The problem is not the versions we define. We already have a setup where all projects use versions consistently based on what is defined in hiero-dependency-versions/build.gradle.kts. All JUnit components we use align through the information there. You can see that, for example, in the build scans of the CI run:

What you see is that the versions are the same, BUT in the first run org.junit.platform:junit-platform-launcher is missing completely. The real problem is this existing configuration issue, which is unfortunately quite hidden at the moment:

The automatic loading of test framework implementation dependencies has been deprecated.	
  This is scheduled to be removed in Gradle 9.0.	
  Declare the desired test framework directly on the test suite or explicitly declare the test framework implementation dependencies on the test's runtime classpath.

Documentation

If the launcher is missing, Gradle takes one packaged into Gradle itself with whatever version Gradle has packaged (hidden to us). Which can cause issues as the one we are seeing. That's why this is deprecated and one should define the launcher dependency explicitly. Gradle does this for you in the "standard" test setup through the "newer" test suites API. That's what is used when you run the test unit tests in other modules. And therefore we do not have the problem there.

Here however, you have the custom setup where the test code is in main and you manually register the runSuites tasks. This is okay, but we then we also need to manually add the launcher dependency at test runtime (see my commit).

@AlfredoG87
Copy link
Contributor

@jjohannes, Thank you for the very detailed and referenced explanation, read a ton of stuff today! 💯

@Nana-EC
Copy link
Contributor

Nana-EC commented Apr 17, 2025

@dependabot rebase

Copy link
Contributor Author

dependabot bot commented on behalf of github Apr 17, 2025

Looks like this PR has been edited by someone other than Dependabot. That means Dependabot can't rebase it - sorry!

If you're happy for Dependabot to recreate it from scratch, overwriting any edits, you can request @dependabot recreate.

Copy link
Contributor

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unblocking based on updates post refactor

@Nana-EC Nana-EC merged commit 3dd4d53 into main Apr 17, 2025
13 checks passed
@Nana-EC Nana-EC deleted the dependabot/gradle/hiero-dependency-versions/org.junit.jupiter-junit-jupiter-api-5.12.2 branch April 17, 2025 19:52
Copy link

codecov bot commented Apr 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

@@             Coverage Diff              @@
##               main    #1014      +/-   ##
============================================
+ Coverage     88.33%   88.45%   +0.12%     
- Complexity      736      741       +5     
============================================
  Files           134      135       +1     
  Lines          3154     3171      +17     
  Branches        237      241       +4     
============================================
+ Hits           2786     2805      +19     
+ Misses          298      297       -1     
+ Partials         70       69       -1     

see 4 files with indirect coverage changes

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

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
dependencies Pull requests that update a dependency file java Pull requests that update Java code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants