fix missing arg, test_basic_windows analysis test

Author: rickeylev

python/private/py_executable.bzl

@@ -701,6 +701,7 @@ def _create_venv_unixy(ctx, *, venv_root, runtime, interpreter_actual_path):
 
     venv_site_packages = "lib/python{}/site-packages".format(version)
     return _venv_details(
+        interpreter = interpreter,
         pyvenv_cfg = pyvenv_cfg,
         venv_site_packages = venv_site_packages,
         bin_dir = bin_dir,

tests/base_rules/py_executable_base_tests.bzl

@@ -44,27 +44,20 @@ def _test_basic_windows(name, config):
         impl = _test_basic_windows_impl,
         target = name + "_subject",
         config_settings = {
-            # NOTE: The default for this flag is based on the Bazel host OS, not
-            # the target platform. For windows, it defaults to true, so force
-            # it to that to match behavior when this test runs on other
-            # platforms.
-            # Pass value to both native and starlark versions of the flag until
-            # the native one is removed.
-            labels.BUILD_PYTHON_ZIP: True,
             "//command_line_option:cpu": "windows_x86_64",
             "//command_line_option:crosstool_top": CROSSTOOL_TOP,
             "//command_line_option:extra_execution_platforms": [platform_targets.WINDOWS_X86_64],
             "//command_line_option:extra_toolchains": [CC_TOOLCHAIN],
             "//command_line_option:platforms": [platform_targets.WINDOWS_X86_64],
-        } | maybe_builtin_build_python_zip("true"),
+        },
         attr_values = {},
     )
 
 def _test_basic_windows_impl(env, target):
     target = env.expect.that_target(target)
     target.executable().path().contains(".exe")
     target.runfiles().contains_predicate(matching.str_endswith(
-        target.meta.format_str("/{name}.zip"),
+        target.meta.format_str("/{name}"),
     ))
     target.runfiles().contains_predicate(matching.str_endswith(
         target.meta.format_str("/{name}.exe"),