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

fix: update workflow activation tests for n8n 2.0+ executeWorkflowTrigger

Browse Source 
- Update test to expect SUCCESS for executeWorkflowTrigger-only workflows
- Remove outdated assertion about "executeWorkflowTrigger cannot activate"
- executeWorkflowTrigger is now a valid activatable trigger in n8n 2.0+

Conceived by Romuald Członkowski - https://www.aiadvisors.pl/en

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Romuald Członkowski
2025-12-17 17:06:51 +01:00
parent fa89d2f18e
commit 5459107dcf
1 changed files with 5 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
tests/unit/services/workflow-diff-engine.test.ts
View File

@@ -4402,7 +4402,6 @@ describe('WorkflowDiffEngine', () => {
expect(result.success).toBe(false);
expect(result.errors).toBeDefined();
expect(result.errors![0].message).toContain('No activatable trigger nodes found');
expect(result.errors![0].message).toContain('executeWorkflowTrigger cannot activate workflows');
});
it('should reject activation if all trigger nodes are disabled', async () => {
@@ -4615,8 +4614,8 @@ describe('WorkflowDiffEngine', () => {
expect(result.shouldActivate).toBe(true);
});
it('should reject activation if workflow has executeWorkflowTrigger only', async () => {
// Create workflow with executeWorkflowTrigger (not activatable - Issue #351)
it('should allow activation if workflow has executeWorkflowTrigger only (n8n 2.0+)', async () => {
// Create workflow with executeWorkflowTrigger (activatable since n8n 2.0+)
const workflowWithExecuteTrigger = createWorkflow('Test Workflow')
.addNode({
id: 'execute-1',
@@ -4659,10 +4658,9 @@ describe('WorkflowDiffEngine', () => {
const result = await diffEngine.applyDiff(workflowWithExecuteTrigger, request);
expect(result.success).toBe(false);
expect(result.errors).toBeDefined();
expect(result.errors![0].message).toContain('No activatable trigger nodes found');
expect(result.errors![0].message).toContain('executeWorkflowTrigger cannot activate workflows');
// executeWorkflowTrigger is now activatable in n8n 2.0+
expect(result.success).toBe(true);
expect(result.shouldActivate).toBe(true);
});
});