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