[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Improve error handling in svn_ra_serf__replay_range()

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Wed, 19 Dec 2012 13:33:11 +0000

vijay <vijay_at_collab.net> writes:

> Index: subversion/tests/cmdline/svnrdump_tests.py
> ===================================================================
> --- subversion/tests/cmdline/svnrdump_tests.py (revision 1423268)
> +++ subversion/tests/cmdline/svnrdump_tests.py (working copy)
> @@ -765,8 +765,28 @@
>
>
> #----------------------------------------------------------------------
> +@SkipUnless(svntest.main.is_ra_type_dav)
> +def dump_url_not_in_head(sbox) :
> + "dump: URL deleted in HEAD should return error"
> + sbox.build(create_wc = False)
> +
> + E_url = sbox.repo_url + '/A/B/E'
>
> + # Delete directory 'E'from repository.
> + svntest.actions.run_and_verify_svn(None, None, [], "rm", E_url, "-m",
> + "delete 'E'")
>
> + expected_dump_fail_err_re = "svnrdump: E160013: '.*' path not found"
> +
> + # Returns error as in issue #4100.
> + svntest.actions.run_and_verify_svnrdump(None, svntest.verify.AnyOutput,
> + expected_dump_fail_err_re, 1, '-q',
> + 'dump', E_url)
> +
> +
> +#----------------------------------------------------------------------
> +
> +
> ########################################################################
> # Run the tests

You haven't added the test to the test_list so it is not being run.

Why is the test DAV only? This test should be run over all RA layers.

-- 
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download
Received on 2012-12-19 14:33:54 CET

This is an archived mail posted to the Subversion Dev mailing list.