Skip to content

Comments

test: enforce sequential tool execution in MCP e2e test#538

Merged
ashwin-ant merged 1 commit intomainfrom
ashwin/fix-mcp-test-sequential-execution
Jan 30, 2026
Merged

test: enforce sequential tool execution in MCP e2e test#538
ashwin-ant merged 1 commit intomainfrom
ashwin/fix-mcp-test-sequential-execution

Conversation

@ashwin-ant
Copy link
Collaborator

Summary

  • Update MCP permission enforcement e2e test to execute greet and echo tools sequentially instead of in parallel
  • Ensures deterministic tool call ordering for more reliable permission validation

Test plan

  • Run python -m pytest e2e-tests/test_sdk_mcp_tools.py::test_sdk_mcp_permission_enforcement to verify the test passes with sequential execution
  • Confirm greet tool is called before echo tool in the test output

Changelog

🤖 Generated with Claude Code (0% 4-shotted by claude)

Update the MCP permission enforcement test to run greet and echo tools
sequentially rather than in parallel. This ensures deterministic ordering
of tool calls, making the test more reliable when validating permission
behavior.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ashwin-ant ashwin-ant requested a review from a team January 30, 2026 16:57
@ashwin-ant ashwin-ant enabled auto-merge (squash) January 30, 2026 17:06
@ashwin-ant ashwin-ant merged commit 1bf665c into main Jan 30, 2026
9 checks passed
@ashwin-ant ashwin-ant deleted the ashwin/fix-mcp-test-sequential-execution branch January 30, 2026 17:07
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.

2 participants