[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: When to commit failing tests (was: Re: svn commit: r17032 - trunk/subversion/tests/clients/cmdline)

From: <kfogel_at_collab.net>
Date: 2005-10-26 19:43:57 CEST

"Peter N. Lundblad" <peter@famlundblad.se> writes:
> You normally don't commit something that you know have failing tests. If
> the nightly build fail, because you did something platform-dependent or
> something, then that's another story. I really don't think knowingly
> commit failing tests non-XFail is a good way of alerting people that there
> is a regression. You can post to dev@, file an issue or just fix it in a
> few days.
> The problem with your approach is that it slows down things for everyone
> else. When I want to run tests and get a failure, I have to examine and
> dscover that it was just that other regression that still fails. If I made
> a commit, I have to examine the nightly failure mails, because I can't
> just assume that it is your test that fail. I think that's a waste of
> time.

+1 IMHO.

The value of the test suite and the nightly builds is in notifying us
of things we don't already know. So if someone knows that a test is
failing, then we don't need to get that information by other means --
the person can just set the test XFail, and file an issue or do
whatever is appropriate, to record the fact of the failure.

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Oct 26 21:02:08 2005

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.