Malcolm Rowe wrote:
> On Tue, Jun 05, 2007 at 05:12:29AM -0700, email@example.com 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 @@
>> - 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)
> 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
To unsubscribe, e-mail: firstname.lastname@example.org
For additional commands, e-mail: email@example.com
Received on Tue Jun 5 14:49:59 2007