Code review comment for lp:~jml/testtools/patch-310770

Revision history for this message
Jonathan Lange (jml) wrote :

I don't like that.

We have a really nice monkey-patching module in Twisted and it simply never gets used. The only thing that ends up being used is TestCase.patch.

The delattr thing rarely comes up in actual usage, but I agree that it's a good behaviour to have. Will fix that.

« Back to merge proposal