Skip to content

Add pytest cancellation subprocess regression coverage - #26058

Merged
Eleanor Boyd (eleanorjboyd) merged 3 commits into
mainfrom
copilot/fix-review-comment-25932
Jul 24, 2026
Merged

Add pytest cancellation subprocess regression coverage#26058
Eleanor Boyd (eleanorjboyd) merged 3 commits into
mainfrom
copilot/fix-review-comment-25932

Conversation

CopilotAI commented Jul 24, 2026

Copy link
Copy Markdown

Pytest cancellation must terminate the child process in the non-environment-extension execution path. This change adds regression coverage for that behavior.

  • Subprocess tracking
    • Retain the spawned process for cancellation handling.
  • Regression coverage
    • Cancel an active pytest run.
    • Verify the child process receives kill().

CopilotAI changed the title [WIP] Fix code according to review comment in PR #25932Add pytest cancellation subprocess regression coverageJul 24, 2026
@eleanorjboyd
Eleanor Boyd (eleanorjboyd) marked this pull request as ready for review July 24, 2026 18:01
@eleanorjboyd
Eleanor Boyd (eleanorjboyd) merged commit 12bd780 into mainJul 24, 2026
131 of 132 checks passed
@eleanorjboyd
Eleanor Boyd (eleanorjboyd) deleted the copilot/fix-review-comment-25932 branch July 24, 2026 20:45
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

debtCode quality issuesskip-issue-check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@rzhao271@dmitrivMS@eleanorjboyd