chore: use bazel 9 by default (#3662)

Updates the project bazel version and bzlmod examples to use Bazel 9

Along the way, upgrade rules_go to 0.60.0; this is a dev-only
dependency. This is needed
because Bazel 9 removed CcInfo, and earlier rules_go versions refer to
the builtin symbol.

Also remove the test to check the bazel version. It doesn't do much
useful.

Author: rickeylev

.bazelversion

@@ -1 +1 @@
-8.x
+9.x

MODULE.bazel

@@ -226,7 +226,7 @@ bazel_dep(name = "another_module", version = "0", dev_dependency = True)
 
 # Extra gazelle plugin deps so that WORKSPACE.bzlmod can continue including it for e2e tests.
 # We use `WORKSPACE.bzlmod` because it is impossible to have dev-only local overrides.
-bazel_dep(name = "rules_go", version = "0.41.0", dev_dependency = True, repo_name = "io_bazel_rules_go")
+bazel_dep(name = "rules_go", version = "0.60.0", dev_dependency = True, repo_name = "io_bazel_rules_go")
 
 internal_dev_deps = use_extension(
     "//python/private:internal_dev_deps.bzl",

examples/bzlmod/.bazelversion

@@ -1 +1 @@
-7.x
+9.x

tests/BUILD.bazel

@@ -1,6 +1,4 @@
 load("@bazel_skylib//rules:build_test.bzl", "build_test")
-load("@rules_shell//shell:sh_test.bzl", "sh_test")
-load("//:version.bzl", "BAZEL_VERSION")
 
 package(default_visibility = ["//visibility:public"])
 
@@ -27,29 +25,3 @@ build_test(
         "//python/entry_points:py_console_script_binary_bzl",
     ],
 )
-
-genrule(
-    name = "assert_bazelversion",
-    srcs = ["//:.bazelversion"],
-    outs = ["assert_bazelversion_test.sh"],
-    cmd = """\
-set -o errexit -o nounset -o pipefail
-current=$$(cat "$(execpath //:.bazelversion)")
-cat > "$@" <<EOF
-#!/usr/bin/env bash
-set -o errexit -o nounset -o pipefail
-if [[ \"$${{current}}\" != \"{expected}\" ]]; then
-    >&2 echo "ERROR: current bazel version '$${{current}}' is not the expected '{expected}'"
-    exit 1
-fi
-EOF
-""".format(
-        expected = BAZEL_VERSION,
-    ),
-    executable = True,
-)
-
-sh_test(
-    name = "assert_bazelversion_test",
-    srcs = [":assert_bazelversion_test.sh"],
-)

tests/integration/local_toolchains/BUILD.bazel

@@ -24,7 +24,10 @@ py_test(
         "//:py": "local",
     },
     # Make this test better respect pyenv
-    env_inherit = ["PYENV_VERSION"],
+    env_inherit = [
+        "PYENV_VERSION",
+        "PATH",
+    ],
 )
 
 py_test(

tests/integration/local_toolchains/local_runtime_test.py

@@ -16,15 +16,20 @@ def test_python_from_path_used(self):
         # that wouldn't be reflected when sub-shells are run later.
         shell_path = shutil.which("python3")
 
+        if shell_path is None:
+            self.fail(
+                "which(python3) returned None.\n" + f"PATH={os.environ.get('PATH')}"
+            )
+
         # We call the interpreter and print its executable because of
         # things like pyenv: they install a shim that re-execs python.
         # The shim is e.g. /home/user/.pyenv/shims/python3, which then
         # runs e.g. /usr/bin/python3
         with tempfile.TemporaryDirectory() as temp_dir:
             file_path = os.path.join(temp_dir, "info.py")
-            with open(file_path, 'w') as f:
+            with open(file_path, "w") as f:
                 f.write(
-                """
+                    """
 import sys
 print(sys.executable)
 print(sys._base_executable)