summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAntoine Pitrou <solipsis@pitrou.net>2014-09-16 22:40:13 (GMT)
committerAntoine Pitrou <solipsis@pitrou.net>2014-09-16 22:40:13 (GMT)
commit6cfa46e45f1bffa593c8f272a4e035c80d333e14 (patch)
tree1e0878372b8eb43622c545abfcf25bf76a532338
parent0d548873262d1e61f6089eccf43e261b259ec96e (diff)
parentbc2c4c9990f6179d878688b04d712ecf75e17e1d (diff)
downloadcpython-6cfa46e45f1bffa593c8f272a4e035c80d333e14.zip
cpython-6cfa46e45f1bffa593c8f272a4e035c80d333e14.tar.gz
cpython-6cfa46e45f1bffa593c8f272a4e035c80d333e14.tar.bz2
Replace bad ftp URLs in test_urllib2net
-rw-r--r--Lib/test/test_urllib2net.py68
1 files changed, 34 insertions, 34 deletions
diff --git a/Lib/test/test_urllib2net.py b/Lib/test/test_urllib2net.py
index d80e39a..2662756 100644
--- a/Lib/test/test_urllib2net.py
+++ b/Lib/test/test_urllib2net.py
@@ -103,9 +103,9 @@ class OtherNetworkTests(unittest.TestCase):
def test_ftp(self):
urls = [
- 'ftp://ftp.kernel.org/pub/linux/kernel/README',
- 'ftp://ftp.kernel.org/pub/linux/kernel/non-existent-file',
- #'ftp://ftp.kernel.org/pub/leenox/kernel/test',
+ 'ftp://ftp.debian.org/debian/README',
+ ('ftp://ftp.debian.org/debian/non-existent-file',
+ None, urllib.error.URLError),
'ftp://gatekeeper.research.compaq.com/pub/DEC/SRC'
'/research-reports/00README-Legal-Rules-Regs',
]
@@ -215,39 +215,39 @@ class OtherNetworkTests(unittest.TestCase):
urlopen = _wrap_with_retry_thrice(urlopen, urllib.error.URLError)
for url in urls:
- if isinstance(url, tuple):
- url, req, expected_err = url
- else:
- req = expected_err = None
-
- with support.transient_internet(url):
- debug(url)
- try:
- f = urlopen(url, req, TIMEOUT)
- except OSError as err:
- debug(err)
- if expected_err:
- msg = ("Didn't get expected error(s) %s for %s %s, got %s: %s" %
- (expected_err, url, req, type(err), err))
- self.assertIsInstance(err, expected_err, msg)
- except urllib.error.URLError as err:
- if isinstance(err[0], socket.timeout):
- print("<timeout: %s>" % url, file=sys.stderr)
- continue
- else:
- raise
+ with self.subTest(url=url):
+ if isinstance(url, tuple):
+ url, req, expected_err = url
else:
+ req = expected_err = None
+
+ with support.transient_internet(url):
try:
- with support.time_out, \
- support.socket_peer_reset, \
- support.ioerror_peer_reset:
- buf = f.read()
- debug("read %d bytes" % len(buf))
- except socket.timeout:
- print("<timeout: %s>" % url, file=sys.stderr)
- f.close()
- debug("******** next url coming up...")
- time.sleep(0.1)
+ f = urlopen(url, req, TIMEOUT)
+ except OSError as err:
+ if expected_err:
+ msg = ("Didn't get expected error(s) %s for %s %s, got %s: %s" %
+ (expected_err, url, req, type(err), err))
+ self.assertIsInstance(err, expected_err, msg)
+ else:
+ raise
+ except urllib.error.URLError as err:
+ if isinstance(err[0], socket.timeout):
+ print("<timeout: %s>" % url, file=sys.stderr)
+ continue
+ else:
+ raise
+ else:
+ try:
+ with support.time_out, \
+ support.socket_peer_reset, \
+ support.ioerror_peer_reset:
+ buf = f.read()
+ debug("read %d bytes" % len(buf))
+ except socket.timeout:
+ print("<timeout: %s>" % url, file=sys.stderr)
+ f.close()
+ time.sleep(0.1)
def _extra_handlers(self):
handlers = []