You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by da...@apache.org on 2022/03/17 23:49:13 UTC
svn commit: r1899011 - /subversion/trunk/subversion/tests/cmdline/upgrade_tests.py
Author: danielsh
Date: Thu Mar 17 23:49:13 2022
New Revision: 1899011
URL: http://svn.apache.org/viewvc?rev=1899011&view=rev
Log:
* subversion/tests/cmdline/upgrade_tests.py
(basic_upgrade_1_0, upgrade_1_0_with_externals):
Update a comment. Note that the test doesn't verify all it could.
The code referenced by the outgoing comments was removed in r1898536.
Modified:
subversion/trunk/subversion/tests/cmdline/upgrade_tests.py
Modified: subversion/trunk/subversion/tests/cmdline/upgrade_tests.py
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/upgrade_tests.py?rev=1899011&r1=1899010&r2=1899011&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/cmdline/upgrade_tests.py (original)
+++ subversion/trunk/subversion/tests/cmdline/upgrade_tests.py Thu Mar 17 23:49:13 2022
@@ -469,7 +469,17 @@ def basic_upgrade_1_0(sbox):
# Now upgrade the working copy
svntest.actions.run_and_verify_svn(None, [],
'upgrade', sbox.wc_dir)
- # And the separate working copy below COPIED or check_format() fails
+ # And the separate working copy below COPIED
+ #
+ # ### This was originally added in r919021, during 1.7 development, because
+ # ### check_format() recursed into the separate working copy.
+ # ###
+ # ### The remainder of the test passes if this call is removed.
+ # ###
+ # ### So, for now, this call serves only as a smoke test, to confirm that the
+ # ### upgrade returns 0. However:
+ # ###
+ # ### TODO: Verify the results of this upgrade
svntest.actions.run_and_verify_svn(None, [],
'upgrade',
os.path.join(sbox.wc_dir, 'COPIED', 'G'))
@@ -1475,7 +1485,18 @@ def upgrade_1_0_with_externals(sbox):
# Now upgrade the working copy
svntest.actions.run_and_verify_svn(None, [],
'upgrade', sbox.wc_dir)
- # And the separate working copy below COPIED or check_format() fails
+ # And the separate working copy below COPIED
+ #
+ # ### This was originally added in r1702474, during 1.10 development, because
+ # ### check_format() recursed into the separate working copy. It was copied
+ # ### from basic_upgrade_1_0() above.
+ # ###
+ # ### The remainder of the test passes if this call is removed.
+ # ###
+ # ### So, for now, this call serves only as a smoke test, to confirm that the
+ # ### upgrade returns 0. However:
+ # ###
+ # ### TODO: Verify the results of this upgrade
svntest.actions.run_and_verify_svn(None, [],
'upgrade',
os.path.join(sbox.wc_dir, 'COPIED', 'G'))