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

>> 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.

Sounds good, thanks.

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

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

Reply via email to