<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 4 July 2016 at 12:27, Craig Rodrigues <span dir="ltr"><<a href="mailto:rodrigc@crodrigues.org" target="_blank">rodrigc@crodrigues.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class="">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></span><div class="gmail_extra"><div class="gmail_quote"><span class=""><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></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></span><div>In <span><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>making changes to test_dist3.py will not help.<br></span> <span><br></span></div></div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><span></span></div><div><span>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" target="_blank">https://github.com/twisted/twisted/pull/288</a> and that<br></span></div><div><span>fixed the coverage problem.<br><br></span></div></div></div></div></blockquote><div><br></div><div>I think that dist.py vs dist3.py was made on purpose as 2 different file.</div><div>That is, you should not have any `if _PY3` code in any of those files.</div><div><br></div><div>But I might be wrong :)</div><div><br></div><div>-- <br></div></div><div class="gmail_signature" data-smartmail="gmail_signature">Adi Roiban</div>
</div></div>