Skip to content

Remove debug print statements from unittest adapter output#25854

Merged
eleanorjboyd merged 2 commits intomainfrom
copilot/fix-unwanted-debug-output
Mar 23, 2026
Merged

Remove debug print statements from unittest adapter output#25854
eleanorjboyd merged 2 commits intomainfrom
copilot/fix-unwanted-debug-output

Conversation

Copy link

Copilot AI commented Mar 11, 2026

PROJECT_ROOT_PATH is set, using /... as cwd for execution payload is printed to stdout during unittest discovery and execution, leaking into the Test Results panel.

  • Remove print() in python_files/unittestadapter/execution.py (execution path)
  • Remove print() in python_files/unittestadapter/discovery.py (discovery path)

All functional logic (reading the env var, setting top_level_dir, updating the global) is unchanged.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…est adapter

Fixes #24743-like issue where "PROJECT_ROOT_PATH is set, using ... as cwd
for execution payload" debug output was appearing in Test Results output.

Removes print() calls from both execution.py and discovery.py while
preserving all functional logic.

Co-authored-by: eleanorjboyd <26030610+eleanorjboyd@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix unwanted debug output in unittest results Remove debug print statements from unittest adapter output Mar 11, 2026
@eleanorjboyd eleanorjboyd marked this pull request as ready for review March 18, 2026 22:37
@eleanorjboyd eleanorjboyd added debt Covers everything internal: CI, testing, refactoring of the codebase, etc. skip tests Updates to tests unnecessary skip-issue-check labels Mar 18, 2026
@vs-code-engineering vs-code-engineering bot added this to the 1.113.0 milestone Mar 18, 2026
@eleanorjboyd eleanorjboyd merged commit 5c2c394 into main Mar 23, 2026
93 of 95 checks passed
@eleanorjboyd eleanorjboyd deleted the copilot/fix-unwanted-debug-output branch March 23, 2026 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

debt Covers everything internal: CI, testing, refactoring of the codebase, etc. skip tests Updates to tests unnecessary skip-issue-check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unwanted debug output when running unittests (again).

3 participants