This is the mail archive of the
mailing list for the Mauve project.
Re: Patch: New Lightweight tests
On Wed, Feb 22, 2006 at 09:51:39AM -0500, Lillian Angel wrote:
> > >+ harness.checkRectangleOuterColors(r, bounds2, Color.red, true);
> > >+ harness.checkRectangleCornerColors(r, bounds2, Color.red, false);
> > >+
> > >+ r.delay(3000);
> > >+ }
> > >
> > >
> > Do you know what the delay is for? Is there some other way to write the
> > test so that the delay isn't required? It's just that it would be nice
> > if the Mauve test run doesn't take longer than it has to - otherwise
> > hackers will stop running it.
> It seemed that there were some race conditions when running these tests.
> These delays were added to avoid them.
Then please document this in the testcase.
Escape the Java Trap with GNU Classpath!
Join the community at http://planet.classpath.org/