[MERGE] selftest fails if the committer name contains non-ASCII characters

Martin Pool mbp at canonical.com
Wed Aug 1 20:04:48 BST 2007


Martin Pool has voted resubmit.
Status is now: Resubmit
Comment:
Thanks for the patch, this does seem to be a test bug.

I guess you're seeing this because you have a non-ascii name yourself. 
In general we don't want tests to pass for some people and fail for 
others, so I think what we actually want to do is run all these tests 
using a non-ascii committer.

+        output_encoding = osutils.get_terminal_encoding()

Also, we probably don't want the tests to depend on the encoding of the 
person running the test.  So, rather, we should print most of them in 
UTF-8, I think.

+    def get_fake_stdio(self, encoding_type=None):

Probably wants a docstring.  Actually, there's nothing about stdio in 
this method, so it could just be called make_encoding_stringio().  I 
think John had some thoughts about whether we should be doing this kind 
of conversion in tests.

Also you should add a NEWS entry under 'Testing' and credit yourself 
there.

For details, see: 
http://bundlebuggy.aaronbentley.com/request/%3C1185960872.6348.24.camel%40nemo%3E



More information about the bazaar mailing list