Re: svn commit: r24898 - trunk/subversion/tests/cmdline

[Lieven Govaerts <sv...@mobsol.be>]

lgo@tigris.org wrote:
> Author: lgo
> Date: Wed May  2 15:09:29 2007
> New Revision: 24898
>
> Log:
> Follow up to r24893. Fix test relocate_deleted_missing_copied: when committing
> a copied directory 'svn commit' should only show the copied directory as being
> added, not all of its children too.
>
> * subversion/tests/cmdline/switch_tests.py
>   (relocate_deleted_missing_copied): remove H2's children from the expected
>    commit output.
>
>
> Modified:
>    trunk/subversion/tests/cmdline/switch_tests.py
>
> Modified: trunk/subversion/tests/cmdline/switch_tests.py
> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/tests/cmdline/switch_tests.py?pathrev=24898&r1=24897&r2=24898
> ==============================================================================
> --- trunk/subversion/tests/cmdline/switch_tests.py	(original)
> +++ trunk/subversion/tests/cmdline/switch_tests.py	Wed May  2 15:09:29 2007
> @@ -587,9 +587,6 @@
>    # Commit to verify that copyfrom URLs have been relocated
>    expected_output = svntest.wc.State(wc_dir, {
>      'A/D/H2'       : Item(verb='Adding'),
> -    'A/D/H2/chi'   : Item(verb='Adding'),
> -    'A/D/H2/omega' : Item(verb='Adding'),
> -    'A/D/H2/psi'   : Item(verb='Adding'),
>      })
>    expected_status.tweak('A/D/H2', 'A/D/H2/chi', 'A/D/H2/omega', 'A/D/H2/psi',
>                          status='  ', wc_rev='3', copied=None)
>   
Can anyone review r24893(-4) and this one? I'm pretty sure the change is
correct, but I don't like changing the behavior of existing tests so a
confirmation would be nice.

Lieven

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