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, <sbut...@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, > > >
