Hi, I submitted a bundle incorporating your (Robert and Martins) requests. <div>On 6/22/07, Robert Collins <<a href="mailto:robertc@robertcollins.net"> robertc@robertcollins.net</a>> wrote: ... <blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">In particular : ("bzr rm file with spaces") cannot be correctly processed with a trivial string splitting helper inside the test suite. To avoid confusing readers - to optimise for folk reading the code - we should make it very clear when splitting is and is not happening. For these reasons we chose a couple of years back to make the recommended method be a list of strings, no magic involved.</blockquote><div> I find it annoying to quote every single parameter for black box tests. I think it should ALWAYS be splitting. In the above case, the test should rather be "bzr rm file\ with\ spaces" or "bzr rm 'file with spaces'" or 'bzr rm "file with spaces"' which my `run_bzr_captured` is catering for since it uses shlex, this is constent with how it is used from the commandline. My `run_bzr_captured` only splits if you give it a single string, so it doesn't break existing tests. Thats just what I think. regards marius   </div></div> -- <a href="http://bazaar-vcs.org">bazaar-vcs.org</a> Because I don't trust Version Control Systems with less than 6350 unit tests.