Integrated: 8323717: Introduce test keyword for tests that need external dependencies
Aleksey Shipilev
shade at openjdk.org
Thu Jan 25 18:05:40 UTC 2024
On Mon, 15 Jan 2024 10:48:23 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> Some jtreg tests require resolvable external dependencies. This resolution is delegated to JIB, which is not used in vanilla OpenJDK testing. It would be convenient to add a keyword that marks tests that require these external dependencies, so that we could exclude those tests from runs. This would allow us to: a) run all tests in hotspot:tier4, which now excludes `applications/` specifically; b) make all tests runs (#17422) cleaner on many environments.
>
> I provisionally call this flag `external-dep`, but I am open for other suggestions.
>
> Note that some tests that pull `@Artifact`-s provide special paths that do limited testing anyway. However, there are tests which cannot run without external dependencies at all. These include at least `applications/jcstress` and `applications/scimark` tests.
>
> Ironically, I cannot run the jcstress test generator because the dependencies are lacking here. I regenerated those test using a self-built jcstress 0.16 bundle.
>
> Additional testing:
> - [x] `make test TEST=applications/` fails
> - [x] `JTREG_KEYWORDS=!external-dep make test TEST=applications/` passes, skipping most of the tests
This pull request has now been integrated.
Changeset: 12b89cd2
Author: Aleksey Shipilev <shade at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/12b89cd2eeb5c2c43a2ce425c96fc4f718e30514
Stats: 62 lines in 32 files changed: 32 ins; 0 del; 30 mod
8323717: Introduce test keyword for tests that need external dependencies
Reviewed-by: dholmes, lmesnik
-------------
PR: https://git.openjdk.org/jdk/pull/17421
More information about the core-libs-dev
mailing list