RFR: 8319574: Exec/process tests should be marked as flagless

Jaikiran Pai jpai at openjdk.org
Thu Nov 9 06:45:56 UTC 2023


On Wed, 8 Nov 2023 21:04:49 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

> Tests that directly use ProcessBuilder to spawn processes and do not pass the standard test command line arguments should be marked as `vm.flagless` to indicate to the testing infrastructure that they do not accept them and should not be run with extra test command line options `test.vm.opts` and `test.java.opts`.
> 
> Add @requires vm.flagless to tests

The changes look OK to me. 

It wasn't clear to me what "flagless" meant. I checked the implementation of how this `@requires` is implemented https://github.com/openjdk/jdk/blob/master/test/jtreg-ext/requires/VMProps.java#L657. So essentially, the definition of "flagless" comes down to "was the JVM in which jtreg is running and about to launch a test, launched using certain set of JVM arguments? If it wasn't launched with those set of JVM arguments, then it is flagless." The set of such JVM arguments appears to be wide and may change over time.

-------------

Marked as reviewed by jpai (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/16572#pullrequestreview-1721810091


More information about the core-libs-dev mailing list