Skip to content

fix(core): Increase timeout for workflow execution tests#2096

Draft
everettbu wants to merge 1 commit into
masterfrom
shortstacked/flaky-retry-timeout
Draft

fix(core): Increase timeout for workflow execution tests#2096
everettbu wants to merge 1 commit into
masterfrom
shortstacked/flaky-retry-timeout

Conversation

@everettbu

Copy link
Copy Markdown

Mirror of n8n-io/n8n#26150
Original author: shortstacked


Summary

  • Increases the Jest timeout for run test workflows tests from the default 5s to 15s
  • The retry_and_continue_on_error test executes ~46 Code node sandbox instances with retries, which intermittently exceeds the 5s default on slow CI runners

Root cause

The workflow test JSON exercises 31 nodes (15 Reset Count + 16 Retry/Continue Code nodes) with retryOnFail: true and waitBetweenTries: 10ms. Each Code node creates a JavaScriptSandbox VM context. The cumulative sandbox creation + execution time is ~3.5s locally (cold) and can exceed 5s on shared CI runners under load.

Test plan

  • Verified test passes locally with new timeout
  • Linting passes (biome pre-commit hook)

🤖 Generated with Claude Code

The retry_and_continue_on_error test executes ~46 Code node sandboxes
with retries, which can exceed the default 5s Jest timeout on slow CI
runners.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@everettbu everettbu added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants