From 6420f52da13c7de0dc142af4935e9e262ca576e4 Mon Sep 17 00:00:00 2001 From: Chris Novakovic Date: Mon, 1 Dec 2025 02:32:33 +0000 Subject: [PATCH] Fix interpreter path in in-repo interpreter test The in-repo interpreters were moved from `third_party/cpython/lib` to `third_party/cc/cpython` in #277, but the interpreter directory path in `//test:interpreter_not_included_test` wasn't also updated, so the test still passes even if please_pex erroneously packs the in-repo interpreter into the .pex file. Update the path to fix the test. --- test/interpreter_not_included_test.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/interpreter_not_included_test.py b/test/interpreter_not_included_test.py index 2721764..f5cd839 100644 --- a/test/interpreter_not_included_test.py +++ b/test/interpreter_not_included_test.py @@ -6,9 +6,9 @@ class InterpreterNotIncludedTest(unittest.TestCase): def test_interpreter_is_not_included(self): - """Test that we don't include the interpreter as a tool.""" + """Test that we don't include any in-repo interpreter in the .pex file.""" zf = zipfile.ZipFile(sys.argv[0]) - names = [name for name in zf.namelist() if "third_party/cpython/lib" in name] + names = [name for name in zf.namelist() if "third_party/cc/cpython/" in name] self.assertFalse(names) def test_arcat_is_not_included(self):