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

Re: ra_serf now passes all tests was Re: svn commit: r22796 - trunk/subversion/tests/cmdline

From: Lieven Govaerts <svnlgo_at_mobsol.be>
Date: 2006-12-24 15:51:13 CET

Justin Erenkrantz wrote:
> On Sat, Dec 23, 2006 at 11:48:19PM -0800, jerenkrantz@tigris.org wrote:
>
>> Author: jerenkrantz
>> Date: Sat Dec 23 23:48:19 2006
>> New Revision: 22796
>>
>> Log:
>> ra_serf: Accept a minor difference in returned error value with copy test.
>>
>
> So, as of this revision, ra_serf (with serf 0.1.0+) passes all regression tests
> on Mac OS X and FreeBSD. I know ra_serf works okay on Windows, but I can't
> figure out how to get the test suite compiled there for the life of me
> (win_tests.py isn't sufficient - the test binaries won't compile for me).
>
I started the tests on Windows, found two issues:
1. update_tests.py 29 fails with an assertion in libsvn_ra_serf/commit.c
line 1216.Now this test is supposed to fail anyway, and as far as I can
see the assertion is triggered exactly because of the bug that the test
is reproducing. It's a bit annoying because an assertion holds the test
run (when you have a debugger installed), but the best way to fix it
would be to fix the actual bug.

2. copy_tests.py.py 25 fails on Windows, because the path in the error
message is printed with '\\' instead of '/' as expected. Now I can
easily fix the test to support this, but doesn't svn always print paths
with forward slashes, even on Windows?

I'll start using ra_serf from now on on Windows.

Lieven

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Dec 24 15:51:34 2006

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.