On Sat, 26 Apr 2025 09:18:35 GMT, Kim Barrett <kbarr...@openjdk.org> wrote:

>> The newly-added `WhiteBox.waitForReferenceProcessing()` (see 
>> [8305186](https://bugs.openjdk.org/browse/JDK-8305186)) always fails with 
>> assertions enabled.
>> I've updated the assertion, and also added the test I used locally to test 
>> the new method (just not with assertions enabled, apparently.)
>
> test/hotspot/jtreg/vmTestbase/gc/gctests/ReferencesGC/WaitForRefProTest.java 
> line 1:
> 
>> 1: /*
> 
> Wrong place for this test. vmTestbase is old tests, converted from an old 
> testing infrastructure.
> See the readme here:  
> https://github.com/openjdk/jdk/tree/master/test/hotspot/jtreg/vmTestbase
> In particular, at the bottom of that page: "New tests must *not* be added 
> into this directory."
> 
> This new test belongs in `test/lib-test/jdk/test/whitebox/`.

Also, s/Pro/Proc/ in the name of the test.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24892#discussion_r2061240134

Reply via email to