chore(#610): Create a test case

Author: arrdem

MODULE.bazel

@@ -114,3 +114,9 @@ oci.pull(
     tag = "latest",
 )
 use_repo(oci, "ubuntu", "ubuntu_linux_amd64", "ubuntu_linux_arm64_v8")
+
+local_repository = use_repo_rule("@bazel_tools//tools/build_defs/repo:local.bzl", "local_repository")
+local_repository(
+    name = "rpy610_test",
+    path = "./py/tests/rpy610/subrepo",
+)

py/tests/rpy610/BUILD.bazel

@@ -0,0 +1,16 @@
+load("//py/private/py_venv:defs.bzl", "py_venv_test")
+
+py_venv_test(
+    name = "test",
+    srcs = [
+        "test.py",
+    ],
+    imports = [
+        ".",
+    ],
+    main = "test.py",
+    deps = [
+        "@pypi_cowsay//:pkg",
+        "@rpy610_test//:foo"
+    ],
+)

py/tests/rpy610/subrepo/BUILD.bazel

@@ -0,0 +1,10 @@
+load("@aspect_rules_py//py:defs.bzl", "py_library")
+
+py_library(
+    name = "foo",
+    srcs = [
+        "foo.py"
+    ],
+    imports = ["."],
+    visibility = ["//visibility:public"]
+)

py/tests/rpy610/subrepo/MODULE.bazel

@@ -0,0 +1,2 @@
+def foo(x):
+    return x ** 3.15

py/tests/rpy610/subrepo/foo.py

@@ -0,0 +1,32 @@
+#!/usr/bin/env python3
+
+import os
+import sys
+import site
+
+print("---")
+print("__file__:", __file__)
+print("sys.prefix:", sys.prefix)
+print("sys.executable:", sys.executable)
+print("site.PREFIXES:")
+for p in site.PREFIXES:
+    print(" -", p)
+
+# The virtualenv module should have already been loaded at interpreter startup
+assert "_virtualenv" in sys.modules
+
+# Note that we can't assume that a `.runfiles` tree has been created as CI may
+# use a different layout.
+
+# The virtualenv changes the sys.prefix, which should be in our runfiles
+assert sys.prefix.endswith("/py/tests/rpy610/.test")
+
+# That prefix should also be "the" prefix per site.PREFIXES
+assert site.PREFIXES[0].endswith("/py/tests/rpy610/.test")
+
+# The virtualenv also changes the sys.executable (if we've done this right)
+assert sys.executable.find("/py/tests/rpy610/.test/bin/python") != -1
+
+# aspect-build/rules_py#610, these imports aren't quite right
+import foo
+print(foo.__file__)

py/tests/rpy610/test.py

@@ -483,7 +483,7 @@ pub fn populate_venv_with_copies(
         .into_diagnostic()?;
 
     for line in BufReader::new(source_pth).lines().map_while(Result::ok) {
-        #[cfg(feature = "debug")]
+        //#[cfg(feature = "debug")]
         eprintln!("Got pth line {}", &line);
 
         let line = line.trim().to_string();
@@ -548,6 +548,13 @@ pub fn populate_venv_with_copies(
             //
             // [1] https://github.com/python/cpython/blob/ce31ae5209c976d28d1c21fcbb06c0ae5e50a896/Lib/site.py#L215
 
+            // aspect-build/rules_py#610
+            //
+            //   While these relative paths seem to work fine for _internal_
+            //   runfiles within the `_main` workspace, problems occur when we
+            //   try to take relative paths to _other_ workspaces because bzlmod
+            //   may munge the directory names to be something that doesn't
+            //   exist.
             let path_to_runfiles =
                 diff_paths(&action_bin_dir, action_bin_dir.join(&venv.site_dir)).unwrap();
 
@@ -561,7 +568,6 @@ pub fn populate_venv_with_copies(
         }
     }
 
-    //Err(miette!("Failing for debug purposes"))
     Ok(())
 }