On Wed, 16 Oct 2024 16:23:41 GMT, Nizar Benalla <nbena...@openjdk.org> wrote:
> Can I get a review for this patch that adds `@since` checker tests to the > following modules: java.compiler, jdk.compiler, jdk.javadoc and jdk.jdeps. > The initial test for `java.base` has been integrated in > [JDK-8331051](https://bugs.openjdk.org/browse/JDK-8331051). > > The jtreg comments are almost the same as the ones for the `java.base` test, > only the Bug ID and the module name being passed to the test are different. > > I've made a small change to `test/jdk/TEST.groups` to include the new tests. > > Here are the emails > [[1](https://mail.openjdk.org/pipermail/jdk-dev/2024-June/009160.html)] > [[2](https://mail.openjdk.org/pipermail/jdk-dev/2024-October/009474.html)] in > `jdk-dev` describing how the tests work and how to run them. > > TIA test/jdk/tools/sincechecker/jdk_compiler/CheckSince_jdkCompiler.java line 27: > 25: * @test > 26: * @bug 8341399 > 27: * @summary Test for `@since` for java.base module You forgot to change the summaries but remembered to change the `@run` directives. Curious! ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21542#discussion_r1825957173