On Wed, 29 Jul 2026 12:04:39 GMT, Coleen Phillimore <[email protected]> wrote:

> This fixes a longstanding problem with desiredAssertionStatus and adds a test.
> Tested with tier1.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

test/jdk/java/lang/Class/desiredAssertionStatus/Sanity.java line 30:

> 28:  * @run testng/othervm -esa -ea -Dsys.option=on  -Duser.option=on Sanity
> 29:  * @run testng/othervm -dsa -ea -Dsys.option=off -Duser.option=on Sanity
> 30:  * @run testng/othervm -esa -ea -Dsys.option=on  -Duser.option=on Sanity

This one is a dup of the 1st run directive. Maybe you want something like:
Suggestion:

 * @run testng/othervm -dsa -da -Dsys.option=off -Duser.option=off Sanity

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32086#discussion_r3675030728

Reply via email to