[Zope-dev] SVN: DateTime/trunk/src/DateTime/tests/testDateTime.py add a failing test for a regression in parsing ISO format datetimes from DateTime 2.10, as discussed at http://dev.plone.org/plone/ticket/10140 ...note that this will give a false positive if run on a computer where GMT is the local timezone

Martin Aspeli optilude+lists at gmail.com
Fri Mar 5 21:53:15 EST 2010


Tres Seaver wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> David Glick wrote:
>> Log message for revision 109667:
>>    add a failing test for a regression in parsing ISO format datetimes from DateTime 2.10, as discussed at http://dev.plone.org/plone/ticket/10140 ...note that this will give a false positive if run on a computer where GMT is the local timezone
>
> Please don't deliberately check in failing tests on the trunk.  If you
> need to do this, make a branch, and ask on the mailing list for people
> to investigate your branch.

Why not? Trunk is (well, was) broken. This makes it clear. The 
regression actually happened ages ago, but no-one had written a decent 
test for the original functionality, so the regression was only 
discovered in application software. This test corrects that omission, 
and helped a few people co-ordinate addressing the issue.

Why benefit do we get from not making the breakage explicit to everyone?

Martin

-- 
Author of `Professional Plone Development`, a book for developers who
want to work with Plone. See http://martinaspeli.net/plone-book



More information about the Zope-Dev mailing list