[MERGE][bug #48623]

[Martin Pool]

Martin Pool has voted +0.
Status is now: Waiting
Comment:
I reviewed an updated version on irc, which included tests, and gave 
some feedback:

<poolie> vezult: normally rather than runbzr('init') you'd do 
self.make_branch_and_tree('.')
<vezult> poolie: what is difference between the two? one calls bzr, and 
the other uses bzrlib?
<poolie> that's right
  it's a little cleaner and faster
  not a big deal in this case but generally we want to use the api not 
the command line for everything in the tests except actually testing the 
command line
  vezult: i would think line 18ff should be outside the finally block?
  we shouldn't run them if there's an error
  otherwise +1
  um
  also i'd be inclined to have this only happen with 'ignore -v'
  maybe not

For details, see: 
http://bundlebuggy.aaronbentley.com/request/%3C1178058897.4168.4.camel%40zepto.home.zettazebra.com%3E