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

Re: svn commit: r985152 - /subversion/trunk/subversion/tests/cmdline/diff_tests.py

From: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: Fri, 13 Aug 2010 08:14:09 -0500

Should this test be XFail over dav, rather than just skipped? It
seems like a reasonable operation to be doing over dav.

-Hyrum

On Fri, Aug 13, 2010 at 5:32 AM, <sbutler_at_apache.org> wrote:
> Author: sbutler
> Date: Fri Aug 13 10:32:14 2010
> New Revision: 985152
>
> URL: http://svn.apache.org/viewvc?rev=985152&view=rev
> Log:
> Skip diff_tests.py 37 (basic_diff_summarize) to unbreak the buildbot, if
> running over ra_dav.
>
> Actually, I think the behavior with ra_dav is correct.  But I was testing
> with ra_file when I edited the test, and both ra_file and ra_svn give
> weird output (status='AM').
>
> * subversion/tests/cmdline/diff_tests.py:
>  (test_list): Skip #37 for ra_dav.
>
> Modified:
>    subversion/trunk/subversion/tests/cmdline/diff_tests.py
>
> Modified: subversion/trunk/subversion/tests/cmdline/diff_tests.py
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/diff_tests.py?rev=985152&r1=985151&r2=985152&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/tests/cmdline/diff_tests.py (original)
> +++ subversion/trunk/subversion/tests/cmdline/diff_tests.py Fri Aug 13 10:32:14 2010
> @@ -3681,7 +3681,8 @@ test_list = [ None,
>               diff_repos_working_added_dir,
>               diff_base_repos_moved,
>               diff_added_subtree,
> -              basic_diff_summarize,
> +              Skip(basic_diff_summarize,
> +                   svntest.main.is_ra_type_dav),
>               diff_weird_author,
>               diff_ignore_whitespace,
>               diff_ignore_eolstyle,
>
>
>
Received on 2010-08-13 15:14:51 CEST

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