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

Re: svn commit: r25292 - trunk/subversion/tests/cmdline

From: Kamesh Jayachandran <kamesh_at_collab.net>
Date: 2007-06-05 14:50:16 CEST

Malcolm Rowe wrote:
> On Tue, Jun 05, 2007 at 05:12:29AM -0700, kameshj@tigris.org wrote:
>> Currently 'authz_svnserve_anon_access_read' test is executed for ra_dav
>> tests too. It is due to 'svntest.main.is_ra_type_dav()', changing that to
>> 'svntest.main.is_ra_type_dav' should skip this testcase for ra_dav tests.
> This is a poor log message: it should primarily describe the change, not
> a description of the current problem.
Will correct it.

> But more importantly...
>> @@ -861,7 +861,7 @@
>> authz_validate,
>> authz_locking,
>> Skip(authz_svnserve_anon_access_read,
>> - svntest.main.is_ra_type_dav()),
>> + svntest.main.is_ra_type_dav),
>> ]
> ... does this actually work? Skip() takes a conditional (evaluated
> at parse time), while XFail() takes a lambda (evaluated immediately
> prior to test execution).
> That discrepancy is annoying, and I suspect that is_ra_type_dav() may
> actually not work (i.e. always returns false) at the point that this
> list is evaluated (because argument parsing doesn't happen until later),
> and so whereas we'd never Skip the test before, we now always skip it.

Just realised it after the commit. Even though I kept running tests on
svnserve and dav often to fix 'authz_svnserve_anon_access_read' test
failure over a unclean WC (due to earlier test runs). I missed this
always skipping problem, my mistake. Working on it.
> See for example the way that the test recover_fsfs (in svnadmin_tests.py)
> works:
> def recover_fsfs(sbox):
> "recover a repository (FSFS only)"
> # Ideally, we'd include a variant of this in a Skip() condition, except
> # that Skip() evaluates its value at construction, rather than accepting
> # a lambda to evaluate later. This pragma isn't available at test
> # construction time.
> if not svntest.main.is_fs_type_fsfs():
> raise svntest.Skip
Yes it seems like a way to go or fixing the discrepancy between XFail
and Skip.

With regards
Kamesh Jayachandran

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Jun 5 14:49:59 2007

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.