You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by sb...@apache.org on 2010/08/13 12:32:14 UTC

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

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,



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

Posted by "Hyrum K. Wright" <hy...@mail.utexas.edu>.
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,  <sb...@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,
>
>
>

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

Posted by "Hyrum K. Wright" <hy...@mail.utexas.edu>.
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,  <sb...@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,
>
>
>