Problem
The canonical location for MCP server tests is test/playwright/unit/ai/mcp/server/. However, the older test/playwright/mcp/ directory still exists and contains grandfathered files. Without automated enforcement, developers or agents might inadvertently place new tests in the deprecated directory, leading to fragmented test discovery and missing coverage.
Value
Automated linting or CI enforcement prevents location drift at the commit or PR stage, removing the reliance on retrospective peer-review catches.
Proposal
Add a pre-commit hook, ESLint rule, or CI step that enforces:
test/playwright/mcp/ MUST NOT contain new additions.- Only the specific grandfathered files are permitted in that directory until they are migrated.
- Any new test related to MCP servers must be placed under
test/playwright/unit/ai/mcp/server/.
Related
Authored by Gemini 3.1 Pro (Antigravity). Session 90dc2e65-962b-419f-91af-55dea55e5d3d.
Problem
The canonical location for MCP server tests is
test/playwright/unit/ai/mcp/server/. However, the oldertest/playwright/mcp/directory still exists and contains grandfathered files. Without automated enforcement, developers or agents might inadvertently place new tests in the deprecated directory, leading to fragmented test discovery and missing coverage.Value
Automated linting or CI enforcement prevents location drift at the commit or PR stage, removing the reliance on retrospective peer-review catches.
Proposal
Add a pre-commit hook, ESLint rule, or CI step that enforces:
test/playwright/mcp/MUST NOT contain new additions.test/playwright/unit/ai/mcp/server/.Related
Authored by Gemini 3.1 Pro (Antigravity). Session 90dc2e65-962b-419f-91af-55dea55e5d3d.