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

[Daniel Rall <dl...@collab.net>]

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

> On Wed, 09 Aug 2006 22:55:06 +0530, Daniel Rall <dl...@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.