[MERGE][1.2?][Bug #137823] Don't blindly raise UnavailableFeature during test setup

John Arbash Meinel john at arbash-meinel.com
Sun Feb 10 19:40:46 GMT 2008


Vincent Ladeuil wrote:
> Hi,
> 
> This bug was nasty, see details on https://bugs.edge.launchpad.net/bzr/+bug/137823
> 
> The risk of broking anything is sufficiently low that this may be
> included in 1.2 if time permits.
> 
> Also note that this was caused by an unusual way to check the
> availability of the feature, it needed to be done in
> transport.ftp.get_test_permutations() with no access to the test
> itself (hence no access to neither _test_needs_features nor
> requireFeature()).
> 
>         Vincent

Why not change it so that raising an exception during setUp still calls tearDown?

John
=:->




More information about the bazaar mailing list