Upgrade rules_python to 1.7.0

This is a fork of #27498.

From the comments/open items on that PR, I've:
1. Removed the tests that were called out as not needed.
2. Added manual tag to `focus_test` to avoid pickup in larger runs until issue with `--experimental_enable_skyfocus` is resolved.
3. Switched failing test cases from using `from bazel_tools.tools.python.runfiles import runfiles` to use `from python.runfiles import runfiles`

There are still a couple failing tests, looking for some guidance on if these can be removed/ignored:
```
test_source_file_does_not_override_standard_library FAILED: Expected regexp 'I am lib!' not found.
```
and
```
1) explicitInitPy_CanBeSelectivelyDisabled(com.google.devtools.build.lib.bazel.rules.python.BazelPyBinaryConfiguredTargetTest)
value of      : getEmptyFilenames()
unexpected (4): pkg/_foo.venv/lib/python3.11/__init__.py, pkg/_foo.venv/__init__.py, pkg/_foo.venv/lib/python3.11/site-packages/__init__.py, pkg/_foo.venv/lib/__init__.py
---
expected      : [pkg/__init__.py]
but was       : [pkg/_foo.venv/lib/python3.11/__init__.py, pkg/_foo.venv/__init__.py, pkg/_foo.venv/lib/python3.11/site-packages/__init__.py, pkg/__init__.py, pkg/_foo.venv/lib/__init__.py]
	at com.google.devtools.build.lib.bazel.rules.python.BazelPyBinaryConfiguredTargetTest.explicitInitPy_CanBeSelectivelyDisabled(BazelPyBinaryConfiguredTargetTest.java:235)
2) explicitInitPy_CanBeGloballyDisabled(com.google.devtools.build.lib.bazel.rules.python.BazelPyBinaryConfiguredTargetTest)
value of      : getEmptyFilenames()
unexpected (4): pkg/_foo.venv/lib/python3.11/__init__.py, pkg/_foo.venv/__init__.py, pkg/_foo.venv/lib/python3.11/site-packages/__init__.py, pkg/_foo.venv/lib/__init__.py
---
expected      : [pkg/__init__.py]
but was       : [pkg/_foo.venv/lib/python3.11/__init__.py, pkg/_foo.venv/__init__.py, pkg/_foo.venv/lib/python3.11/site-packages/__init__.py, pkg/__init__.py, pkg/_foo.venv/lib/__init__.py]
	at com.google.devtools.build.lib.bazel.rules.python.BazelPyBinaryConfiguredTargetTest.explicitInitPy_CanBeGloballyDisabled(BazelPyBinaryConfiguredTargetTest.java:250)
```

Closes #27792.

PiperOrigin-RevId: 839448559
Change-Id: I888614dd92fcbc4deda63a86a452bbd94e8f1523

Author: dabanki

MODULE.bazel

@@ -29,7 +29,7 @@ bazel_dep(name = "rules_java", version = "9.1.0")
 bazel_dep(name = "rules_jvm_external", version = "6.6")
 bazel_dep(name = "rules_license", version = "1.0.0")
 bazel_dep(name = "rules_pkg", version = "1.1.0")
-bazel_dep(name = "rules_python", version = "1.6.0")
+bazel_dep(name = "rules_python", version = "1.7.0")
 bazel_dep(name = "rules_shell", version = "0.6.1")
 bazel_dep(name = "rules_testing", version = "0.9.0")
 bazel_dep(name = "stardoc", version = "0.8.0", repo_name = "io_bazel_skydoc")
@@ -138,9 +138,9 @@ maven.install(
         # keep sorted
         "args4j:args4j:2.37",
         "com.beust:jcommander:1.82",
-        "com.dylibso.chicory:log:1.5.2",
         "com.dylibso.chicory:runtime:1.5.2",
         "com.dylibso.chicory:wasm:1.5.2",
+        "com.dylibso.chicory:log:1.5.2",
         "com.github.ben-manes.caffeine:caffeine:3.1.8",
         "com.github.stephenc.jcip:jcip-annotations:1.0-1",
         "com.google.api-client:google-api-client:1.35.2",

MODULE.bazel.lock

@@ -35,7 +35,7 @@ use_repo(buildozer_binary, "buildozer_binary")
 bazel_dep(name = "protobuf", version = "33.0", repo_name = "com_google_protobuf")
 bazel_dep(name = "rules_java", version = "9.1.0")
 bazel_dep(name = "rules_cc", version = "0.2.14")
-bazel_dep(name = "rules_python", version = "1.6.0")
+bazel_dep(name = "rules_python", version = "1.7.0")
 bazel_dep(name = "rules_shell", version = "0.6.1")
 bazel_dep(name = "apple_support", version = "1.24.2") # xcode_configure needs it
 bazel_dep(name = "rules_apple", version = "4.1.0", repo_name = None) # version bump, older versions contribute lockfile entries

src/MODULE.tools

@@ -17,17 +17,8 @@ java_library(
     name = "PythonTests_lib",
     srcs = glob(["*.java"]),
     deps = [
-        "//src/main/java/com/google/devtools/build/lib/actions",
-        "//src/main/java/com/google/devtools/build/lib/actions:artifacts",
-        "//src/main/java/com/google/devtools/build/lib/analysis:actions/substitution",
-        "//src/main/java/com/google/devtools/build/lib/analysis:actions/template_expansion_action",
-        "//src/main/java/com/google/devtools/build/lib/analysis:analysis_cluster",
-        "//src/main/java/com/google/devtools/build/lib/analysis:configured_target",
         "//src/main/java/com/google/devtools/build/lib/analysis/config:invalid_configuration_exception",
         "//src/main/java/com/google/devtools/build/lib/bazel/rules/python",
-        "//src/main/java/com/google/devtools/build/lib/util:os",
-        "//src/main/java/com/google/devtools/build/lib/vfs",
-        "//src/main/java/com/google/devtools/build/lib/vfs:pathfragment",
         "//src/test/java/com/google/devtools/build/lib/analysis/util",
         "//src/test/java/com/google/devtools/build/lib/rules/python:PythonTestUtils",
         "//src/test/java/com/google/devtools/build/lib/testutil:TestConstants",

src/test/java/com/google/devtools/build/lib/bazel/rules/python/BUILD

@@ -17,17 +17,8 @@
 import static com.google.common.truth.Truth.assertThat;
 import static com.google.devtools.build.lib.rules.python.PythonTestUtils.getPyLoad;
 
-import com.google.devtools.build.lib.actions.Action;
-import com.google.devtools.build.lib.actions.Artifact;
-import com.google.devtools.build.lib.analysis.ConfiguredTarget;
-import com.google.devtools.build.lib.analysis.FilesToRunProvider;
-import com.google.devtools.build.lib.analysis.actions.Substitution;
-import com.google.devtools.build.lib.analysis.actions.TemplateExpansionAction;
 import com.google.devtools.build.lib.analysis.util.BuildViewTestCase;
 import com.google.devtools.build.lib.testutil.TestConstants;
-import com.google.devtools.build.lib.util.OS;
-import com.google.devtools.build.lib.vfs.FileSystemUtils;
-import com.google.devtools.build.lib.vfs.PathFragment;
 import java.util.regex.Pattern;
 import org.junit.Test;
 import org.junit.runner.RunWith;
@@ -44,157 +35,11 @@ private static String join(String... lines) {
     return String.join("\n", lines);
   }
 
-  /**
-   * Given a {@code py_binary} or {@code py_test} target and substitution key, returns the
-   * corresponding substitution value used by the generated stub script.
-   *
-   * <p>This works by casting the stub script's generating action to a template expansion action and
-   * looking for the requested substitution key. It's therefore linked to the implementation of the
-   * rule, but that's the cost we pay for avoiding an execution-time test.
-   */
-  private String getSubstitutionValueFromStub(
-      ConfiguredTarget pyExecutableTarget, String substitutionKey) {
-    // First find the stub script. Normally this is just the executable associated with the target.
-    // But for Windows the executable is a separate launcher with an ".exe" extension, and the stub
-    // script artifact has the same base name with the extension ".temp" instead. (At least, when
-    // --build_python_zip is enabled, which is the default on Windows.)
-    Artifact executable = pyExecutableTarget.getProvider(FilesToRunProvider.class).getExecutable();
-    Artifact stub;
-    if (OS.getCurrent() == OS.WINDOWS) {
-      stub =
-          getDerivedArtifact(
-              FileSystemUtils.replaceExtension(executable.getRootRelativePath(), ".temp"),
-              executable.getRoot(),
-              executable.getArtifactOwner());
-    } else {
-      stub = executable;
-    }
-    assertThat(stub).isNotNull();
-    // Now grab its generating action, which should be a template action, and get the key for the
-    // binary path.
-    Action generatingAction = getGeneratingAction(stub);
-    assertThat(generatingAction).isInstanceOf(TemplateExpansionAction.class);
-    TemplateExpansionAction templateAction = (TemplateExpansionAction) generatingAction;
-    for (Substitution sub : templateAction.getSubstitutions()) {
-      if (sub.getKey().equals(substitutionKey)) {
-        return sub.getValueUnchecked();
-      }
-    }
-    throw new AssertionError(
-        "Failed to find the '"
-            + substitutionKey
-            + "' key in the stub script's template "
-            + "expansion action");
-  }
-
-  private String getInterpreterPathFromStub(ConfiguredTarget pyExecutableTarget) {
-    return getSubstitutionValueFromStub(pyExecutableTarget, "%python_binary%");
-  }
-
-  private String getShebangFromStub(ConfiguredTarget pyExecutableTarget) {
-    return getSubstitutionValueFromStub(pyExecutableTarget, "%shebang%");
-  }
-
   // TODO(brandjon): Move generic toolchain tests that don't access legacy behavior to
   // PyExecutableConfiguredtargetTestBase. Asserting on the chosen PyRuntimeInfo is problematic to
   // do at analysis time though. It's easier in this test because we know the PythonSemantics is
   // BazelPythonSemantics.
 
-  /** Adds toolchain definitions to a //toolchains package, for user by the below tests. */
-  private void defineToolchains() throws Exception {
-    scratch.file(
-        "toolchains/BUILD",
-        getPyLoad("py_runtime"),
-        getPyLoad("py_runtime_pair"),
-        "py_runtime(",
-        "    name = 'py3_runtime',",
-        "    interpreter_path = '/system/python3',",
-        "    python_version = 'PY3',",
-        "    stub_shebang = '#!/usr/bin/env python3',",
-        ")",
-        "py_runtime_pair(",
-        "    name = 'py_runtime_pair',",
-        "    py3_runtime = ':py3_runtime',",
-        ")",
-        "toolchain(",
-        "    name = 'py_toolchain',",
-        "    toolchain = ':py_runtime_pair',",
-        "    toolchain_type = '" + TOOLCHAIN_TYPE + "',",
-        ")",
-        "py_runtime_pair(",
-        "    name = 'py_runtime_pair_for_py3_only',",
-        "    py3_runtime = ':py3_runtime',",
-        ")",
-        "toolchain(",
-        "    name = 'py_toolchain_for_py3_only',",
-        "    toolchain = ':py_runtime_pair_for_py3_only',",
-        "    toolchain_type = '" + TOOLCHAIN_TYPE + "',",
-        ")");
-  }
-
-  @Test
-  public void runtimeObtainedFromToolchain() throws Exception {
-    defineToolchains();
-    scratch.file(
-        "pkg/BUILD",
-        getPyLoad("py_binary"),
-        """
-        py_binary(
-            name = "py3_bin",
-            srcs = ["py3_bin.py"],
-            python_version = "PY3",
-        )
-        """);
-    useConfiguration("--extra_toolchains=//toolchains:py_toolchain");
-
-    ConfiguredTarget py3 = getConfiguredTarget("//pkg:py3_bin");
-
-    String py3Path = getInterpreterPathFromStub(py3);
-    assertThat(py3Path).isEqualTo("/system/python3");
-
-    String py3Shebang = getShebangFromStub(py3);
-    assertThat(py3Shebang).isEqualTo("#!/usr/bin/env python3");
-  }
-
-  @Test
-  public void toolchainCanOmitUnusedRuntimeVersion() throws Exception {
-    defineToolchains();
-    scratch.file(
-        "pkg/BUILD",
-        getPyLoad("py_binary"),
-        """
-        py_binary(
-            name = "py3_bin",
-            srcs = ["py3_bin.py"],
-            python_version = "PY3",
-        )
-        """);
-    useConfiguration("--extra_toolchains=//toolchains:py_toolchain_for_py3_only");
-
-    String path = getInterpreterPathFromStub(getConfiguredTarget("//pkg:py3_bin"));
-    assertThat(path).isEqualTo("/system/python3");
-  }
-
-  @Test
-  public void toolchainTakesPrecedenceOverLegacyFlags() throws Exception {
-    defineToolchains();
-    scratch.file(
-        "pkg/BUILD",
-        getPyLoad("py_binary"),
-        """
-        py_binary(
-            name = "py3_bin",
-            srcs = ["py3_bin.py"],
-            python_version = "PY3",
-        )
-        """);
-    useConfiguration(
-        "--extra_toolchains=//toolchains:py_toolchain", "--python_path=/better/not/be/this/one");
-
-    String path = getInterpreterPathFromStub(getConfiguredTarget("//pkg:py3_bin"));
-    assertThat(path).isEqualTo("/system/python3");
-  }
-
   /**
    * Creates a custom toolchain at //toolchains:custom that has the given lines in its rule
    * implementation function.
@@ -282,37 +127,6 @@ public void explicitInitPy_CanBeGloballyEnabled() throws Exception {
     assertThat(getDefaultRunfiles(getConfiguredTarget("//pkg:foo")).getEmptyFilenames()).isEmpty();
   }
 
-  @Test
-  public void explicitInitPy_CanBeSelectivelyDisabled() throws Exception {
-    scratch.file(
-        "pkg/BUILD",
-        getPyLoad("py_binary"),
-        join(
-            "py_binary(", //
-            "    name = 'foo',",
-            "    srcs = ['foo.py'],",
-            "    legacy_create_init = True,",
-            ")"));
-    useConfiguration("--incompatible_default_to_explicit_init_py=true");
-    assertThat(getDefaultRunfiles(getConfiguredTarget("//pkg:foo")).getEmptyFilenames())
-        .containsExactly(PathFragment.create("pkg/__init__.py"));
-  }
-
-  @Test
-  public void explicitInitPy_CanBeGloballyDisabled() throws Exception {
-    scratch.file(
-        "pkg/BUILD",
-        getPyLoad("py_binary"),
-        join(
-            "py_binary(", //
-            "    name = 'foo',",
-            "    srcs = ['foo.py'],",
-            ")"));
-    useConfiguration("--incompatible_default_to_explicit_init_py=false");
-    assertThat(getDefaultRunfiles(getConfiguredTarget("//pkg:foo")).getEmptyFilenames())
-        .containsExactly(PathFragment.create("pkg/__init__.py"));
-  }
-
   @Test
   public void explicitInitPy_CanBeSelectivelyEnabled() throws Exception {
     scratch.file(

src/test/java/com/google/devtools/build/lib/bazel/rules/python/BazelPyBinaryConfiguredTargetTest.java

@@ -102,22 +102,6 @@ java_test(
     ],
 )
 
-java_test(
-    name = "CompileOneDependencyIntegrationTest",
-    srcs = ["CompileOneDependencyIntegrationTest.java"],
-    tags = [
-        "no_windows",
-    ],
-    deps = [
-        "//src/main/java/com/google/devtools/build/lib/actions",
-        "//src/main/java/com/google/devtools/build/lib/events",
-        "//src/test/java/com/google/devtools/build/lib/buildtool/util",
-        "//src/test/java/com/google/devtools/build/lib/rules/python:PythonTestUtils",
-        "//third_party:junit4",
-        "//third_party:truth",
-    ],
-)
-
 java_test(
     name = "ContextProviderInitializationTest",
     srcs = ["ContextProviderInitializationTest.java"],