RFR: 8374434: Several JShell tests report JUnit discovery warnings
Jan Lahoda
jlahoda at openjdk.org
Mon Jan 5 12:07:48 UTC 2026
JShell tests where converted to use JUnit instead of TestNG by this commit:
https://github.com/openjdk/jdk/commit/b0ca9bf61e0390a3b022a0915eacabb0cfd92e93
As part of the conversion, the `@Test` annotation had to be copied from classes to test methods, as JUnit requires every test method to be annotated. Sadly the tool doing the conversion moved the annotation to some non-test methods as well. This PR fixes that. The tool has been fixed as well.
-------------
Commit messages:
- 8374434: Several JShell tests report JUnit discovery warnings
Changes: https://git.openjdk.org/jdk/pull/29034/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=29034&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8374434
Stats: 4 lines in 3 files changed: 0 ins; 4 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/29034.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/29034/head:pull/29034
PR: https://git.openjdk.org/jdk/pull/29034
More information about the compiler-dev
mailing list