RFR: 8286620: Create regression test for verifying setMargin() of JRadioButton [v4]

Phil Race prr at openjdk.java.net
Wed Jun 1 18:04:37 UTC 2022


On Wed, 1 Jun 2022 17:37:42 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

> > > Shall the test be located in `test/jdk/javax/swing/JRadioButton/4380543/bug4380543.java` instead of `test/jdk/javax/swing/JRadioButton/bug4380543.java` ?
> > > Usually, there's been a folder with bugid (`/4380543/`) for each test. Do we use the flat layout now?
> > 
> > 
> > @aivanov-jdk Recently we have started to use the latter and remove the extra folder (`/4380543/`)
> 
> Thank you for clarification, Harshitha.

Yes, blame (?) me :-) 
We may not always remember to point it out but it is what we want.
I've never seen the point in an extra level of folder except when the test is composed of multiple files all unique to the test.
I also request that new tests not be given names like bug87654321.java but instead be named in a way that you can tell what they are supposed to be testing like in this case something like RadioButtonMarginTest.java

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

PR: https://git.openjdk.java.net/jdk/pull/8721



More information about the client-libs-dev mailing list