Skip to content

test(mcp): cover stdio disconnect cleanup - #42140

Open
Harsh Mathur (harshmathurx) wants to merge 2 commits into
microsoft:mainfrom
harshmathurx:jeeves/mcp-disconnect-cleanup
Open

test(mcp): cover stdio disconnect cleanup#42140
Harsh Mathur (harshmathurx) wants to merge 2 commits into
microsoft:mainfrom
harshmathurx:jeeves/mcp-disconnect-cleanup

Conversation

@harshmathurx

Copy link
Copy Markdown
Contributor

Summary

  • add regression coverage for isolated MCP stdio cleanup when the client transport disconnects
  • expose the stdio transport from the MCP test fixture so the test can close the transport directly
  • make fixture cleanup tolerate an already-closed client after simulated disconnect

Test Plan

  • npm run build
  • npm run test-mcp -- --project=chromium tests/mcp/launch.spec.ts -g "isolated stdio server closes browser on client transport disconnect"
  • npm run test-mcp -- --project=chromium tests/mcp/launch.spec.ts
  • npx eslint tests/mcp/fixtures.ts tests/mcp/launch.spec.ts
  • npm run tsc

References #41918

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Test results for "MCP"

1 failed
❌ [firefox] › mcp/cli-parsing.spec.ts:76 › should preserve leading zeros in string arguments @mcp-windows-latest-firefox

8062 passed, 1284 skipped


Merge workflow run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant