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

概要#

　test/jdk/java/net/httpclient 配下の多数のテストでは，JTReg（OpenJDK 独自のリグレッション・テスト・フレームワーク）の @run タグにテスト・クラス名を直接書いていました。たとえば，以下のような指定です。

@run junit/othervm AltServiceUsageTest

　しかしながら，このように自己参照的なテスト・クラス名がコード内にあると，少々厄介な問題が生じます。テストは往々にしてコピ・ペされるものですから，これの変更を忘れると意図しないテストを実行してしまいます。

　これを受けて JTReg 7.5.2 からは ${test.main.class} というプレース・ホルダが導入されました¹。これは JTReg アノテーションが付けられたクラスをメタ的に指定するもので，例えば Main クラスに @run ${test.main.class} がついていれば @run Main というふうになります。

　さて，今回の修正ではこのような @run タグ上の明示的なクラス名を ${test.main.class} に置き換えるようにしました。

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

　変更対象は 313 ファイルで，機械的な置換が中心です²。

2026/03/18 JBS で発見#

　本件を JBS で発見し，java/net/httpclient 配下のテストを確認しました。 @run タグにテスト・クラス名を直接書いているファイルが多数あり，形式も main，junit，othervm，timeout 付きなど複数ありました。

　PR では @run タグで実行対象として現れるクラス名だけを ${test.main.class} へ置き換える方針にしました³。一方で @build に残っている明示的なクラス名は，ビルド依存として別の意味を持つ可能性があるため，この PR では触らないことにしました。

　変更量は大きいものの，内容は以下のような単純な置換です。

- * @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

2026/04/02 レビュー対応#

　Daniel Fuchs 氏⁴から，差分を確認したうえで，master の変更を取り込んで conflict を解消するようコメントをいただきました。

　また，SendaoYan 氏⁵から，HttpClientExceptionTest の @build やシステム・プロパティ中のクラス名も ${test.main.class} にできるか，というコメントをいただきました。これについては Fuchs 氏⁴によると，@build 句やシステム・プロパティはこの PR では触らない方がよいとのことでした。

　その後，SendaoYan 氏⁵から HttpVersionsTest.java が漏れているとの指摘があり，これを追加で修正しました。

2026/04/09 レビュー#

　SendaoYan 氏⁵と Fuchs 氏⁴にレビューをいただき，Approve されました。レビューを確認したのち，/integrate を行いました。

2026/04/09 統合#

　Fuchs 氏⁴に /sponsor をしていただき，261011a⁶ として JDK へ統合されました。

概要#

2026/03/18 JBS で​発見#

2026/04/02 レビュー対応#

2026/04/09 レビュー#

2026/04/09 統合#

Footnotes#

2026/03/18 JBS で発見#