On Wed, 4 Mar 2026 06:05:37 GMT, SendaoYan <[email protected]> wrote: >> Daniel Fuchs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Review feedback: Assertions import static > > test/jdk/java/net/httpclient/whitebox/java.net.http/jdk/internal/net/http/common/DemandTest.java > line 96: > >> 94: @Test >> 95: public void test08() { >> 96: Assertions.assertThrows(IllegalArgumentException.class, () -> { > > Why not "import static org.junit.jupiter.api.Assertions.assertThrows;" to > avoid declare the class name everytime
No particular reason. It was how the test was originally coded. I have pushed a commit that makes the suggested change. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30027#discussion_r2883302102
