Jean-Baptiste Quenot wrote:
Mmm actually  I don't really  agree, making the build  fail should
not  be  intentional.  We  could  have  some convention  like  for
example naming  the failing  unit test methods  after bugTestXXX()
instead of testXXX() to make it obvious that the test does not run
and that there is a bug.

Sorry for this, this is probably my fault. ;-)

Having failing unit tests for days on end is definitely bad - it massively reduces the utility of your continuous integration server.

I originally intended this to be fixed pretty rapidly, hence the immediate tender for ideas on how to fix it on IRC and the dev mailing list. Once it wasn't, I should have commented the test out and made appropriate notes on the JIRA issue.

We have a bug tracker to work out what bugs need fixing before we do a release, so commenting things doesn't shove them into "a big black hole where no one is looking at it". It's a bug we currently have scheduled for fixing before we push 1.3.0-rc1 out. That most definitely makes it on the immediate radar.

Regards,

Al
--
Alastair Maw
Wicket-biased blog at http://herebebeasties.com

Reply via email to