Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fixes sbt#121: Suite task naming no longer crashes; or mangles vintage test names #122

Merged
merged 3 commits into from
Nov 2, 2024

Conversation

dji
Copy link
Contributor

@dji dji commented Oct 25, 2024

Fixes (and tests!) the two related issues in #121:

  • In some situations @Suites were used, a RuntimeException would be thrown when trying to report the test status: Test class path.to.my.TestClass is not enclosed by path.to.my.SuiteClass
  • Junit 4 tests run as part of a suite had their names mangled

… Vintage tests in Suites have their names properly formatted.
@@ -13,6 +13,7 @@ object Dependencies {
val testInterface = "org.scala-sbt" % "test-interface" % testInterfaceVer
val junitJupiterParams = "org.junit.jupiter" % "junit-jupiter-params" % junitJupiterVer
val junitVintageEngine = "org.junit.vintage" % "junit-vintage-engine" % junitVintageVer
val junitPlatformSuite = "org.junit.platform" % "junit-platform-suite" % junitPlatformVer
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Alternatively, only junit-platform-suite-api and junit-platform-suite-engine are actually needed, if you prefer more specific dependencies

Copy link
Member

@eed3si9n eed3si9n left a comment

Choose a reason for hiding this comment

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

Thanks!

@dji
Copy link
Contributor Author

dji commented Nov 1, 2024

Fixed the formatting issue (I think!) and merged in the main branch - what are the next steps to get this merged in?

@eed3si9n
Copy link
Member

eed3si9n commented Nov 1, 2024

Rerunning CI now.

@eed3si9n eed3si9n merged commit 359679d into sbt:main Nov 2, 2024
5 checks passed
@dji dji deleted the suite-naming-fixes branch November 4, 2024 23:21
@dji
Copy link
Contributor Author

dji commented Nov 4, 2024

And you did a release too! :) Thanks.

dongjoon-hyun pushed a commit to apache/spark that referenced this pull request Nov 14, 2024
…o 5.11.3

### What changes were proposed in this pull request?
This pr aims to upgrade `jupiter-interface` from 0.13.0 to 0.13.1 and Junit5 to the latest version(Platform 1.11.3 + Jupiter 5.11.3).

### Why are the changes needed?
The new version of `jupiter-interface` brings two fixes:

- sbt/sbt-jupiter-interface#122
- sbt/sbt-jupiter-interface#116

Simultaneously upgraded Junit dependencies to Platform 1.11.3 + Jupiter 5.11.3:

- sbt/sbt-jupiter-interface#119

The full release notes of `jupiter-interface` as follows:

- https://github.com/sbt/sbt-jupiter-interface/releases/tag/v0.13.1

and the full release notes between Junit 5.11.0 to 5.11.3 as follows:

- https://junit.org/junit5/docs/5.11.3/release-notes/#release-notes-5.11.3
- https://junit.org/junit5/docs/5.11.3/release-notes/#release-notes-5.11.2
- https://junit.org/junit5/docs/5.11.3/release-notes/#release-notes-5.11.1

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Pass GitHub Actions

### Was this patch authored or co-authored using generative AI tooling?
No

Closes #48834 from LuciferYang/junit5-5.11.3.

Authored-by: yangjie01 <yangjie01@baidu.com>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
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.

2 participants