Code review comment for lp:~mbp/bzr/doc

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

On 5 August 2011 23:16, John Arbash Meinel <email address hidden> wrote:
>> Another test that these actually pass would be welcome.
>
> I agree that adding something would be nice. But needing root access to
> test it seems risky. Not to mention having it accidentally set the
> system to add 500ms delay on the loopback could be really terrible for pqm.

Oh, I meant a human reviewer interactively running the tests. I agree
adding an automated test would be difficult and not especially useful.
 It would be better to actually use them from usertest.

Aaron added a script based on this to Launchpad.

  merge approve

m

review: Approve

« Back to merge proposal