[issue3602] Move test.test_suport.catch_warning() to the 'warnings' module

Benjamin Peterson report at bugs.python.org
Wed Aug 20 23:53:46 CEST 2008


Benjamin Peterson <musiccomposition at gmail.com> added the comment:

On Wed, Aug 20, 2008 at 4:51 PM, Brett Cannon <report at bugs.python.org> wrote:
>
> Brett Cannon <brett at python.org> added the comment:
>
> On Wed, Aug 20, 2008 at 10:29 AM, Benjamin Peterson
> <report at bugs.python.org> wrote:
>>
>> Benjamin Peterson <musiccomposition at gmail.com> added the comment:
>>
>> I don't think the using argument should live in warnings; it's too test
>> specific. I recommend we still keep a catch_warning in test_support (or
>> even just test_warnings since this seems to be the only use-case).
>>
>
> But decoupling from the core code of the context manager for this is
> not straight-forward without mucking around in sys.modules and that is
> always a risky thing to do.

Why would you have to much around in sys.modules?
>
> _______________________________________
> Python tracker <report at bugs.python.org>
> <http://bugs.python.org/issue3602>
> _______________________________________
>

-- 
Cheers,
Benjamin Peterson
"There's no place like 127.0.0.1."

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue3602>
_______________________________________


More information about the Python-bugs-list mailing list