On Thu, 31 May 2007, Malcolm Rowe wrote:
> On Thu, May 31, 2007 at 11:16:05AM -0400, David Glasser wrote:
> > On 5/30/07, Paul Burba <pburba@collab.net> wrote:
> > > > -----Original Message-----
> > > > From: Malcolm Rowe [mailto:malcolm-svn-dev@farside.org.uk]
> > > > chdir() calls should be inside a try/finally block so that an
> > > > exception inside won't break the rest of the tests (due to
> > > > failure to reset the cwd).
> >
> > > I made this one change (r25225) mentioned by Malcolm as I added a new
> > > merge test today. Without this change my test, which comes after
> > > copy_src_detection_bug_if_target_has_many_ancestors_in_same_commit() in
> > > the test_list, fails.
> >
> > Hmm. Could we just put cwd save/reset into the test driver, and
> > delete all of the repetitive try/finally blocks in the tests?
> >
>
> Yes, that would seem like a much more sensible option. We'd still need
> to save/restore the cwd in a few places, but most would simply become
> os.chdir('foo').
+1
- application/pgp-signature attachment: stored
Received on Thu May 31 20:01:31 2007