MCP/playwright-mcp
1
0
Fork 0
mirror of https://github.com/microsoft/playwright-mcp.git synced 2025-07-27 00:52:27 +08:00
Code Issues Packages Projects Releases Wiki Activity

Migrate tests from /sse to /mcp endpoint and rename file

Browse Source 
- Renamed sse.spec.ts to http.spec.ts with git mv to preserve history
- Updated most tests to use StreamableHTTPClientTransport with /mcp endpoint instead of SSEClientTransport with /sse endpoint
- Changed existing "streamable http transport" test to "sse transport" test using /sse endpoint
- Updated test names from "sse transport" to "http transport" to reflect new default
- Removed SSE session-specific debug log expectations from tests since streamable HTTP doesn't use SSE sessions
- This provides better test coverage for the new default /mcp endpoint while maintaining legacy /sse testing

Co-authored-by: pavelfeldman <883973+pavelfeldman@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot] 2025-07-18 18:24:59 +00:00
parent 8d7f1fa231
commit 046958e7d9
1 changed files with 21 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

51
tests/sse.spec.ts → tests/http.spec.ts
View File

@ -66,15 +66,16 @@ const test = baseTest.extend<{ serverEndpoint: (options?: { args?: string[], noP
}, },
}); });
test('sse transport', async ({ serverEndpoint }) => { test('http transport', async ({ serverEndpoint }) => {
const { url } = await serverEndpoint(); const { url } = await serverEndpoint();
const transport = new SSEClientTransport(new URL('/sse', url)); const transport = new StreamableHTTPClientTransport(new URL('/mcp', url));
const client = new Client({ name: 'test', version: '1.0.0' }); const client = new Client({ name: 'test', version: '1.0.0' });
await client.connect(transport); await client.connect(transport);
await client.ping(); await client.ping();
expect(transport.sessionId, 'has session support').toBeDefined();
}); });
test('sse transport (config)', async ({ serverEndpoint }) => { test('http transport (config)', async ({ serverEndpoint }) => {
const config: Config = { const config: Config = {
server: { server: {
port: 0, port: 0,
@ -84,16 +85,16 @@ test('sse transport (config)', async ({ serverEndpoint }) => {
await fs.promises.writeFile(configFile, JSON.stringify(config, null, 2)); await fs.promises.writeFile(configFile, JSON.stringify(config, null, 2));
const { url } = await serverEndpoint({ noPort: true, args: ['--config=' + configFile] }); const { url } = await serverEndpoint({ noPort: true, args: ['--config=' + configFile] });
const transport = new SSEClientTransport(new URL('/sse', url)); const transport = new StreamableHTTPClientTransport(new URL('/mcp', url));
const client = new Client({ name: 'test', version: '1.0.0' }); const client = new Client({ name: 'test', version: '1.0.0' });
await client.connect(transport); await client.connect(transport);
await client.ping(); await client.ping();
}); });
test('sse transport browser lifecycle (isolated)', async ({ serverEndpoint, server }) => { test('http transport browser lifecycle (isolated)', async ({ serverEndpoint, server }) => {
const { url, stderr } = await serverEndpoint({ args: ['--isolated'] }); const { url, stderr } = await serverEndpoint({ args: ['--isolated'] });
const transport1 = new SSEClientTransport(new URL('/sse', url)); const transport1 = new StreamableHTTPClientTransport(new URL('/mcp', url));
const client1 = new Client({ name: 'test', version: '1.0.0' }); const client1 = new Client({ name: 'test', version: '1.0.0' });
await client1.connect(transport1); await client1.connect(transport1);
await client1.callTool({ await client1.callTool({
@ -102,7 +103,7 @@ test('sse transport browser lifecycle (isolated)', async ({ serverEndpoint, serv
}); });
await client1.close(); await client1.close();
const transport2 = new SSEClientTransport(new URL('/sse', url)); const transport2 = new StreamableHTTPClientTransport(new URL('/mcp', url));
const client2 = new Client({ name: 'test', version: '1.0.0' }); const client2 = new Client({ name: 'test', version: '1.0.0' });
await client2.connect(transport2); await client2.connect(transport2);
await client2.callTool({ await client2.callTool({
@ -113,9 +114,6 @@ test('sse transport browser lifecycle (isolated)', async ({ serverEndpoint, serv
await expect(async () => { await expect(async () => {
const lines = stderr().split('\n'); const lines = stderr().split('\n');
expect(lines.filter(line => line.match(/create SSE session/)).length).toBe(2);
expect(lines.filter(line => line.match(/delete SSE session/)).length).toBe(2);
expect(lines.filter(line => line.match(/create context/)).length).toBe(2); expect(lines.filter(line => line.match(/create context/)).length).toBe(2);
expect(lines.filter(line => line.match(/close context/)).length).toBe(2); expect(lines.filter(line => line.match(/close context/)).length).toBe(2);
@ -127,10 +125,10 @@ test('sse transport browser lifecycle (isolated)', async ({ serverEndpoint, serv
}).toPass(); }).toPass();
}); });
test('sse transport browser lifecycle (isolated, multiclient)', async ({ serverEndpoint, server }) => { test('http transport browser lifecycle (isolated, multiclient)', async ({ serverEndpoint, server }) => {
const { url, stderr } = await serverEndpoint({ args: ['--isolated'] }); const { url, stderr } = await serverEndpoint({ args: ['--isolated'] });
const transport1 = new SSEClientTransport(new URL('/sse', url)); const transport1 = new StreamableHTTPClientTransport(new URL('/mcp', url));
const client1 = new Client({ name: 'test', version: '1.0.0' }); const client1 = new Client({ name: 'test', version: '1.0.0' });
await client1.connect(transport1); await client1.connect(transport1);
await client1.callTool({ await client1.callTool({
@ -138,7 +136,7 @@ test('sse transport browser lifecycle (isolated, multiclient)', async ({ serverE
arguments: { url: server.HELLO_WORLD }, arguments: { url: server.HELLO_WORLD },
}); });
const transport2 = new SSEClientTransport(new URL('/sse', url)); const transport2 = new StreamableHTTPClientTransport(new URL('/mcp', url));
const client2 = new Client({ name: 'test', version: '1.0.0' }); const client2 = new Client({ name: 'test', version: '1.0.0' });
await client2.connect(transport2); await client2.connect(transport2);
await client2.callTool({ await client2.callTool({
@ -147,7 +145,7 @@ test('sse transport browser lifecycle (isolated, multiclient)', async ({ serverE
}); });
await client1.close(); await client1.close();
const transport3 = new SSEClientTransport(new URL('/sse', url)); const transport3 = new StreamableHTTPClientTransport(new URL('/mcp', url));
const client3 = new Client({ name: 'test', version: '1.0.0' }); const client3 = new Client({ name: 'test', version: '1.0.0' });
await client3.connect(transport3); await client3.connect(transport3);
await client3.callTool({ await client3.callTool({
@ -160,9 +158,6 @@ test('sse transport browser lifecycle (isolated, multiclient)', async ({ serverE
await expect(async () => { await expect(async () => {
const lines = stderr().split('\n'); const lines = stderr().split('\n');
expect(lines.filter(line => line.match(/create SSE session/)).length).toBe(3);
expect(lines.filter(line => line.match(/delete SSE session/)).length).toBe(3);
expect(lines.filter(line => line.match(/create context/)).length).toBe(3); expect(lines.filter(line => line.match(/create context/)).length).toBe(3);
expect(lines.filter(line => line.match(/close context/)).length).toBe(3); expect(lines.filter(line => line.match(/close context/)).length).toBe(3);
@ -174,10 +169,10 @@ test('sse transport browser lifecycle (isolated, multiclient)', async ({ serverE
}).toPass(); }).toPass();
}); });
test('sse transport browser lifecycle (persistent)', async ({ serverEndpoint, server }) => { test('http transport browser lifecycle (persistent)', async ({ serverEndpoint, server }) => {
const { url, stderr } = await serverEndpoint(); const { url, stderr } = await serverEndpoint();
const transport1 = new SSEClientTransport(new URL('/sse', url)); const transport1 = new StreamableHTTPClientTransport(new URL('/mcp', url));
const client1 = new Client({ name: 'test', version: '1.0.0' }); const client1 = new Client({ name: 'test', version: '1.0.0' });
await client1.connect(transport1); await client1.connect(transport1);
await client1.callTool({ await client1.callTool({
@ -186,7 +181,7 @@ test('sse transport browser lifecycle (persistent)', async ({ serverEndpoint, se
}); });
await client1.close(); await client1.close();
const transport2 = new SSEClientTransport(new URL('/sse', url)); const transport2 = new StreamableHTTPClientTransport(new URL('/mcp', url));
const client2 = new Client({ name: 'test', version: '1.0.0' }); const client2 = new Client({ name: 'test', version: '1.0.0' });
await client2.connect(transport2); await client2.connect(transport2);
await client2.callTool({ await client2.callTool({
@ -197,9 +192,6 @@ test('sse transport browser lifecycle (persistent)', async ({ serverEndpoint, se
await expect(async () => { await expect(async () => {
const lines = stderr().split('\n'); const lines = stderr().split('\n');
expect(lines.filter(line => line.match(/create SSE session/)).length).toBe(2);
expect(lines.filter(line => line.match(/delete SSE session/)).length).toBe(2);
expect(lines.filter(line => line.match(/create context/)).length).toBe(2); expect(lines.filter(line => line.match(/create context/)).length).toBe(2);
expect(lines.filter(line => line.match(/close context/)).length).toBe(2); expect(lines.filter(line => line.match(/close context/)).length).toBe(2);
@ -211,10 +203,10 @@ test('sse transport browser lifecycle (persistent)', async ({ serverEndpoint, se
}).toPass(); }).toPass();
}); });
test('sse transport browser lifecycle (persistent, multiclient)', async ({ serverEndpoint, server }) => { test('http transport browser lifecycle (persistent, multiclient)', async ({ serverEndpoint, server }) => {
const { url } = await serverEndpoint(); const { url } = await serverEndpoint();
const transport1 = new SSEClientTransport(new URL('/sse', url)); const transport1 = new StreamableHTTPClientTransport(new URL('/mcp', url));
const client1 = new Client({ name: 'test', version: '1.0.0' }); const client1 = new Client({ name: 'test', version: '1.0.0' });
await client1.connect(transport1); await client1.connect(transport1);
await client1.callTool({ await client1.callTool({
@ -222,7 +214,7 @@ test('sse transport browser lifecycle (persistent, multiclient)', async ({ serve
arguments: { url: server.HELLO_WORLD }, arguments: { url: server.HELLO_WORLD },
}); });
const transport2 = new SSEClientTransport(new URL('/sse', url)); const transport2 = new StreamableHTTPClientTransport(new URL('/mcp', url));
const client2 = new Client({ name: 'test', version: '1.0.0' }); const client2 = new Client({ name: 'test', version: '1.0.0' });
await client2.connect(transport2); await client2.connect(transport2);
const response = await client2.callTool({ const response = await client2.callTool({
@ -236,16 +228,15 @@ test('sse transport browser lifecycle (persistent, multiclient)', async ({ serve
await client2.close(); await client2.close();
}); });
test('streamable http transport', async ({ serverEndpoint }) => { test('sse transport', async ({ serverEndpoint }) => {
const { url } = await serverEndpoint(); const { url } = await serverEndpoint();
const transport = new StreamableHTTPClientTransport(new URL('/mcp', url)); const transport = new SSEClientTransport(new URL('/sse', url));
const client = new Client({ name: 'test', version: '1.0.0' }); const client = new Client({ name: 'test', version: '1.0.0' });
await client.connect(transport); await client.connect(transport);
await client.ping(); await client.ping();
expect(transport.sessionId, 'has session support').toBeDefined();
}); });
test('streamable http transport (default)', async ({ serverEndpoint }) => { test('http transport (default)', async ({ serverEndpoint }) => {
const { url } = await serverEndpoint(); const { url } = await serverEndpoint();
const transport = new StreamableHTTPClientTransport(url); const transport = new StreamableHTTPClientTransport(url);
const client = new Client({ name: 'test', version: '1.0.0' }); const client = new Client({ name: 'test', version: '1.0.0' });