R. David Murray <rdmur...@bitdance.com> added the comment:

I think the best we can do here is add a message explaining that the error may 
be due to a broken DNS server (one with a wildcard dns record for all 
non-existent top level domains).  However, assertRaises, even in context 
manager form, doesn't take a msg argument (yet).  I've opened an issue with a 
feature request to fix that and made it a dependency of this issue.

Note that the test uses a domain name ending in ".d", and for a while before 
that used '.invalid', so the test should not fail if the ISP is only capturing 
valid top level domains with wildcards, something that seems to be far more 
common than catching invalid domains.

----------
dependencies: +assertRaises as a context manager should accept a 'msg' keyword 
argument.
nosy: +r.david.murray
priority: normal -> low
stage: unit test needed -> needs patch
versions: +Python 3.3 -Python 2.6, Python 3.1

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue3583>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to