John Arbash Meinel has voted +1.
Status is now: Approved
Comment:
Looks fine to me. Though the test you are modifying looks like it could really use some cleanup. For example, we should be using tree.rename_one() rather than run_bzr('mv x y') sort of things.
For details, see: http://bundlebuggy.aaronbentley.com/request/%3C4575F09A.1080108%40utoronto.ca%3E