ros/n8n-mcp
1
0
Fork 0
mirror of https://github.com/czlonkowski/n8n-mcp.git synced 2026-02-06 13:33:11 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: update handlers-workflow-diff tests to use 'updates' property

Browse Source 
Fixed remaining test cases that were still using 'changes' instead of 'updates'
for updateNode operations. All tests now pass.
This commit is contained in:
czlonkowski
2025-09-17 23:29:17 +02:00
parent 17530c0f72
commit 44f92063c3
1 changed files with 9 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
tests/unit/mcp/handlers-workflow-diff.test.ts
View File

@@ -159,7 +159,7 @@ describe('handlers-workflow-diff', () => {
{ {
type: 'updateNode', type: 'updateNode',
nodeId: 'node2', nodeId: 'node2',
changes: { name: 'Updated HTTP Request' }, updates: { name: 'Updated HTTP Request' },
}, },
], ],
validateOnly: true, validateOnly: true,
@@ -196,7 +196,7 @@ describe('handlers-workflow-diff', () => {
{ {
type: 'updateNode', type: 'updateNode',
nodeId: 'node1', nodeId: 'node1',
changes: { name: 'Updated Start' }, updates: { name: 'Updated Start' },
}, },
{ {
type: 'addNode', type: 'addNode',
@@ -243,7 +243,7 @@ describe('handlers-workflow-diff', () => {
{ {
type: 'updateNode', type: 'updateNode',
nodeId: 'non-existent-node', nodeId: 'non-existent-node',
changes: { name: 'Updated' }, updates: { name: 'Updated' },
}, },
], ],
}; };
@@ -320,7 +320,7 @@ describe('handlers-workflow-diff', () => {
const result = await handleUpdatePartialWorkflow({ const result = await handleUpdatePartialWorkflow({
id: 'test-id', id: 'test-id',
operations: [{ type: 'updateNode', nodeId: 'node1', changes: {} }], operations: [{ type: 'updateNode', nodeId: 'node1', updates: {} }],
}); });
expect(result).toEqual({ expect(result).toEqual({
@@ -341,7 +341,7 @@ describe('handlers-workflow-diff', () => {
{ {
// Missing required 'type' field // Missing required 'type' field
nodeId: 'node1', nodeId: 'node1',
changes: {}, updates: {},
}, },
], ],
}; };
@@ -417,7 +417,7 @@ describe('handlers-workflow-diff', () => {
await handleUpdatePartialWorkflow({ await handleUpdatePartialWorkflow({
id: 'test-id', id: 'test-id',
operations: [{ type: 'updateNode', nodeId: 'node1', changes: {} }], operations: [{ type: 'updateNode', nodeId: 'node1', updates: {} }],
}); });
expect(logger.debug).toHaveBeenCalledWith( expect(logger.debug).toHaveBeenCalledWith(
@@ -502,7 +502,7 @@ describe('handlers-workflow-diff', () => {
type: 'updateNode', type: 'updateNode',
nodeId: 'node1', nodeId: 'node1',
nodeName: 'Start', // Both nodeId and nodeName provided nodeName: 'Start', // Both nodeId and nodeName provided
changes: { name: 'New Start' }, updates: { name: 'New Start' },
description: 'Update start node name', description: 'Update start node name',
}, },
{ {
@@ -561,8 +561,8 @@ describe('handlers-workflow-diff', () => {
const diffRequest = { const diffRequest = {
id: 'test-workflow-id', id: 'test-workflow-id',
operations: [ operations: [
{ type: 'updateNode', nodeId: 'node1', changes: { name: 'Updated' } }, { type: 'updateNode', nodeId: 'node1', updates: { name: 'Updated' } },
{ type: 'updateNode', nodeId: 'invalid-node', changes: { name: 'Fail' } }, { type: 'updateNode', nodeId: 'invalid-node', updates: { name: 'Fail' } },
{ type: 'addTag', tag: 'test' }, { type: 'addTag', tag: 'test' },
], ],
}; };