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

Fix redundant conditional logic in browser_evaluate error test

Browse Source 
Co-authored-by: pavelfeldman <883973+pavelfeldman@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot] 2025-07-18 22:16:55 +00:00
parent d9d5e3f7ed
commit 47a73de357
1 changed files with 2 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
tests/evaluate.spec.ts
View File

@ -50,7 +50,7 @@ test('browser_evaluate (element)', async ({ client, server }) => {
})).toContainTextContent(`- Result: "red"`);
});
test('browser_evaluate (error)', async ({ client, server, mcpBrowser }) => {
test('browser_evaluate (error)', async ({ client, server }) => {
expect(await client.callTool({
name: 'browser_navigate',
arguments: { url: server.HELLO_WORLD },
@ -68,11 +68,5 @@ test('browser_evaluate (error)', async ({ client, server, mcpBrowser }) => {
expect(result.isError).toBe(true);
// Check that JavaScript error details are contained in the response
if (mcpBrowser === 'webkit') {
// Webkit has different error message format
expect(result.content?.[0].text).toContain('undefinedVariable is not defined');
} else {
// Chrome, Firefox, etc. include the Playwright evaluation context
expect(result.content?.[0].text).toContain('undefinedVariable is not defined');
}
expect(result.content?.[0].text).toContain('undefinedVariable is not defined');
});