On Fri, Nov 20, 2009 at 1:52 PM, Jim Fulton <j...@zope.com> wrote: > On Fri, Nov 20, 2009 at 1:36 PM, Tres Seaver <tsea...@palladion.com> wrote: ... >>>> I just checked in a stupid test to ensure that repozo can be imported. >>>> >>>> - --- src/ZODB/scripts/tests.py (revision 105913) >>>> +++ src/ZODB/scripts/tests.py (working copy) >>>> @@ -26,6 +26,11 @@ >>>> (re.compile('hash=[0-9a-f]{40}'), >>>> 'hash=b16422d09fabdb45d4e4325e4b42d7d6f021d3c3')]) >>>> >>>> +class RepozoTests(unittest.TestCase): >>>> + >>>> + def test_importability(self): >>>> + from ZODB.scripts import repozo >>>> +
... >>> That test passes under ZODB 3.9.3 with Python 2.6. >> >> It emits the same DeprecationWarning (the "bug" Withers fixed). I did >> have to munge the 'test_suite' dinosaur as well. > > Fair enough. Given that this provokes the symptom that was fixed. > I can live with this. Well, not really fair enough. I just looked at the change that Chris made. It has a bug that would be caught by your test if anyone tried to port repozo to Python 3. More importantly, Chris' change touches non-trivial code that isn't exercised by your test. It *looks* OK (aside from the minor bug). We are doing users a serious disservice giving them such an important tool with minimal tests and no automated tests. I'm going to back out these changes. If someone really cares about repozo in the slightest, they'll at least convert the existing manual test into an automated test. I'm pretty sure that this is a straightforward project. With an automated version of the manual test, I'd be comfortable reapplying Chris' change. Is anyone willing to convert the manual test to an automated one? Jim -- Jim Fulton _______________________________________________ For more information about ZODB, see the ZODB Wiki: http://www.zope.org/Wikis/ZODB/ ZODB-Dev mailing list - ZODB-Dev@zope.org https://mail.zope.org/mailman/listinfo/zodb-dev