REPL launch print test test_print_statement_non_darwin is skipped on Linux environments #26156

Activity

1. Satyam Pandey (SatyamPandey-07)

   added

   feature-requestRequest for new features or functionalityRequest for new features or functionality

   on Sep 19, 2026
2. github-actions

   added

   triage-neededNeeds assignment to the proper sub-teamNeeds assignment to the proper sub-team

   on Sep 19, 2026
3. vs-code-engineering

   removed

   triage-neededNeeds assignment to the proper sub-teamNeeds assignment to the proper sub-team

   on Sep 19, 2026
4. vs-code-engineering

   assigned

   Anthony Kim (anthonykim1)

   on Sep 19, 2026
5. saitanallensantiago26-beep commented on Sep 19, 2026

   

   Allen Bacabis Santiago (saitanallensantiago26-beep)

   on Sep 19, 2026

   More actions

   Description

   In python_files/pythonrc.py, the launch banner for the VS Code Native REPL displays a platform-dependent shortcut key: "Cmd" on Darwin (macOS) and "Ctrl" on all other platforms (Linux, Windows, etc.):

   ctrl_key = "Cmd" if sys.platform == "darwin" else "Ctrl"
   print(f"{ctrl_key} click to launch VS Code Native REPL (https://aka.ms/python-native-repl)")

   However, in python_files/tests/test_shell_integration.py, the test verifying this behavior for non-Darwin operating systems (test_print_statement_non_darwin) is guarded with if sys.platform == "win32": instead of if sys.platform != "darwin":.

   As a consequence, when test suites run on Linux (such as Ubuntu CI runners or local Linux developer environments), sys.platform == "darwin" evaluates to False and sys.platform == "win32" also evaluates to False. Neither test_print_statement_darwin nor test_print_statement_non_darwin gets defined or executed, causing a silent test coverage gap on Linux platforms.

   Steps to Reproduce

   1. Check out the repository on a Linux system or container (sys.platform == "linux").
   2. Run pytest targeting python_files/tests/test_shell_integration.py:

      python -m pytest python_files/tests/test_shell_integration.py -v -k "test_print_statement"

   3. Inspect the collected tests.
   4. Notice that 0 print statement tests are collected or executed on Linux (collected 0 items / 0 selected).

   Expected Behavior

   On Linux systems (which are non-Darwin platforms), test_print_statement_non_darwin should be collected and executed to verify that "Ctrl click to launch VS Code Native REPL (https://aka.ms/python-native-repl)" is printed.

   Actual Behavior / Error Logs

   $ python -m pytest python_files/tests/test_shell_integration.py -v -k "test_print_statement"
   ============================= test session starts =============================
   platform linux -- Python 3.10.x, pytest-8.4.2
   collected 6 items / 6 deselected / 0 selected
   
   ============================ 6 deselected in 0.03s ============================
   

   (On Linux, test_print_statement_non_darwin is missing from the test suite because it is conditionally defined only when sys.platform == "win32".)

   Proposed Fix

   Update line 122 of python_files/tests/test_shell_integration.py from:

   if sys.platform == "win32":

   to:

   if sys.platform != "darwin":

   This ensures test_print_statement_non_darwin is defined and executed across all non-Darwin platforms, including both Linux and Windows.
6. Eduardo Villalpando Mello (edvilme)

   closed this as completedin 3d7eb01on Sep 21, 2026
7. pull

   added a commit that references this issue on Sep 21, 2026

   fix(testing): enable REPL launch print test for all non-Darwin platfo…

   ...

   This issue will close once commit 3d7eb01 is merged into the 'main' branch.

   3d7eb01