ros/n8n-mcp
1
0
Fork 0
mirror of https://github.com/czlonkowski/n8n-mcp.git synced 2026-01-30 06:22:04 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(tests): remove get_node_for_task references from test suites

Browse Source 
Remove references to get_node_for_task tool that was removed in v2.15.0
as part of P0-R3 implementation.

Changes:
- parameter-validation.test.ts: Remove getNodeForTask mock spy
- parameter-validation.test.ts: Remove get_node_for_task from validation test array
- tools.test.ts: Remove get_node_for_task from templates category

Test results:
✅ parameter-validation.test.ts: 52/52 passed
✅ tools.test.ts: 57/57 passed

This completes the removal of get_node_for_task tool across the entire codebase.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
czlonkowski
2025-10-03 08:06:51 +02:00
parent 9fb847a16f
commit 0bf0e1cd74
2 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tests/unit/mcp/parameter-validation.test.ts
View File

@@ -145,7 +145,7 @@ describe('Parameter Validation', () => {
vi.spyOn(server as any, 'getNodeDocumentation').mockResolvedValue({ docs: 'test' });
vi.spyOn(server as any, 'getNodeEssentials').mockResolvedValue({ essentials: true });
vi.spyOn(server as any, 'searchNodeProperties').mockResolvedValue({ properties: [] });
vi.spyOn(server as any, 'getNodeForTask').mockResolvedValue({ node: 'test' });
// Note: getNodeForTask removed in v2.15.0
vi.spyOn(server as any, 'validateNodeConfig').mockResolvedValue({ valid: true });
vi.spyOn(server as any, 'validateNodeMinimal').mockResolvedValue({ missing: [] });
vi.spyOn(server as any, 'getPropertyDependencies').mockResolvedValue({ dependencies: {} });
@@ -477,7 +477,7 @@ describe('Parameter Validation', () => {
{ name: 'get_node_documentation', args: {}, expected: 'Missing required parameters for get_node_documentation: nodeType' },
{ name: 'get_node_essentials', args: {}, expected: 'Missing required parameters for get_node_essentials: nodeType' },
{ name: 'search_node_properties', args: {}, expected: 'Missing required parameters for search_node_properties: nodeType, query' },
{ name: 'get_node_for_task', args: {}, expected: 'Missing required parameters for get_node_for_task: task' },
// Note: get_node_for_task removed in v2.15.0
{ name: 'get_property_dependencies', args: {}, expected: 'Missing required parameters for get_property_dependencies: nodeType' },
{ name: 'get_node_as_tool_info', args: {}, expected: 'Missing required parameters for get_node_as_tool_info: nodeType' },
{ name: 'get_template', args: {}, expected: 'Missing required parameters for get_template: templateId' },

2
tests/unit/mcp/tools.test.ts
View File

@@ -254,7 +254,7 @@ describe('n8nDocumentationToolsFinal', () => {
discovery: ['list_nodes', 'search_nodes', 'list_ai_tools'],
configuration: ['get_node_info', 'get_node_essentials', 'get_node_documentation'],
validation: ['validate_node_operation', 'validate_workflow', 'validate_node_minimal'],
templates: ['list_tasks', 'get_node_for_task', 'search_templates', 'list_templates', 'get_template', 'list_node_templates'],
templates: ['list_tasks', 'search_templates', 'list_templates', 'get_template', 'list_node_templates'], // get_node_for_task removed in v2.15.0
documentation: ['tools_documentation']
};