[patch] new merge tests suggested by Aaron

duchier at ps.uni-sb.de duchier at ps.uni-sb.de
Wed Dec 21 16:13:40 GMT 2005


John Arbash Meinel <john at arbash-meinel.com> writes:

> I think you got the diff backwards :)

rats! non-commuting arguments...

> I have to reverse my request. I thought they came with more description
> about the operation, not just a set of shell commands.

they did: that's what got trimmed down to produce the first line.  do
you want that back?

> Would it be okay if I just change the name of the test? So you have:
>
>    def test_rename_before_create(self):

Yes.  I should have thought of that myself.

> Though I might leave a single comment line in for:
>
>    def test_create_before_rename(self):
> 	# create before rename, target parents before children

So you mean to rename the tests and then drop the docstring
altogether?

--Denys






More information about the bazaar mailing list