Fix linting issues

Author: sepehr-rs

src/pip/_internal/index/collector.py

@@ -231,8 +231,9 @@ def parse_links(page: IndexContent) -> Iterable[Link]:
     if content_type_l.startswith("application/vnd.pypi.simple.v1+json"):
         data = json.loads(page.content)
         project_track_urls = set(data.get("meta", {}).get("tracks", []))
+        # repo_alt_urls = set(data.get("alternate-locations", []))
+        # repo_alt_urls.add(page.url)
         repo_alt_urls = set(data.get("alternate-locations", []))
-        repo_alt_urls.add(page.url)
         for file in data.get("files", []):
             link = Link.from_json(
                 file,
@@ -251,8 +252,9 @@ def parse_links(page: IndexContent) -> Iterable[Link]:
 
     base_url = parser.base_url or url
     for anchor in parser.anchors:
+        # repo_alt_urls = parser.repo_alt_urls or set()
+        # repo_alt_urls.add(page.url)
         repo_alt_urls = parser.repo_alt_urls or set()
-        repo_alt_urls.add(page.url)
         link = Link.from_element(
             anchor,
             page_url=url,

src/pip/_internal/index/package_finder.py

@@ -1276,7 +1276,7 @@ def check_multiple_remote_repositories(
     # between all locations where that project is
     # found as to what the alternate
     # locations are.
-    # Only validate alternate locations if any 
+    # Only validate alternate locations if any
     # candidate actually declares alternate locations.
     all_explicit_alt_urls = [
         rc.alternate_location_urls - ({rc.url} if rc.url else set())
@@ -1344,7 +1344,9 @@ def check_multiple_remote_repositories(
             single_origin = len({rc.url for rc in remote_candidates}) == 1
 
             if not single_origin:
-                if track_repo not in {_repo_base(u) for u in known_owner_repo_urls}:
+                if track_repo not in {
+                    _repo_base(u) for u in known_owner_repo_urls if u is not None
+                }:
                     raise InvalidTracksUrl(
                         package=project_name,
                         remote_repositories=(

src/pip/_internal/models/candidate.py

@@ -85,8 +85,8 @@ def project_track_urls(self) -> set[str]:
     def alternate_location_urls(self) -> set[str]:
         """Remote repository urls from Alternate Locations metadata."""
         urls: set[str] = set()
-        if self.url:
-            urls.add(self.url)
+        # if self.url:
+        #     urls.add(self.url)
         link = self._link
         if link and link.repo_alt_urls:
             urls.update({i for i in link.repo_alt_urls if i})

tests/unit/test_index.py

@@ -1,7 +1,7 @@
 from __future__ import annotations
 
 import logging
-from typing import Optional
+from typing import Set
 
 import pytest
 
@@ -916,11 +916,11 @@ def test_extract_version_from_fragment(
 
 
 def _make_mock_candidate_check_remote_repo(
-    candidate_name: Optional[str] = None,
-    version: Optional[str] = None,
-    comes_from_url: Optional[str] = None,
-    project_track_urls: Optional[Set[str]] = None,
-    repo_alt_urls: Optional[Set[str]] = None,
+    candidate_name: str | None = None,
+    version: str | None = None,
+    comes_from_url: str | None = None,
+    project_track_urls: Set[str] | None = None,
+    repo_alt_urls: Set[str] | None = None,
 ) -> InstallationCandidate:
     if candidate_name is None:
         candidate_name = "mypackage"
@@ -1089,10 +1089,10 @@ def _make_mock_candidate_check_remote_repo(
 )
 def test_check_multiple_remote_repositories(
     caplog, candidates: list[InstallationCandidate], project_name: str, expected
-):
+) -> None:
     caplog.set_level(logging.DEBUG)
     if expected:
         with pytest.raises(expected):
             check_multiple_remote_repositories(candidates, project_name)
     else:
-        assert check_multiple_remote_repositories(candidates, project_name) is None
+        check_multiple_remote_repositories(candidates, project_name)