<div dir="ltr">Hi,<div><br></div><div>What is the best practice for deprecating a class in Twisted and writing tests for it ... and updating existing tests.</div><div><br></div><div>I think that the part in which the deprecation code is written, is well understood and documented here</div><div><br></div><div><a href="http://twistedmatrix.com/documents/current/core/development/policy/compatibility-policy.html#classes">http://twistedmatrix.com/documents/current/core/development/policy/compatibility-policy.html#classes</a></div><div><br></div><div>What is missing, is the part talking about how to test these changes and how to update existing tests.<br clear="all"><div><br></div><div>The compatibility / deprecation documentation include a section about how to test deprecation code, but the section is brief.</div><div><br></div><div><a href="http://twistedmatrix.com/documents/current/core/development/policy/compatibility-policy.html#testing-deprecation-code">http://twistedmatrix.com/documents/current/core/development/policy/compatibility-policy.html#testing-deprecation-code</a><br></div><div><br></div><div>----</div><div><br></div><div>This email is a follow up of a review done for this ticket <a href="https://twistedmatrix.com/trac/ticket/8368">https://twistedmatrix.com/trac/ticket/8368</a> </div><div><br></div><div>Please also check the comments on that ticket.</div><div><br></div><div>Please leave your feedback and we can try to apply and document it as part of the work for ticket #8368</div><div><br></div><div>Regards,</div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Adi Roiban</div>

</div></div>