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

Revision history for this message
Simon Poirier (simpoir) 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.

review: Approve

« Back to merge proposal