The Metal L&F does not check the colour profile. Because of this I submitted a patch to correct the metal L&F colors to match with the test. But still I feel the test is inconsistent because it checks different colours at different points for the same component which causes the test to fail.
Also, the purpose is not to test the metal L&F but Basic L&F? Should the tests be cancelled off? Regards, Tharindu On Fri, Jun 20, 2008 at 4:16 PM, Alexei Zakharov <[EMAIL PROTECTED]> wrote: > But do we have the corresponding test for Metal L&F? > > Regards, > Alexei > > 2008/6/20, Mackie Mathew <[EMAIL PROTECTED]>: > > Some of the tests are trying to assert the color profile of the basic > L&F, > > which is abstract. It's extended and methods are overridden by the Metal > > L&F. So the test is carried out against the color profile of the latter. > I > > don't see how it can be tested against the basic L&F's color profile. > > > > Regards, > > > > Tharindu > > > > > > On Thu, Jun 19, 2008 at 7:46 PM, Nathan Beyer <[EMAIL PROTECTED]> > wrote: > > > > > Are you saying that the test is asserting a default behavior? It would > make > > > sense to set a specific color and then assert that color actually > > > displayed, > > > but testing a default can be difficult. > > > > > > -Nathan > > > > > > On Thu, Jun 19, 2008 at 5:10 AM, Mackie Mathew <[EMAIL PROTECTED]> > > > wrote: > > > > > > > The Basic Borders test checks to match colour patterns in the Basic > Look > > > > and > > > > Feel. > > > > > > > > But the Metal Look and Feel is taken as the default L&F. Since the > Basic > > > > L&F > > > > is abstract anyway, the test checks against colours of the Metal L&F > > > > instead > > > > of the Basic L&F and thus fails under some tests. > > > > > > > > Should color testing be taken out of this test? > > > > > > > > Regards, > > > > > > > > Tharindu > > > > > > > > > >
