You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ju...@apache.org on 2015/01/23 11:10:35 UTC
svn commit: r1654162 -
/subversion/trunk/subversion/tests/cmdline/svnrdump_tests.py
Author: julianfoad
Date: Fri Jan 23 10:10:34 2015
New Revision: 1654162
URL: http://svn.apache.org/r1654162
Log:
Clean up a bit of test code.
* subversion/tests/cmdline/svnrdump_tests.py
(load_non_deltas_copy_with_props): Set the properties only on the nodes
that need it, avoiding a recursive prop-set. The recursive prop-set
worked, but not quite as intended, because sbox.simple_propset appended
the directory path to all args except the first, which meant the prop val
got altered too.
Modified:
subversion/trunk/subversion/tests/cmdline/svnrdump_tests.py
Modified: subversion/trunk/subversion/tests/cmdline/svnrdump_tests.py
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/svnrdump_tests.py?rev=1654162&r1=1654161&r2=1654162&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/cmdline/svnrdump_tests.py (original)
+++ subversion/trunk/subversion/tests/cmdline/svnrdump_tests.py Fri Jan 23 10:10:34 2015
@@ -834,8 +834,8 @@ def load_non_deltas_copy_with_props(sbox
# at (new rev - 1) does not exist
# Set properties on each node to be copied
- sbox.simple_propset('-R', 'p', 'pval', 'A')
- sbox.simple_propset('-R', 'q', 'qval', 'A')
+ sbox.simple_propset('p', 'v', 'A/mu', 'A/B', 'A/B/E')
+ sbox.simple_propset('q', 'v', 'A/mu', 'A/B', 'A/B/E')
sbox.simple_commit()
sbox.simple_update() # avoid mixed-rev