mirror of
https://github.com/python/cpython.git
synced 2024-11-28 04:15:11 +08:00
Avoid failing in test_robotparser when mueblesmoraleda.com is flaky and
an overzealous DNS service (e.g. OpenDNS) redirects to a placeholder Web site.
This commit is contained in:
parent
72fff046a6
commit
95531ea2f1
@ -1,7 +1,8 @@
|
||||
import io
|
||||
import unittest
|
||||
import urllib.robotparser
|
||||
from urllib.error import URLError
|
||||
from urllib.error import URLError, HTTPError
|
||||
from urllib.request import urlopen
|
||||
from test import support
|
||||
|
||||
class RobotTestCase(unittest.TestCase):
|
||||
@ -237,13 +238,27 @@ class NetworkTestCase(unittest.TestCase):
|
||||
support.requires('network')
|
||||
with support.transient_internet('mueblesmoraleda.com'):
|
||||
url = 'http://mueblesmoraleda.com'
|
||||
robots_url = url + "/robots.txt"
|
||||
# First check the URL is usable for our purposes, since the
|
||||
# test site is a bit flaky.
|
||||
try:
|
||||
urlopen(robots_url)
|
||||
except HTTPError as e:
|
||||
if e.code not in {401, 403}:
|
||||
self.skipTest(
|
||||
"%r should return a 401 or 403 HTTP error, not %r"
|
||||
% (robots_url, e.code))
|
||||
else:
|
||||
self.skipTest(
|
||||
"%r should return a 401 or 403 HTTP error, not succeed"
|
||||
% (robots_url))
|
||||
parser = urllib.robotparser.RobotFileParser()
|
||||
parser.set_url(url)
|
||||
try:
|
||||
parser.read()
|
||||
except URLError:
|
||||
self.skipTest('%s is unavailable' % url)
|
||||
self.assertEqual(parser.can_fetch("*", url+"/robots.txt"), False)
|
||||
self.assertEqual(parser.can_fetch("*", robots_url), False)
|
||||
|
||||
def testPythonOrg(self):
|
||||
support.requires('network')
|
||||
|
@ -47,6 +47,10 @@ C-API
|
||||
Tests
|
||||
-----
|
||||
|
||||
- Avoid failing in test_robotparser when mueblesmoraleda.com is flaky and
|
||||
an overzealous DNS service (e.g. OpenDNS) redirects to a placeholder
|
||||
Web site.
|
||||
|
||||
- Avoid failing in test_urllibnet.test_bad_address when some overzealous
|
||||
DNS service (e.g. OpenDNS) resolves a non-existent domain name. The test
|
||||
is now skipped instead.
|
||||
|
Loading…
Reference in New Issue
Block a user