On Tue, 23 Sep 2025 16:58:23 GMT, Josiah Noel <[email protected]> wrote:

>> test/jdk/com/sun/net/httpserver/InputRead100Test.java line 28:
>> 
>>> 26:  * @bug 8349670
>>> 27:  * @summary Test 1xx response handling
>>> 28:  * @run testng/othervm InputRead1xxTest
>> 
>> Did you forget to rename the test class and test file? The class is still 
>> called InputRead100Test.
>
> looks so

Rather than having two `@run` lines I'd suggest to split this into two full 
blown jtreg comments - using e.g. 

@test id=default

for the first, and

@test id=preferIPv6

for the second. That makes it easier to track down which configuration is 
causing problems when the test fails.

Also we tend to use Junit 5 (jupiter) in new tests in preference to TestNG when 
possible.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27069#discussion_r2372957273

Reply via email to