Skip to content

Commit 6420f52

Browse files
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.
1 parent ce5cae0 commit 6420f52

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/interpreter_not_included_test.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@
66
class InterpreterNotIncludedTest(unittest.TestCase):
77

88
def test_interpreter_is_not_included(self):
9-
"""Test that we don't include the interpreter as a tool."""
9+
"""Test that we don't include any in-repo interpreter in the .pex file."""
1010
zf = zipfile.ZipFile(sys.argv[0])
11-
names = [name for name in zf.namelist() if "third_party/cpython/lib" in name]
11+
names = [name for name in zf.namelist() if "third_party/cc/cpython/" in name]
1212
self.assertFalse(names)
1313

1414
def test_arcat_is_not_included(self):

0 commit comments

Comments
 (0)