From 5459107dcffa7096c14a5fca95be43c72655d35a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Romuald=20Cz=C5=82onkowski?= Date: Wed, 17 Dec 2025 17:06:51 +0100 Subject: [PATCH] fix: update workflow activation tests for n8n 2.0+ executeWorkflowTrigger MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - 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 --- tests/unit/services/workflow-diff-engine.test.ts | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/tests/unit/services/workflow-diff-engine.test.ts b/tests/unit/services/workflow-diff-engine.test.ts index 751e6e9..3082279 100644 --- a/tests/unit/services/workflow-diff-engine.test.ts +++ b/tests/unit/services/workflow-diff-engine.test.ts @@ -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); }); });