Re: Re: svn commit: r25204 - trunk/subversion/tests/cmdline
On Thu, May 31, 2007 at 11:16:05AM -0400, David Glasser wrote:
> On 5/30/07, Paul Burba <firstname.lastname@example.org> wrote:
> > > -----Original Message-----
> > > From: Malcolm Rowe [mailto:email@example.com]
> > > 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
Received on Thu May 31 17:29:04 2007
- application/pgp-signature attachment: stored
This is an archived mail posted to the Subversion Dev