Re: [PATCH] simple optimization and new test case for 'svn cp URL wc'

On Thu, 10 Aug 2006, Madan S. wrote:

> On Wed, 09 Aug 2006 22:55:06 +0530, Daniel Rall <dlr@collab.net> wrote:
>
> >On Wed, 09 Aug 2006, Madan S. wrote:
>
> [snip]
>
> >> Also, I almost broke a case trying one more optimization - Using
> >>svn_wc_adm_open3() instead of svn_wc_adm_probe_open3(). But the tests
> >>kept
> >>passing - of course, that case was not being tested by the
> >>respos_to_wc()
> >>test of copy tests. So, I added it.
> >
> >Could you describe the test case? This is not enough context to
> >understand.
>
> If a target path of the copy operation already exists and is a directory,
> the source should be copied under the target directory. This specific case
> is not currently tested in repos_to_wc(). I inadverdently broke this
> functionality with some stupid simplifications, but the tests kept
> passing. Hence the itch to fix that.

Thanks Madan! Nice addition. I've committed the copy_tests.py change
to trunk in r21497.