Re: [PATCH] Add a test in dav-mirror-autocheck.sh to showcase a out-of-date slave related bug

On Mon, Apr 18, 2011 at 6:51 AM, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
> On Mon, 18 Apr 2011 16:05 +0530, "Arwin Arni" <arwin_at_collab.net> wrote:
>> Hi All,
>>
>> This patch adds a test case to
>> subversion/tests/cmdline/dav-mirror-autocheck.sh to showcase the
>> following bug.
>>
>> In a master/slave repositories setup where writes are proxied to the
>> master and reads are handles by the slave repository, if the slave
>> repository is behind the master by one or more revisions, certain
>> operations that one would expect to succeed are failing.
>>
>
> Why are you expecting it to succeed?
>
> Consider:
>
> % svn co ^/ wc
> % cd wc
> % svn mkdir -mm ^/branches/foo
> % svn cp trunk branches/bar
> % svn ci -mm
>
> which should fail with 'out of date'.
>
> Therefore I think your case should fail as well.

I know why it fails, but I would not expect it to fail as a user, even
with a proxy. I did not look at Arwin's test but it does not require
a WC to show the failure. This also fails:

$ svn mkdir url://branches/branch1
$ svn mkdir url://branches/branch2

-- 
Thanks
Mark Phippard
http://markphip.blogspot.com/