You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ha...@apache.org on 2021/01/28 15:17:03 UTC

svn commit: r1885981 - /subversion/branches/issue-4869/subversion/tests/cmdline/info_tests.py

Author: hartmannathan
Date: Thu Jan 28 15:17:02 2021
New Revision: 1885981

URL: http://svn.apache.org/viewvc?rev=1885981&view=rev
Log:
branches/issue-4869: Improve a few comments in new regression test

* subversion/tests/cmdline/info_tests.py:
  (info_tree_conflict_source): Improve a few comments: One minor grammar fix;
    also remove mention of svn_cl__append_conflict_info_xml() and
    append_tree_conflict_info_xml() because the test should be agnostic of the
    internals under test.

Modified:
    subversion/branches/issue-4869/subversion/tests/cmdline/info_tests.py

Modified: subversion/branches/issue-4869/subversion/tests/cmdline/info_tests.py
URL: http://svn.apache.org/viewvc/subversion/branches/issue-4869/subversion/tests/cmdline/info_tests.py?rev=1885981&r1=1885980&r2=1885981&view=diff
==============================================================================
--- subversion/branches/issue-4869/subversion/tests/cmdline/info_tests.py (original)
+++ subversion/branches/issue-4869/subversion/tests/cmdline/info_tests.py Thu Jan 28 15:17:02 2021
@@ -864,12 +864,9 @@ def info_tree_conflict_source(sbox):
   sbox.simple_update()
 
   # Rev 3:
-  # - edit A/B/lambda
-  #   (tests text_conflicted case of svn_cl__append_conflict_info_xml())
-  # - add property to A/B/E/alpha
-  #   (tests props_conflicted case of svn_cl__append_conflict_info_xml())
-  # - rename A/B/F to A/B/Z
-  #   (tests the case in append_tree_conflict_info_xml())
+  # edit A/B/lambda to test text conflict case
+  # add property to A/B/E/alpha to test property conflict case
+  # rename A/B/F to A/B/Z to test tree conflict case
 
   svntest.main.file_write(lambda_path, 'B/lambda side of conflict')
   sbox.simple_propset('blue', 'azul', 'A/B/E/alpha')
@@ -886,7 +883,7 @@ def info_tree_conflict_source(sbox):
   sbox.simple_move('A/B2/F', 'A/B2/Y');
   sbox.simple_commit()
 
-  # Now merge B2 into B and cause a text conflict, property conflict, and
+  # Now merge B2 into B to cause a text conflict, property conflict, and
   # tree conflict
   sbox.simple_update()