Skip to content

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

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

test(mcp): cover stdio disconnect cleanup#42140
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

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