On Tue, 30 Dec 2025 13:07:33 GMT, Michael McMahon <[email protected]> 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
