RFR: 8345335: Add excluded jdk_foreign tests to manual group [v3]
Jorn Vernee
jvernee at openjdk.org
Tue Dec 10 19:29:40 UTC 2024
On Tue, 10 Dec 2024 16:12:55 GMT, Ivan Šipka <isipka at openjdk.org> wrote:
>> @AlanBateman @mahendrachhipa @bwhuang-us @serhiysachkov @mcimadamore @JornVernee
>>
>> adding as manual tests
>
> Ivan Šipka has updated the pull request incrementally with one additional commit since the last revision:
>
> both tests declared automatic, labeled as jdk_foreign_stress and that group added to tier3
test/jdk/TEST.groups line 96:
> 94: :jdk_jpackage \
> 95: :jdk_since_checks \
> 96: :jdk_foreign_stress
Shouldn't it be in tier4?
test/jdk/TEST.groups line 385:
> 383: jdk_foreign = \
> 384: java/foreign \
> 385: jdk/internal/reflect/CallerSensitive/CheckCSMs.java
Please exclude the `jdk_foreign_stress` group from the regular `jdk_foreign` test group as well. We regularly run `jdk_foreign` as part of the normal dev cycle.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22580#discussion_r1878706279
PR Review Comment: https://git.openjdk.org/jdk/pull/22580#discussion_r1878706306
More information about the core-libs-dev
mailing list