[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.
--
Martin
More information about the bazaar
mailing list