On Wed, 9 Sep 2026 17:37:23 GMT, Andy Goryachev <[email protected]> wrote:
>> I had the same thought actually, but did not comment as it was the same >> before. >> Would be fine for me as well -- while here we could also drop `public` as it >> is not needed as well (since JUnit 5). Also very minor obviously. > > maybe keep `public` for human pattern recognition. there is no reason to > drop it, I think. - I see both patterns in controls unit tests - `@Test` specified on a separate line as well on the same line of the test. Anyway, I will change it for this test. - I agree to keep `public` keyword. If decided in future, we can get rid of all occurrences from all tests. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/2304#discussion_r3975822472
