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 2011/12/20 01:36:41 UTC

svn commit: r1221047 - /subversion/trunk/subversion/tests/cmdline/externals_tests.py

Author: danielsh
Date: Tue Dec 20 00:36:41 2011
New Revision: 1221047

URL: http://svn.apache.org/viewvc?rev=1221047&view=rev
Log:
Add a regression test for issue #4085.

* subversion/tests/cmdline/externals_tests.py
  (shadowing): New XFailing test.
  (test_list): Run it.

Modified:
    subversion/trunk/subversion/tests/cmdline/externals_tests.py

Modified: subversion/trunk/subversion/tests/cmdline/externals_tests.py
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/externals_tests.py?rev=1221047&r1=1221046&r2=1221047&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/cmdline/externals_tests.py (original)
+++ subversion/trunk/subversion/tests/cmdline/externals_tests.py Tue Dec 20 00:36:41 2011
@@ -2651,6 +2651,26 @@ def include_immediate_dir_externals(sbox
     None, '--include-externals', '--depth=immediates', X)
 
 
+@Issue(4085)
+@XFail()
+def shadowing(sbox):
+  "external shadows an existing dir"
+
+  sbox.build(read_only=True)
+  wc_dir = sbox.wc_dir
+
+  # Setup external: /A/B/F as 'C' child of /A
+  externals_prop = "^/A/B/F C\n"
+
+  raised = False
+  try:
+    change_external(sbox.ospath('A'), externals_prop, commit=False)
+  except:
+    raised = True
+  if not raised:
+    raise svntest.Failure("Creating conflicting child 'C' of 'A' didn't error")
+
+
 ########################################################################
 # Run the tests
 
@@ -2694,6 +2714,7 @@ test_list = [ None,
               copy_file_externals,
               include_externals,
               include_immediate_dir_externals,
+              shadowing,
              ]
 
 if __name__ == '__main__':