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