At this level, in bzrlib.tests rather than bzrlib.tests.blackbox, you really want to be writing assertions based on api calls rather than the commandline ui. Mostly this just confirms the check above, but adding an assertion for the current tree shape does seem reasonable.
I see you've already fixed the other style stuff I mentioned on IRC, thanks!
Thanks for working on this!
I'm still a little uncertain on the core logic here, will look again later.
+ if self.tree. has_filename( self.tree. id2path( parent[ 0])) == False:
Normal style is not to compare against booleans, just use `if not condition` instead.
+ self.assertEqua l('added: \n folder/\nrenamed:\n file => folder/file2\n', bzr("status" )[0])
+ self.run_
At this level, in bzrlib.tests rather than bzrlib. tests.blackbox, you really want to be writing assertions based on api calls rather than the commandline ui. Mostly this just confirms the check above, but adding an assertion for the current tree shape does seem reasonable.
I see you've already fixed the other style stuff I mentioned on IRC, thanks!