RFR: 8286620: Create regression test for verifying setMargin() of JRadioButton [v4]
Tejesh R
tr at openjdk.java.net
Thu Jun 2 04:51:36 UTC 2022
On Wed, 1 Jun 2022 18:00:44 GMT, Phil Race <prr 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
Sure @prrace , will rename the test. Harshitha also suggested that previously, I thought of retaining for this test.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8721
More information about the client-libs-dev
mailing list