[BUG] 8372325: Refactor tests in jdk/java/net/httpclient to use ${test.main.class}

Overview#

In the numerous tests under test/jdk/java/net/httpclient, the test class names were directly written in JTReg's (@run tag, a regression testing framework unique to OpenJDK). For example:

@run junit/othervm AltServiceUsageTest

However, having such self-referential test class names within the code can lead to some tricky issues. Since tests are often copied and pasted, forgetting to update them can result in unintended tests being executed.

Starting with JTReg 7.5.2, a placeholder ${test.main.class} was introduced¹. This allows for meta-specification of classes annotated with JTReg annotations; if @run ${test.main.class} is present on the Main class, it effectively becomes @run Main.

In this update, we replaced explicit class names in @run tags with ${test.main.class}.

@run junit/othervm ${test.main.class}

The changes affected 313 files and were primarily mechanical replacements².

Discovered on JBS 2026/03/18#

This issue was discovered via JBS, where I confirmed that many test files under java/net/httpclient directly wrote the test class names in @run tags. These forms included main, junit, othervm, and timeout.

In the PR, we decided to replace only the class names appearing as execution targets in @run tags with ${test.main.class}³. Meanwhile, explicit class names remaining in @build were left untouched since they might have different meanings related to build dependencies.

Despite being extensive, the changes consisted of simple replacements like:

- * @run main/othervm BasicAuthTest
+ * @run main/othervm ${test.main.class}

- * @run main/othervm CertificateTest GOOD_CERT expectSuccess
+ * @run main/othervm ${test.main.class} GOOD_CERT expectSuccess

Addressing Reviews on 2026/04/02#

Daniel Fuchs⁴ commented after reviewing the diffs, suggesting to incorporate changes from master and resolve conflicts.

SendaoYan⁵ also suggested replacing class names in @build or system properties with ${test.main.class}. However, according to Daniel Fuchs⁴, it was better not to touch these parts in this PR.

Later, SendaoYan⁵ pointed out that HttpVersionsTest.java had been overlooked and I made additional corrections for it.

Review on 2026/04/09#

SendaoYan⁵ and Daniel Fuchs⁴ reviewed the changes and approved them. After confirming the review, /integrate was performed.

Integration on 2026/04/09#

Daniel Fuchs⁴ sponsored it with /sponsor, integrating it into JDK as 261011a⁶.

Overview#

Discovered on JBS 2026/03/18#

Addressing Reviews on 2026/04/02#

Review on 2026/04/09#

Integration on 2026/04/09#

Footnotes#