mirror of https://github.com/scrapy/scrapy.git
spelling: overridden
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
This commit is contained in:
parent
a839b61147
commit
1a5cf00db7
|
|
@ -98,7 +98,7 @@ class ScrapyHTTPPageGetter(HTTPClient):
|
|||
# This class used to inherit from Twisted’s
|
||||
# twisted.web.client.HTTPClientFactory. When that class was deprecated in
|
||||
# Twisted (https://github.com/twisted/twisted/pull/643), we merged its
|
||||
# non-overriden code into this class.
|
||||
# non-overridden code into this class.
|
||||
class ScrapyHTTPClientFactory(ClientFactory):
|
||||
|
||||
protocol = ScrapyHTTPPageGetter
|
||||
|
|
|
|||
|
|
@ -147,8 +147,8 @@ class ImagesPipeline(FilesPipeline):
|
|||
if self._deprecated_convert_image is None:
|
||||
self._deprecated_convert_image = 'response_body' not in get_func_args(self.convert_image)
|
||||
if self._deprecated_convert_image:
|
||||
warnings.warn(f'{self.__class__.__name__}.convert_image() method overriden in a deprecated way, '
|
||||
'overriden method does not accept response_body argument.',
|
||||
warnings.warn(f'{self.__class__.__name__}.convert_image() method overridden in a deprecated way, '
|
||||
'overridden method does not accept response_body argument.',
|
||||
category=ScrapyDeprecationWarning)
|
||||
|
||||
if self._deprecated_convert_image:
|
||||
|
|
|
|||
|
|
@ -173,8 +173,8 @@ class ImagesPipelineTestCase(unittest.TestCase):
|
|||
self.assertEqual(orig_im.getcolors(), thumb_img.getcolors())
|
||||
self.assertEqual(buf.getvalue(), thumb_buf.getvalue())
|
||||
|
||||
expected_warning_msg = ('.convert_image() method overriden in a deprecated way, '
|
||||
'overriden method does not accept response_body argument.')
|
||||
expected_warning_msg = ('.convert_image() method overridden in a deprecated way, '
|
||||
'overridden method does not accept response_body argument.')
|
||||
self.assertEqual(len([warning for warning in w if expected_warning_msg in str(warning.message)]), 1)
|
||||
|
||||
def test_convert_image_old(self):
|
||||
|
|
|
|||
|
|
@ -11,12 +11,12 @@ from tests.mockserver import MockServer
|
|||
from tests.spiders import SingleRequestSpider
|
||||
|
||||
|
||||
OVERRIDEN_URL = "https://example.org"
|
||||
OVERRIDDEN_URL = "https://example.org"
|
||||
|
||||
|
||||
class ProcessResponseMiddleware:
|
||||
def process_response(self, request, response, spider):
|
||||
return response.replace(request=Request(OVERRIDEN_URL))
|
||||
return response.replace(request=Request(OVERRIDDEN_URL))
|
||||
|
||||
|
||||
class RaiseExceptionRequestMiddleware:
|
||||
|
|
@ -30,7 +30,7 @@ class CatchExceptionOverrideRequestMiddleware:
|
|||
return Response(
|
||||
url="http://localhost/",
|
||||
body=b"Caught " + exception.__class__.__name__.encode("utf-8"),
|
||||
request=Request(OVERRIDEN_URL),
|
||||
request=Request(OVERRIDDEN_URL),
|
||||
)
|
||||
|
||||
|
||||
|
|
@ -52,7 +52,7 @@ class AlternativeCallbacksSpider(SingleRequestSpider):
|
|||
class AlternativeCallbacksMiddleware:
|
||||
def process_response(self, request, response, spider):
|
||||
new_request = request.replace(
|
||||
url=OVERRIDEN_URL,
|
||||
url=OVERRIDDEN_URL,
|
||||
callback=spider.alt_callback,
|
||||
cb_kwargs={"foo": "bar"},
|
||||
)
|
||||
|
|
@ -127,13 +127,13 @@ class CrawlTestCase(TestCase):
|
|||
yield crawler.crawl(seed=url, mockserver=self.mockserver)
|
||||
|
||||
response = crawler.spider.meta["responses"][0]
|
||||
self.assertEqual(response.request.url, OVERRIDEN_URL)
|
||||
self.assertEqual(response.request.url, OVERRIDDEN_URL)
|
||||
|
||||
self.assertEqual(signal_params["response"].url, url)
|
||||
self.assertEqual(signal_params["request"].url, OVERRIDEN_URL)
|
||||
self.assertEqual(signal_params["request"].url, OVERRIDDEN_URL)
|
||||
|
||||
log.check_present(
|
||||
("scrapy.core.engine", "DEBUG", f"Crawled (200) <GET {OVERRIDEN_URL}> (referer: None)"),
|
||||
("scrapy.core.engine", "DEBUG", f"Crawled (200) <GET {OVERRIDDEN_URL}> (referer: None)"),
|
||||
)
|
||||
|
||||
@defer.inlineCallbacks
|
||||
|
|
@ -154,7 +154,7 @@ class CrawlTestCase(TestCase):
|
|||
yield crawler.crawl(seed=url, mockserver=self.mockserver)
|
||||
response = crawler.spider.meta["responses"][0]
|
||||
self.assertEqual(response.body, b"Caught ZeroDivisionError")
|
||||
self.assertEqual(response.request.url, OVERRIDEN_URL)
|
||||
self.assertEqual(response.request.url, OVERRIDDEN_URL)
|
||||
|
||||
@defer.inlineCallbacks
|
||||
def test_downloader_middleware_do_not_override_in_process_exception(self):
|
||||
|
|
|
|||
Loading…
Reference in New Issue