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
a4d58d26a1
commit
2960308c0c
|
@ -1,5 +1,6 @@
|
||||||
import unittest, StringIO, robotparser
|
import unittest, StringIO, robotparser
|
||||||
from test import test_support
|
from test import test_support
|
||||||
|
from urllib2 import urlopen, HTTPError
|
||||||
|
|
||||||
class RobotTestCase(unittest.TestCase):
|
class RobotTestCase(unittest.TestCase):
|
||||||
def __init__(self, index, parser, url, good, agent):
|
def __init__(self, index, parser, url, good, agent):
|
||||||
|
@ -234,13 +235,27 @@ class NetworkTestCase(unittest.TestCase):
|
||||||
test_support.requires('network')
|
test_support.requires('network')
|
||||||
with test_support.transient_internet('mueblesmoraleda.com'):
|
with test_support.transient_internet('mueblesmoraleda.com'):
|
||||||
url = 'http://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 = robotparser.RobotFileParser()
|
parser = robotparser.RobotFileParser()
|
||||||
parser.set_url(url)
|
parser.set_url(url)
|
||||||
try:
|
try:
|
||||||
parser.read()
|
parser.read()
|
||||||
except IOError:
|
except IOError:
|
||||||
self.skipTest('%s is unavailable' % url)
|
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):
|
def testPythonOrg(self):
|
||||||
test_support.requires('network')
|
test_support.requires('network')
|
||||||
|
|
|
@ -92,6 +92,10 @@ Build
|
||||||
Tests
|
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
|
- Avoid failing in test_urllibnet.test_bad_address when some overzealous
|
||||||
DNS service (e.g. OpenDNS) resolves a non-existent domain name. The test
|
DNS service (e.g. OpenDNS) resolves a non-existent domain name. The test
|
||||||
is now skipped instead.
|
is now skipped instead.
|
||||||
|
|
Loading…
Reference in New Issue