attempt to make tests pass

Author: chrisirhc

Diff for: python/private/pypi/parse_requirements.bzl

@@ -215,16 +215,22 @@ def parse_requirements(
                 logger = logger,
             )
 
+            target_platforms = env_marker_target_platforms.get(r.requirement_line, r.target_platforms)
+
             if sdist:
                 sha = sdist.sha256
                 if sha not in sdists_by_sha:
                     sdists_by_sha[sha] = struct(
+                        distribution = r.distribution,
+                        srcs = r.srcs,
+                        extra_pip_args = r.extra_pip_args,
                         sdist = sdist,
                         platforms = [],
                     )
-                sdists_by_sha[sha].platforms.extend(r.target_platforms)
-
-            target_platforms = env_marker_target_platforms.get(r.requirement_line, r.target_platforms)
+                sdists_by_sha[sha].platforms.extend(target_platforms)
+            
+                if len(whls) == 0:
+                    continue
 
             ret_requirements.append(
                 struct(
@@ -241,10 +247,10 @@ def parse_requirements(
         for sha, sdist_info in sdists_by_sha.items():
             ret_requirements.append(
                 struct(
-                    distribution = r.distribution,
-                    srcs = r.srcs,
+                    distribution = sdist_info.distribution,
+                    srcs = sdist_info.srcs,
                     target_platforms = sorted(sdist_info.platforms),
-                    extra_pip_args = r.extra_pip_args,
+                    extra_pip_args = sdist_info.extra_pip_args,
                     whls = [],
                     sdist = sdist_info.sdist,
                     is_exposed = is_exposed,

Diff for: tests/pypi/parse_requirements/parse_requirements_tests.bzl

@@ -679,6 +679,7 @@ def _test_sdist_different_hashes(env):
                     requirement = "foo==0.0.1",
                     requirement_line = "foo==0.0.1 --hash=sha256:deadbaaf --hash=sha256:cafebabe",
                     shas = ["cafebabe", "deadbaaf"],
+                    url = "",
                     version = "0.0.1",
                 ),
                 target_platforms = ["cp315_linux_x86_64"],
@@ -701,6 +702,7 @@ def _test_sdist_different_hashes(env):
                     requirement = "foo==0.0.1",
                     requirement_line = "foo==0.0.1 --hash=sha256:deadbeef --hash=sha256:cafebabe",
                     shas = ["cafebabe", "deadbeef"],
+                    url = "",
                     version = "0.0.1",
                 ),
                 target_platforms = ["cp315_windows_x86_64"],
@@ -726,8 +728,9 @@ def _test_sdist_different_hashes(env):
                 srcs = struct(
                     marker = "",
                     requirement = "foo==0.0.1",
-                    requirement_line = "foo==0.0.1 --hash=sha256:deadbeef --hash=sha256:cafebabe",
-                    shas = ["cafebabe", "deadbeef"],
+                    requirement_line = "foo==0.0.1 --hash=sha256:deadbaaf --hash=sha256:cafebabe",
+                    shas = ["cafebabe", "deadbaaf"],
+                    url = "",
                     version = "0.0.1",
                 ),
                 target_platforms = ["cp315_linux_x86_64", "cp315_windows_x86_64"],