Re: [PATCH] Add a test to cover the regression introduced in r1075802

On 03/11/2011 10:03 AM, Arwin Arni wrote:
> Index: ../subversion/tests/cmdline/merge_tests.py
> ===================================================================
> --- ../subversion/tests/cmdline/merge_tests.py (revision 1080126)
> +++ ../subversion/tests/cmdline/merge_tests.py (working copy)
> @@ -16586,6 +16586,102 @@
> if not os.access(beta_path, os.X_OK):
> raise svntest.Failure("beta is not marked as executable after commit")
>
> +@XFail()
> +def dry_run_merge_conflicting_binary(sbox):
> + "dry run merge should not create conflict resolution files"

This long description line triggers the AssertionError about the test
docstring needing to be 50 characters or less.

> + svntest.actions.run_and_verify_merge(other_wc, '2', '3',
> + sbox.repo_url, None,
> + expected_output,
> + expected_mergeinfo_output,
> + expected_elision_output,
> + expected_disk,
> + expected_status,
> + expected_skip,
> + None, None, None, None, None,
> + True, True, '--allow-mixed-revisions',
> + other_wc)

As this is a test of a dry-run merge, I find the use of
run_and_verify_merge() a bit obfuscating. I think it'd be better to
explicitly run a --dry-run merge so that it's obvious that what you're
testing is exactly that.

And, as I said elsethread, the patch didn't even apply to HEAD. So that
needs to be reworked.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand