On Mon, 16 Dec 2024 20:33:38 GMT, Naoto Sato <na...@openjdk.org> wrote:

> A very minor test fix.

test/jdk/java/time/tck/java/time/TCKZoneId.java line 99:

> 97:     // SHORT_IDS
> 98:     
> //-----------------------------------------------------------------------
> 99:     @Test

A passing comment is that I personally find it confusing to see `@Test` on the 
class and the test methods. With TestNG then I think you can use one or the 
other. It looks like the intention here is to put the annotation on the methods 
and eventually migrate to JUnit, is that right?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22773#discussion_r1888053945

Reply via email to