Re: [PATCH] Issue #854 Regression Test

Shlomi Fish <shlomif@vipe.stud.technion.ac.il> writes:

> +def revert_newly_added_dir(sbox):
> + "revert inside a newly added dir"
> +
> + sbox.build()
> +
> + wc_dir = sbox.wc_dir
> +
> + origin_dir = os.getcwd()
> +
> + os.chdir(wc_dir)
> +
> + os.mkdir("foo")
> +
> + svntest.main.run_svn(None, 'add', "foo")
> +
> + os.chdir("foo")
>
> + svntest.main.run_svn(None, 'revert', ".")

What is revert supposed to do in this circumstance?

> +
> + os.chdir("..")

$ svn st
~ foo

That doesn't look right, foo is still schedule add although it is
incomplete.

>
> + out, err = svntest.main.run_svn(None, 'update')

$ svn up
D foo
Updated to revision 1.

That doesn't look right either. Although the originally reported
error has changed, I don't think 854 can be described as fixed.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org