<div dir="ltr">On Sun, Jul 3, 2016 at 11:00 PM, Adi Roiban <span dir="ltr"><<a href="mailto:adi@roiban.ro" target="_blank">adi@roiban.ro</a>></span> wrote: <br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote"><span class=""></span><div>Only the modules (including the test modules) from dist3.py are executed on Python3</div><div><a href="https://github.com/twisted/twisted/blob/trunk/twisted/python/dist3.py" target="_blank">https://github.com/twisted/twisted/blob/trunk/twisted/python/dist3.py</a><br></div><div><br></div><div>In the header of that file you can see</div><div><br></div><div># -*- test-case-name: twisted.python.test.test_dist3 -*-<br></div><div><br></div><div>and also in the test modules we have</div><div><br></div><div>twisted.python.test.test_dist3<br></div><div><br></div><div>---------------</div><div><br></div><div>That is you need to make the changes in that module</div><div><br></div><div>twisted.python.test.test_dist is not executed on Python3 ... hence its coverage is not reported on Python3<br></div><br></div></div></div></blockquote><div><br><br><br></div><div>In <span class="im"><a href="https://github.com/twisted/twisted/pull/224" target="_blank">https://github.com/twisted/twisted/pull/224</a> , I made no changes to twisted/python/dist3.py, so<br></span></div><div><span class="im">making changes to test_dist3.py will not help.<br><br></span></div><div><span class="im">I added twisted.python.test.test_dist to the list of tests executed on Python 3 in <a href="https://github.com/twisted/twisted/pull/288">https://github.com/twisted/twisted/pull/288</a> and that<br></span></div><div><span class="im">fixed the coverage problem.<br><br></span></div><div><span class="im">I still think that setup.py should be invoked under coverage (I assume in tox.ini)  in the coverage build,<br></span></div><div><span class="im">without writing a separate test for setup.py<br><br></span></div><div><span class="im">--<br></span></div><div><span class="im">Craig<br></span></div><div><br><br><br> </div></div></div></div>