<Swing Dev> RFR: 8058704: Nimbus does not honor JTextPane background color [v2]
Prasanta Sadhukhan
psadhukhan at openjdk.java.net
Mon Aug 9 13:28:12 UTC 2021
On Mon, 9 Aug 2021 13:01:17 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>
>
> > > The tests are nearly the same: the only difference is the component being tested. I suggest merging the two tests into one test which tests both components one after another.
> >
> >
> > I would rather have as separate test as the fix is in 2 different component, I could have raised separate PRs as fix area is different although fix seems to be same..and I don't see it as anyhting other than test optimization..
> > If you absolutely cannot approve without test merged, then please let me know. I would try to do
>
> I'm for fixing these two issues under one PR. The fix is basically the same, so it makes it easier to review and track changes later.
>
> Yes, it's kind of a test optimisation. Yet the tests are similar in nature, they exercise the same behaviour in two different components. It just makes sense to re-use the code. If a fix is necessary to one test, it is necessary for the other — same idea why avoiding duplicate code is good.
>
> It's not a show-stopper though if everyone agrees it's better to have two different tests.
OK..Not a problem. Merged...
-------------
PR: https://git.openjdk.java.net/jdk/pull/4930
More information about the swing-dev
mailing list