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'"