You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ko...@apache.org on 2022/11/24 15:58:36 UTC

svn commit: r1905512 - /subversion/branches/pristines-on-demand-on-mwf/subversion/tests/cmdline/authz_tests.py

Author: kotkov
Date: Thu Nov 24 15:58:35 2022
New Revision: 1905512

URL: http://svn.apache.org/viewvc?rev=1905512&view=rev
Log:
On the 'pristines-on-demand-on-mwf' branch: In the authz_tests.py#31, specify
and test the current behavior.

An `svn update` currently requires hydrating all text-bases beforehand.
If some of them cannot be fetched due to an authz access denied error
(or any other error, actually), then the whole operation is currently
expected to fail.

* subversion/tests/cmdline/authz_tests.py
  (remove_access_after_commit): Remove Wimp() marker.  Expect an error if
   the working copy doesn't store local pristine copies.

Modified:
    subversion/branches/pristines-on-demand-on-mwf/subversion/tests/cmdline/authz_tests.py

Modified: subversion/branches/pristines-on-demand-on-mwf/subversion/tests/cmdline/authz_tests.py
URL: http://svn.apache.org/viewvc/subversion/branches/pristines-on-demand-on-mwf/subversion/tests/cmdline/authz_tests.py?rev=1905512&r1=1905511&r2=1905512&view=diff
==============================================================================
--- subversion/branches/pristines-on-demand-on-mwf/subversion/tests/cmdline/authz_tests.py (original)
+++ subversion/branches/pristines-on-demand-on-mwf/subversion/tests/cmdline/authz_tests.py Thu Nov 24 15:58:35 2022
@@ -1613,7 +1613,6 @@ def authz_log_censor_revprops(sbox):
           '-r1', sbox.repo_url])
 
 @Skip(svntest.main.is_ra_type_file)
-@Wimp("Applying delta to a local mod needs access to the text base")
 def remove_access_after_commit(sbox):
   "remove a subdir with authz file"
 
@@ -1658,11 +1657,26 @@ def remove_access_after_commit(sbox):
 
   # And expect a mixed rev copy
   expected_status.tweak('A/D/G/rho', status='A ', entry_status='  ')
-  svntest.actions.run_and_verify_update(wc_dir,
-                                        expected_output,
-                                        expected_disk,
-                                        expected_status,
-                                        [], True)
+
+  if svntest.actions.get_wc_store_pristine(wc_dir):
+    svntest.actions.run_and_verify_update(wc_dir,
+                                          expected_output,
+                                          expected_disk,
+                                          expected_status,
+                                          [], True)
+  else:
+    # We are unable to fetch the pristine without read access.
+    # So in a working copy without local pristines, the update is
+    # currently expected to fail.
+    if svntest.main.is_ra_type_dav():
+      expected_err = ".*svn: E175013: .*[Ff]orbidden.*"
+    elif svntest.main.is_ra_type_svn():
+      expected_err = ".*svn: E170001: Authorization failed.*"
+    else:
+      raise svntest.Failure
+
+    svntest.actions.run_and_verify_update(wc_dir, None, None, None,
+                                          expected_err, True)
 
 @Issue(4793)
 @Skip(svntest.main.is_ra_type_file)