[issue5354] Add test.support.import_python_only

[Brett Cannon]

Brett Cannon <br...@python.org> added the comment:

On Sat, Apr 11, 2009 at 07:28, Nick Coghlan <rep...@bugs.python.org> wrote:

>
> Nick Coghlan <ncogh...@gmail.com> added the comment:
>
> Implemented for 2.7 in r71465
> Implemented for 3.1 in r7146
>

Style nit: why the extra check for block_names being None to later set it to
an empty tuple? Why not make the default argument the empty tuple?

>
> Leaving issue open for the moment - the 3.1 test_warnings fails if I
> don't get a fresh copy of _warnings before running the unit tests. I
> want to figure out why that is necessary before closing the issue.

Might have to do with _warnings trying to import warnings for certain things
in certain cases.

And just something I thought about, Nick, is possibly coming up with a class
decorator that sets self.module on the class. This could have the perk of
taking an optional argument that checks if the optimized module is even in
existence so as to skip running the tests twice if it is not needed. Then
again this might over-optimizing and best to just let the tests run twice
even if it is not needed.

----------
Added file: http://bugs.python.org/file13675/unnamed

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue5354>
_______________________________________
<br><br><div class="gmail_quote">On Sat, Apr 11, 2009 at 07:28, Nick Coghlan 
<span dir="ltr">&lt;<a 
href="mailto:rep...@bugs.python.org";>rep...@bugs.python.org</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;">

<br>
Nick Coghlan &lt;<a href="mailto:ncogh...@gmail.com";>ncogh...@gmail.com</a>&gt; 
added the comment:<br>
<br>
Implemented for 2.7 in r71465<br>
Implemented for 3.1 in r7146<br>
</blockquote><div><br>Style nit: why the extra check for block_names being None 
to later set it to an empty tuple? Why not make the default argument the empty 
tuple?<br> </div><blockquote class="gmail_quote" style="border-left: 1px solid 
rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<br>
Leaving issue open for the moment - the 3.1 test_warnings fails if I<br>
don&#39;t get a fresh copy of _warnings before running the unit tests. I<br>
want to figure out why that is necessary before closing the 
issue.</blockquote><div><br>Might have to do with _warnings trying to import 
warnings for certain things in certain cases.<br><br>And just something I 
thought about, Nick, is possibly coming up with a class decorator that sets 
self.module on the class. This could have the perk of taking an optional 
argument that checks if the optimized module is even in existence so as to skip 
running the tests twice if it is not needed. Then again this might 
over-optimizing and best to just let the tests run twice even if it is not 
needed.<br>

</div></div>

_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com