[merge][#261315][1.7] RemoteBranch sets up stacking (more) correctly

John Arbash Meinel john at arbash-meinel.com
Sat Sep 6 02:16:49 BST 2008


John Arbash Meinel has voted approve.
Status is now: Approved
Comment:
It would be interesting to put this into the per_repository 
(per_branch?) tests for all formats that support stacking. You would 
probably need to special case Remote*, to set it up behind the scenes, 
and then access it only over Remote*.

It does cause a permutation, though. As you now need to test that 
"Repo.foo()" works when the object is stacked, versus when it isn't 
stacked.

Maybe as an alternative, we just create a StackedBranch format that is 
tested?

Anyway, the patch is fine, it just feels like there should be more 
systematic tests for how things are handled when we have a Stacked 
object.

For details, see: 
http://bundlebuggy.aaronbentley.com/project/bzr/request/%3Ce01316480809050416r72b3f857h4a664a2cc4e157b0%40mail.gmail.com%3E
Project: Bazaar



More information about the bazaar mailing list