Re: svn commit: r16304 - trunk/subversion/bindings/swig/python/tests/trac/versioncontrol/tests

On 9/27/05, Branko Èibej <brane@xbc.nu> wrote:
> djames@tigris.org wrote:
>
> >+ # Remove the trac-svnrepos directory, so that we can
> >+ # ensure a fresh start.
> >+ try:
> >+ self.tearDown()
> >+ except:
> >+ pass
> >
> >
> Why the last two lines? If the test repository can't be removed, it's a
> safe bet that the tests won't work anyway.
If the test repository doesn't exist, then self.tearDown() throws an
exception. We catch this exception, so that the test scripts will work
if the trac-svnrepos directory has not been created yet.

(Perhaps it would be better to fix self.tearDown to check whether the
repository actually exists, before trying to delete it.)

Cheers,

David

--
David James -- http://www.cs.toronto.edu/~james