-
Notifications
You must be signed in to change notification settings - Fork 67
Open
Labels
Milestone
Description
Overview
Migrate MCP integration tests to tests_v2/mcp/ with hardcoded port fixes.
Scope
- Migrate
tests/mcp/*→tests_v2/mcp/* - Fix hardcoded ports (5000, 8080, 8765) with
free_portfixture - Maintain all existing test functionality
Files to Migrate
From tests/mcp/:
test_agent_mcp_server.pytest_mcp_http_validation.pytest_mcp_integration.pytest_mcp_jira.pytest_mcp_simple.pytest_mcp_summarize.py
Anti-Patterns to Fix
Hardcoded ports:
| File | Ports |
|---|---|
test_agent_mcp_server.py |
5000, 8080 |
test_mcp_integration.py |
8765 |
Solution: Use free_port fixture for dynamic port allocation.
Migration Process
CRITICAL: Follow 5-step process for each test file:
- Copy file to
tests_v2/mcp/ - Update hardcoded ports to use
free_portfixture - Run the new test:
pytest tests_v2/mcp/test_file.py -v - Verify all tests pass and test same behavior
- Continue to next file
Acceptance Criteria
- All 6 MCP test files migrated to
tests_v2/mcp/ - All hardcoded ports replaced with dynamic allocation
- All tests pass:
pytest tests_v2/mcp/ -v - Existing tests unchanged:
pytest tests/mcp/ -vstill passes - Test count verified: same number of tests in new versions
Parent Issue
Part of #209 - Test Suite Overhaul Epic
Dependencies
Requires #239 (tests_v2 foundation) to be completed first.
Reactions are currently unavailable