That is completely fine.  I was just worried that you found that you
actually needed them to get the test suite to run.  That would have
implied something weird going on with the test suite.  I think we are
fine then on this one.<br>
<br>
Brian<br><br><div class="gmail_quote">On Tue, Aug 4, 2009 at 12:20 PM, Fernando Perez <span dir="ltr">&lt;<a href="http://fperez.net">fperez.net</a>@<a href="http://gmail.com">gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div></div><div class="h5">On Tue, Aug 4, 2009 at 11:17 AM, Brian Granger&lt;<a href="http://ellisonbg.net" target="_blank">ellisonbg.net</a>@<a href="http://gmail.com" target="_blank">gmail.com</a>&gt; wrote:<br>
&gt; In one of the recent commits to trunk:<br>
&gt;<br>
&gt; <a href="http://bazaar.launchpad.net/%7Eipython-dev/ipython/trunk/revision/1196" target="_blank">http://bazaar.launchpad.net/~ipython-dev/ipython/trunk/revision/1196</a><br>
&gt;<br>
&gt; New logic was added in iptest and other places for twisted using modules to<br>
&gt; not be tested if twisted is missing.<br>
&gt;<br>
&gt; BUT, in three modules:<br>
&gt;<br>
&gt; kernel.engineservice<br>
&gt; kernel.error<br>
&gt; kernel.newserialized<br>
&gt;<br>
&gt; Nose specific logic was added that looks like this:<br>
&gt;<br>
&gt; __test__ = {}<br>
&gt;<br>
&gt; But, these tests should *never* be run with nose because of the bad<br>
&gt; interactions between nose and twisteds reactor.  Why was this logic added to<br>
&gt; these modules?<br>
<br>
</div></div>I had seen those before in some of the other twisted files, and in<br>
adding the twisted-skipping logic elsewhere, I must have accidentally<br>
added that in those files, sorry.  It is harmless, since nose will not<br>
see those at all, but it was put in there by accident.  In fact, what<br>
we should do is to remove that logic from *all* twisted-using modules<br>
in your cleanup branch, since now we are sending those to trial<br>
instead of nose.  But I didn&#39;t want to delete anything I wasn&#39;t 100%<br>
sure about this late in the game, and instead I ended up accidentally<br>
adding stuff :)<br>
<br>
I suggest we:<br>
<br>
- leave those as they are for now in 0.10, they are harmless<br>
- remove them in your reorg branch altogether, so they don&#39;t cause<br>
visual/mental clutter.  Twisted&#39;s trial should manage that code<br>
anyway, not nose.<br>
<br>
How does that sound?<br>
<font color="#888888"><br>
f<br>
</font></blockquote></div><br>