svn commit: r1925478 - /subversion/trunk/subversion/tests/cmdline/input_validation_tests.py

Author: rinrab
Date: Fri May  9 14:23:58 2025
New Revision: 1925478

URL: http://svn.apache.org/viewvc?rev=1925478&view=rev
Log:
tests/input_validation_tests.py::invalid_patch_targets(): Improve the test
by adding more assertions for different combinations of valid and bad paths
of patch files and working copies.

* subversion/tests/cmdline/input_validation_tests.py
  (invalid_patch_targets): Ditto (and expand for loop).

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

Modified: subversion/trunk/subversion/tests/cmdline/input_validation_tests.py
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/input_validation_tests.py?rev=1925478&r1=1925477&r2=1925478&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/cmdline/input_validation_tests.py (original)
+++ subversion/trunk/subversion/tests/cmdline/input_validation_tests.py Fri May  9 14:23:58 2025
@@ -222,9 +222,18 @@ def invalid_status_targets(sbox):
 def invalid_patch_targets(sbox):
   "non-working copy paths for 'patch'"
   sbox.build(read_only=True)
-  for (target1, target2) in [("foo", "^/"), ("^/", "^/"), ("^/", "foo")]:
-    run_and_verify_svn_in_wc(sbox, "svn:.*is not a local path", 'patch',
-                             target1, target2)
+
+  run_and_verify_svn_in_wc(sbox, "svn:.*is not a local path", 'patch',
+                           "iota", "^/")
+
+  run_and_verify_svn_in_wc(sbox, "svn:.*is not a local path", 'patch',
+                           "^/", "^/")
+
+  run_and_verify_svn_in_wc(sbox, "svn:.*does not exist", 'patch',
+                           "non-existent.patch", "")
+
+  run_and_verify_svn_in_wc(sbox, "svn:.*does not exist", 'patch',
+                           "iota", "non-existent-wc")
 
 def invalid_switch_targets(sbox):
   "non-working copy paths for 'switch'"