[merge] test_transport needs to provide get_test_permutations

Martin Pool mbp at sourcefrog.net
Sat May 5 02:37:15 BST 2007

On 5/5/07, John Arbash Meinel <john at arbash-meinel.com> wrote:
> John Arbash Meinel has voted +1 (conditional).
> Status is now: Conditionally approved
> Comment:
> Are we sure that we want all transport modules to provide
> test_permutations?

I'm not sure.  At least in my other patch intended to merge to 0.16 i
should change it back to a warning.

But given the problem I previously observed of many repetitions of
this warning, I wanted to check that we had fixed every case.  I don't
think requiring transports to declare how they are tested is an
unreasonable burden (the stub is just two lines).

I should probably check *why* this was being called so many times...
Maybe it's just once per transport test or something.


More information about the bazaar mailing list