Code review comment for lp:~vila/bzr/430749-no-extensions-warning

Revision history for this message
Martin Pool (mbp) wrote :

2009/9/18 fullermd <email address hidden>:
>> I chose to modify the tests that were expecting a clean stderr (or
>> some expected stderr) because they were only a few of them.
>
> This makes me a little nervous, in that it adds a new undocumented
> rule about what you need to do if you care about stderr when writing a
> test (and since pretty much anybody writing a test probably HAS
> extensions, they'll never notice it either).

It does seem a bit hacky.

--
Martin <http://launchpad.net/~mbp/>

« Back to merge proposal