Code review comment for lp://qastaging/~benji/landscape-client/mocker-purge-test_reporter.py

Revision history for this message
Benji York (benji) wrote :

> +1
> Nitpick : wouldn't it be safer and closer to mocker's behaviour to
> self.addCleanup(x_patcher.stop) after start()
> instead of at the theoretical end of a test.

+1

Done.

« Back to merge proposal