Remove obsolete "noinspection" comments

mediacloud · Jul 17, 2019 · a5cd546 · a5cd546
1 parent 4182e59
commit a5cd546
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 13 deletions.
diff --git a/tests/test_tree.py b/tests/test_tree.py
@@ -276,7 +276,6 @@ def test_sitemap_tree_for_homepage(self):
                 text="<h1>404 Not Found!</h1>",
             )
 
-            # noinspection PyArgumentList
             expected_sitemap_tree = IndexWebsiteSitemap(
                 url='{}/'.format(self.TEST_BASE_URL),
                 sub_sitemaps=[
@@ -979,7 +978,6 @@ def test_sitemap_tree_for_homepage_no_sitemap(self):
                 """.format(base_url=self.TEST_BASE_URL)).strip(),
             )
 
-            # noinspection PyArgumentList
             expected_sitemap_tree = IndexWebsiteSitemap(
                 url='{}/'.format(self.TEST_BASE_URL),
                 sub_sitemaps=[
@@ -1103,7 +1101,6 @@ def test_sitemap_tree_for_homepage_robots_txt_no_content_type(self):
                 """.format(base_url=self.TEST_BASE_URL)).strip(),
             )
 
-            # noinspection PyArgumentList
             expected_sitemap_tree = IndexWebsiteSitemap(
                 url='{}/'.format(self.TEST_BASE_URL),
                 sub_sitemaps=[
@@ -1138,7 +1135,6 @@ def test_sitemap_tree_for_homepage_no_robots_txt(self):
                 text="<h1>404 Not Found!</h1>",
             )
 
-            # noinspection PyArgumentList
             expected_sitemap_tree = IndexWebsiteSitemap(
                 url='{}/'.format(self.TEST_BASE_URL),
                 sub_sitemaps=[

diff --git a/usp/fetch_parse.py b/usp/fetch_parse.py
@@ -77,7 +77,6 @@ def sitemap(self) -> AbstractSitemap:
         log.info("Fetching level {} sitemap from {}...".format(self._recursion_level, self._url))
         response = get_url_retry_on_client_errors(url=self._url, web_client=self._web_client)
         if not response.is_success():
-            # noinspection PyArgumentList
             return InvalidSitemap(
                 url=self._url,
                 reason="Unable to fetch sitemap from {}: {} {}".format(
@@ -178,7 +177,6 @@ def sitemap(self) -> AbstractSitemap:
             fetched_sitemap = fetcher.sitemap()
             sub_sitemaps.append(fetched_sitemap)
 
-        # noinspection PyArgumentList
         index_sitemap = IndexRobotsTxtSitemap(url=self._url, sub_sitemaps=sub_sitemaps)
 
         return index_sitemap
@@ -205,7 +203,6 @@ def sitemap(self) -> AbstractSitemap:
             page = SitemapPage(url=page_url)
             pages.append(page)
 
-        # noinspection PyArgumentList
         text_sitemap = PagesTextSitemap(url=self._url, pages=pages)
 
         return text_sitemap
@@ -242,7 +239,6 @@ def sitemap(self) -> AbstractSitemap:
             log.error("Parsing sitemap from URL {} failed: {}".format(self._url, ex))
 
         if not self._concrete_parser:
-            # noinspection PyArgumentList
             return InvalidSitemap(
                 url=self._url,
                 reason="No parsers support sitemap from {}".format(self._url),
@@ -423,15 +419,13 @@ def sitemap(self) -> AbstractSitemap:
                                          web_client=self._web_client)
                 fetched_sitemap = fetcher.sitemap()
             except Exception as ex:
-                # noinspection PyArgumentList
                 fetched_sitemap = InvalidSitemap(
                     url=sub_sitemap_url,
                     reason="Unable to add sub-sitemap from URL {}: {}".format(sub_sitemap_url, str(ex)),
                 )
 
             sub_sitemaps.append(fetched_sitemap)
 
-        # noinspection PyArgumentList
         index_sitemap = IndexXMLSitemap(url=self._url, sub_sitemaps=sub_sitemaps)
 
         return index_sitemap
@@ -639,7 +633,6 @@ def sitemap(self) -> AbstractSitemap:
             if page:
                 pages.append(page)
 
-        # noinspection PyArgumentList
         pages_sitemap = PagesXMLSitemap(url=self._url, pages=pages)
 
         return pages_sitemap
@@ -755,7 +748,6 @@ def sitemap(self) -> AbstractSitemap:
             if page:
                 pages.append(page)
 
-        # noinspection PyArgumentList
         pages_sitemap = PagesRSSSitemap(url=self._url, pages=pages)
 
         return pages_sitemap
@@ -888,7 +880,6 @@ def sitemap(self) -> AbstractSitemap:
             if page:
                 pages.append(page)
 
-        # noinspection PyArgumentList
         pages_sitemap = PagesAtomSitemap(url=self._url, pages=pages)
 
         return pages_sitemap