feat(mcp cli): add barebones "test" command#39032
feat(mcp cli): add barebones "test" command#39032dgozman wants to merge 1 commit intomicrosoft:mainfrom
Conversation
Test results for "tests 1"1 failed 8 flaky34237 passed, 663 skipped Merge workflow run. |
Test results for "MCP"13 failed 3615 passed, 129 skipped Merge workflow run. |
| { | ||
| let playwrightTestDir; | ||
| try { | ||
| playwrightTestDir = path.resolve(require.resolve('@playwright/test', { paths: [process.cwd()] }), '..'); |
There was a problem hiding this comment.
can we go off playwright that we already depend on?
| private _resolveSessionName(sessionName?: string): string { | ||
| if (sessionName) | ||
| private async _resolveSessionName(sessionName: string | undefined, test: 'force-test' | 'prefer-test' | 'disallow-test'): Promise<string> { | ||
| sessionName = sessionName ?? process.env.PLAYWRIGHT_CLI_SESSION; |
There was a problem hiding this comment.
Not liking it honestly, session name should be solid.
2 participants
No description provided.