Transport w/ extra tests

Martin Pool mbp at sourcefrog.net
Mon Jul 18 21:35:19 BST 2005


On Mon, 2005-07-18 at 15:05 -0500, John A Meinel wrote:
> On my branch of bzr which uses a Transport hierarchy to support multiple
> protocols, I've added a test suite for people attempting to implement a
> RW Transport. I've put the LocalTransport class through it.
> 
> http://bzr.arbash-meinel.com/bzr-split-storage/

Should I merge this in?

> Just as a brief reminder, my Transport work allows read/write access to
> control directories. But not Working trees. Partially because we haven't
> decided if we want to support remote working trees.
> 
> And in testing, I found that
> bzr branch -r 100 rsync://bazaar-ng.org::bazaar-ng/bzr/bzr.dev
> takes only 30s to complete, while
> bzr branch -r 100 http://bazaar-ng.org/bzr/bzr.dev
> takes 1m51s
> The non-Transport version of bzr also takes 1m50s to download over http.
> Neither one of these is pipelined, but I believe pipelining my
> HttpTransport should be fairly easy, since I already have the
> "get_multi" function call. (Part of the improvements in my Transport
> branch is to change most loops over files into a single large request,
> and then looping)

Well done.

> PS> It would be nice if we had a way to have "bzr selftest" only run
> certain tests. So that when you are doing some development, you can only
> run the tests you care about, until you get them working, and then go
> back and run everything.

Yes, I agree.  It'd also be nice to have a way to skip tests -- I think
by just raising a TestSkipped exception, which would be reported
differently to a regular test failure.

-- 
Martin






More information about the bazaar mailing list