finish cleaning up the args

Author: aignas

tests/pypi/extension/extension_tests.bzl

@@ -65,20 +65,13 @@ def _parse(
         *,
         hub_name,
         python_version,
-        experimental_index_url = "",
-        requirements_lock = None,
-        **kwargs):
-    return struct(
-        hub_name = hub_name,
-        python_version = python_version,
-        requirements_lock = requirements_lock,
-        # Constants for now
-        # TODO @aignas 2024-10-21: cover with tests
+        _evaluate_markers_srcs = [],
         auth_patterns = {},
         download_only = False,
         enable_implicit_namespace_pkgs = False,
         environment = {},
         envsubst = {},
+        experimental_index_url = "",
         experimental_index_url = experimental_index_url,
         experimental_requirement_cycles = {},
         experimental_target_platforms = [],
@@ -92,10 +85,37 @@ def _parse(
         requirements_by_platform = {},
         requirements_darwin = None,
         requirements_linux = None,
+        requirements_lock = None,
         requirements_windows = None,
         timeout = 42,
         whl_modifications = {},
-        _evaluate_markers_srcs = [],
+        **kwargs):
+    return struct(
+        _evaluate_markers_srcs = _evaluate_markers_srcs,
+        auth_patterns = auth_patterns,
+        download_only = download_only,
+        enable_implicit_namespace_pkgs = enable_implicit_namespace_pkgs,
+        environment = environment,
+        envsubst = envsubst,
+        experimental_index_url = experimental_index_url,
+        experimental_requirement_cycles = experimental_requirement_cycles,
+        experimental_target_platforms = experimental_target_platforms,
+        extra_pip_args = extra_pip_args,
+        hub_name = hub_name,
+        isolated = isolated,
+        netrc = netrc,
+        pip_data_exclude = pip_data_exclude,
+        python_interpreter = python_interpreter,
+        python_interpreter_target = python_interpreter_target,
+        python_version = python_version,
+        quiet = quiet,
+        requirements_by_platform = requirements_by_platform,
+        requirements_darwin = requirements_darwin,
+        requirements_linux = requirements_linux,
+        requirements_lock = requirements_lock,
+        requirements_windows = requirements_windows,
+        timeout = timeout,
+        whl_modifications = whl_modifications,
         **kwargs
     )