Re: svn commit: r16304 - trunk/subversion/bindings/swig/python/tests/trac/versioncontrol/tests
On 9/27/05, Branko ╚ibej <firstname.lastname@example.org> wrote:
> email@example.com 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.)
David James -- http://www.cs.toronto.edu/~james
Received on Tue Sep 27 20:46:41 2005
This is an archived mail posted to the Subversion Dev