On Wed, 18 Feb 2026 18:43:06 GMT, Alan Bateman <[email protected]> wrote:

>> Kieran Farrell has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   newline
>
> test/hotspot/jtreg/serviceability/dcmd/vm/SecurityPropertiesTest.java line 40:
> 
>> 38:  *          java.management
>> 39:  *          jdk.internal.jvmstat/sun.jvmstat.monitor
>> 40:  * @run testng SecurityPropertiesTest
> 
> Probably best to have this test start out as a JUnit rather than TestNG tests 
> as there is effort to try to have new tests not use TestNG.

The existing dcmd tests are testng so understandable to duplicate that.

@AlanBateman  Alan added some JUnit tests in this area for the virtual thread 
tests, but is that policy/advice/preference...?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29124#discussion_r2826896318

Reply via email to