RFR: 8341399: Add since checker tests to the langtools modules [v2]
Nizar Benalla
nbenalla at openjdk.org
Fri Nov 1 17:39:30 UTC 2024
On Fri, 1 Nov 2024 15:51:10 GMT, Nizar Benalla <nbenalla at 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
>
> Nizar Benalla has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
>
> - remove redundant line
> - Merge remote-tracking branch 'upstream/master' into since-checks-langtools
> - fix test summary, add jshell module
> - Add since checker tests to other modules
The tests are all meant to be under the `modules` directory, there was a problem with how subdirectories were shown in my IDE.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/21542#issuecomment-2452297552
More information about the core-libs-dev
mailing list