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