diff options
| author | N. Taylor Mullen <[email protected]> | 2025-08-06 16:59:50 -0700 |
|---|---|---|
| committer | GitHub <[email protected]> | 2025-08-06 23:59:50 +0000 |
| commit | 01f7c4b7406cd9ce94667dcbd5e586dad9573d67 (patch) | |
| tree | 93fd9a76f1f8886674fa12e2170a71ba92588622 /integration-tests | |
| parent | 9ac3e8b79ecc584805c27d3602612c30f2adee80 (diff) | |
Fix(tests): update mcp_server_cyclic_schema test (#5733)
Diffstat (limited to 'integration-tests')
| -rw-r--r-- | integration-tests/mcp_server_cyclic_schema.test.js | 13 |
1 files changed, 3 insertions, 10 deletions
diff --git a/integration-tests/mcp_server_cyclic_schema.test.js b/integration-tests/mcp_server_cyclic_schema.test.js index a78e0922..1ace98f1 100644 --- a/integration-tests/mcp_server_cyclic_schema.test.js +++ b/integration-tests/mcp_server_cyclic_schema.test.js @@ -14,7 +14,7 @@ import { strict as assert } from 'node:assert'; import { TestRig } from './test-helper.js'; import { join } from 'path'; import { fileURLToPath } from 'url'; -import { writeFileSync, readFileSync } from 'fs'; +import { writeFileSync } from 'fs'; const __dirname = fileURLToPath(new URL('.', import.meta.url)); @@ -191,16 +191,9 @@ describe('mcp server with cyclic tool schema is detected', () => { // Or, possibly it could mean that gemini has fixed the issue. const output = await rig.run('hello'); - // The error message is in a log file, so we need to extract the path and read it. - const match = output.match(/Full report available at: (.*\.json)/); - assert(match, `Could not find log file path in output: ${output}`); - - const logFilePath = match[1]; - const logFileContent = readFileSync(logFilePath, 'utf-8'); - assert.match( - logFileContent, - / - tool_with_cyclic_schema \(cyclic-schema-server MCP Server\)/, + output, + /Skipping tool 'tool_with_cyclic_schema' from MCP server 'cyclic-schema-server' because it has missing types in its parameter schema/, ); }); }); |
