RFR: 8373893: Refactor networking http server tests to use JUnit [v5]

Daniel Jeliński djelinski at openjdk.org
Tue Dec 30 18:30:01 UTC 2025


On Tue, 30 Dec 2025 13:07:33 GMT, Michael McMahon <michaelm at openjdk.org> wrote:

>> as far as I can tell, we only need to change the provider methods and the beforeAll/afterAll methods to static in order to remove the annotation. The test methods must not be static.
>
>> as far as I can tell, we only need to change the provider methods and the beforeAll/afterAll methods to static in order to remove the annotation. The test methods must not be static.
> 
> Okay, that seemed to work for `DateFormatterTest.java` Obviously fields have to be declared static as well, but I'll see how it works for the others now.

The warning can be turned into an error by executing the tests with:

JTREG=VM_OPTIONS=-Djunit.platform.discovery.issue.severity.critical=INFO

wonder if we can make it the default.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/28908#discussion_r2653594852


More information about the net-dev mailing list