svn commit: r997070 - /subversion/trunk/subversion/tests/cmdline/authz_tests.py

[cm...@apache.org]

Author: cmpilato
Date: Tue Sep 14 20:15:37 2010
New Revision: 997070

URL: http://svn.apache.org/viewvc?rev=997070&view=rev
Log:
Add a regression test for issue #3709 ("Inconsistency between "svn
list" and "svn checkout").

* subversion/tests/cmdline/authz_tests.py
  (authz_recursive_ls): New test.
  (test_list): Add reference to new test, skipping over ra-local, and
    expecting failure over ra-svn.

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

Modified: subversion/trunk/subversion/tests/cmdline/authz_tests.py
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/authz_tests.py?rev=997070&r1=997069&r2=997070&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/cmdline/authz_tests.py (original)
+++ subversion/trunk/subversion/tests/cmdline/authz_tests.py Tue Sep 14 20:15:37 2010
@@ -1026,6 +1026,40 @@ def wc_wc_copy_revert(sbox):
   expected_status.remove('A/B/E', 'A/B/E/alpha', 'A/B/E/beta')
   svntest.actions.run_and_verify_status(sbox.wc_dir, expected_status)
 
+def authz_recursive_ls(sbox):
+  "recursive ls with private subtrees"
+
+  sbox.build(create_wc = False)
+  local_dir = sbox.wc_dir
+  write_restrictive_svnserve_conf(sbox.repo_dir)
+
+  write_authz_file(sbox, {'/'       : '* = r',
+                          '/A/B/E'  : '* =',
+                          '/A/mu'   : '* =',
+                          })
+  expected_entries = [
+    'A/',
+    'A/B/',
+    'A/B/F/',
+    'A/B/lambda',
+    'A/C/',
+    'A/D/',
+    'A/D/G/',
+    'A/D/G/pi',
+    'A/D/G/rho',
+    'A/D/G/tau',
+    'A/D/H/',
+    'A/D/H/chi',
+    'A/D/H/omega',
+    'A/D/H/psi',
+    'A/D/gamma',
+    'iota',
+    ]
+  svntest.actions.run_and_verify_svn('recursive ls from /',
+                                     map(lambda x: x + '\n', expected_entries),
+                                     [], 'ls', '-R',
+                                     sbox.repo_url)
+
 ########################################################################
 # Run the tests
 
@@ -1057,6 +1091,9 @@ test_list = [ None,
               Skip(wc_wc_copy, svntest.main.is_ra_type_file),
               Skip(wc_wc_copy_revert,
                    svntest.main.is_ra_type_file),
+              XFail(Skip(authz_recursive_ls,
+                         svntest.main.is_ra_type_file),
+                    svntest.main.is_ra_type_svn),
              ]
 
 if __name__ == '__main__':