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

Re: Tests RE: svn commit: r39716 - trunk/subversion/tests/cmdline/svntest

From: Lieven Govaerts <svnlgo_at_mobsol.be>
Date: Fri, 2 Oct 2009 19:16:58 +0200

On Fri, Oct 2, 2009 at 7:03 PM, Julian Foad <julianfoad_at_btopenworld.com> wrote:
> Bert Huijben wrote:
>>
>> > Author: julianfoad
>> > Date: Wed Sep 30 10:57:51 2009
>> > New Revision: 39716
>> >
>> > Log:
>> > Make the test suite better about choosing between RA-Neon and RA-Serf.
>> > Update the RA type detection functions to reflect that the default is
>> > "serf",
>> > so that they work when "--http-library" is not specified. Give the test
>> > suite's "--http-library" option its own default value ("serf") so that
>> > it no longer depends on svn's compiled-in default value.
>> >
>> > * subversion/tests/cmdline/svntest/main.py
>> >   (http_library): Rename this variable to preferred_http_library
>> > throughout.
>> >     Clarify its doc string.
>> >   (is_not_serf): Remove this unused function.
>> >   (is_ra_type_dav_neon, is_ra_type_dav_serf): No longer assume the
>> > default is
>> >     "neon". Add doc strings.
>> >   (is_ra_type_svn, is_ra_type_file): Add doc strings.
>> >   (usage): Mention the default value of --http-library in the help
>> > message.
>>
>> For some reason it seems this change fails two tests on the neon/serf
>> buildbots.
>>
>> Neon:
>> XPASS: merge_tests.py 132: replace vs. edit tree-conflicts
>>
>> And Serf:
>> FAIL:  merge_authz_tests.py 1: skipped paths get overriding mergeinfo
>>
>> The first has an:
>> XFail(merge_replace_causes_tree_conflict,
>>                     svntest.main.is_ra_type_dav_serf),
>> marking and the second a
>> SkipUnless(Skip(mergeinfo_and_skipped_paths, svntest.main.is_ra_type_file),
>>                          svntest.main.server_has_mergeinfo),
>
> I think I have worked out what is wrong.
>
> Before my change, the test suite's "--http-library" option defaulted to
> "neon". Back in the 1.6 days and earlier, that was sensible.
>
> Because Serf is now the default, I changed the test suite's
> "--http-library" option to default to "serf", and also made the test
> suite always tell Subversion (via a config option) to prefer to use
> whichever library the test suite was told to use. I thought that made
> sense.
>
> What's happening here, I think, is the macosx/neon build-bot is building
> Subversion with Neon only, and passing no "--http-library" option to the
> test suite. I've changed the default assumption so it now gets it wrong.

Yup, that's correct. The two buildbots running on the mac have either
ra_neon or ra_serf, not both at the same time.

Lieven

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2403005
Received on 2009-10-02 19:17:26 CEST

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.