Skip to content

Conversation

@cstuncsik
Copy link
Contributor

Summary

Related Linear tickets, Github issues, and Community forum posts

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

This fix addresses PAY-2559 where pinned data was losing paired item information.

Changes:
1. workflows.store.ts: Store full item object in pinData instead of just json/binary
   - Changed from: { json: item.json, ...(item.binary && { binary: item.binary }) }
   - Changed to: item (when isJsonKeyObject)
   - This preserves pairedItem data along with json and binary

2. workflow-data-proxy.ts: Improve error messaging for paired item failures
   - Replaced createInvalidPairedItemError with createMissingPairedItemError
   - Provides better guidance when item pairing fails with pinned data
   - Removed unused createInvalidPairedItemError function

3. execution-base.error.ts: Fix name property assignment
   - Use Object.defineProperty to avoid "Cannot assign to read only property" error
   - More robust than direct assignment

4. Updated test to expect new error message and type

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

Co-Authored-By: Claude <[email protected]>
@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Oct 31, 2025
@bundlemon
Copy link

bundlemon bot commented Oct 31, 2025

BundleMon

Unchanged files (2)
Status Path Size Limits
WASM Dependencies
tree-sitter-bash.wasm
181.26KB -
WASM Dependencies
tree-sitter.wasm
74.47KB -

No change in files bundle size

Groups updated (2)
Status Path Size Limits
**/*.js
5.88MB (+476.63KB +8.59%) -
**/*.css
218.13KB (+4.87KB +2.28%) -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@codecov
Copy link

codecov bot commented Oct 31, 2025

❌ 11 Tests Failed:

Tests completed Failed Passed Skipped
13308 11 13297 1
View the top 3 failed test(s) by shortest run time
SubworkflowPolicyChecker `workflows-from-list` caller policy should deny if caller list does not contain parent workflow ID
Stack Traces | 0.001s run time
Error: expect(received).rejects.toThrowError(expected)

Expected constructor: SubworkflowPolicyDenialError
Received constructor: TypeError

Received message: "Cannot assign to read only property 'name' of object 'SubworkflowPolicyDenialError: The sub-workflow (subworkflow-id) cannot be called by this workflow'"

      13 | 		super(message, { cause: undefined });
      14 | 		this.level = 'warning';
    > 15 | 		this.name = this.constructor.name;
         | 		         ^
      16 | 		if (description) this.description = description;
      17 | 		this.node = node;
      18 | 		this.timestamp = Date.now();

      at new WorkflowOperationError (...../src/errors/workflow-operation.error.ts:15:12)
      at new SubworkflowPolicyDenialError (src/errors/subworkflow-policy-denial.error.ts:61:3)
      at SubworkflowPolicyChecker.check (.../executions/pre-execution-checks/subworkflow-policy-checker.ts:52:9)
      at Object.<anonymous> (.../pre-execution-checks/__tests__/subworkflow-policy-checker.test.ts:96:4)
    at Object.toThrowError (.../n8n/node_modules/.pnpm/[email protected]..../expect/build/index.js:218:22)
    at Object.<anonymous> (/home/runner/_work/n8n/n8n/packages/cli/.../pre-execution-checks/__tests__/subworkflow-policy-checker.test.ts:96:32)
    at Promise.then.completed (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:300:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:233:10)
    at _callCircusTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:315:40)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at _runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:251:3)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:125:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at run (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:70:3)
    at runAndTransformResultsToJestFormat (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:367:16)
    at runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/testWorker.js:106:12)
SubworkflowPolicyChecker `workflows-from-same-owner` caller policy should deny if the two workflows are owned by different projects
Stack Traces | 0.001s run time
Error: expect(received).rejects.toThrowError(expected)

Expected constructor: SubworkflowPolicyDenialError
Received constructor: TypeError

Received message: "Cannot assign to read only property 'name' of object 'SubworkflowPolicyDenialError: The sub-workflow (subworkflow-id) cannot be called by this workflow'"

      13 | 		super(message, { cause: undefined });
      14 | 		this.level = 'warning';
    > 15 | 		this.name = this.constructor.name;
         | 		         ^
      16 | 		if (description) this.description = description;
      17 | 		this.node = node;
      18 | 		this.timestamp = Date.now();

      at new WorkflowOperationError (...../src/errors/workflow-operation.error.ts:15:12)
      at new SubworkflowPolicyDenialError (src/errors/subworkflow-policy-denial.error.ts:61:3)
      at SubworkflowPolicyChecker.check (.../executions/pre-execution-checks/subworkflow-policy-checker.ts:52:9)
      at Object.<anonymous> (.../pre-execution-checks/__tests__/subworkflow-policy-checker.test.ts:132:4)
    at Object.toThrowError (.../n8n/node_modules/.pnpm/[email protected]..../expect/build/index.js:218:22)
    at Object.<anonymous> (/home/runner/_work/n8n/n8n/packages/cli/.../pre-execution-checks/__tests__/subworkflow-policy-checker.test.ts:132:32)
    at Promise.then.completed (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:300:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:233:10)
    at _callCircusTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:315:40)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at _runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:251:3)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:125:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at run (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:70:3)
    at runAndTransformResultsToJestFormat (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:367:16)
    at runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/testWorker.js:106:12)
SubworkflowPolicyChecker error details should contain description for default (e.g. error workflow) case
Stack Traces | 0.001s run time
Error: expect(received).rejects.toMatchObject(expected)

- Expected  - 4
+ Received  + 1

- Object {
-   "description": "The sub-workflow you’re trying to execute limits which workflows it can be called by.",
-   "message": "The sub-workflow (subworkflow-id) cannot be called by this workflow",
- }
+ [TypeError: Cannot assign to read only property 'name' of object 'SubworkflowPolicyDenialError: The sub-workflow (subworkflow-id) cannot be called by this workflow']
    at Object.toMatchObject (.../n8n/node_modules/.pnpm/[email protected]..../expect/build/index.js:218:22)
    at Object.<anonymous> (.../pre-execution-checks/__tests__/subworkflow-policy-checker.test.ts:257:32)
    at Promise.then.completed (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:300:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:233:10)
    at _callCircusTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:315:40)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at _runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:251:3)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:125:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at run (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:70:3)
    at runAndTransformResultsToJestFormat (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:367:16)
    at runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/testWorker.js:106:12)
SubworkflowPolicyChecker error details should contain description for inaccessible personal project case
Stack Traces | 0.001s run time
Error: expect(received).rejects.toMatchObject(expected)

- Expected  - 4
+ Received  + 1

- Object {
-   "description": "The sub-workflow you’re trying to execute limits which workflows it can be called by. You will need John Doe to update the sub-workflow (subworkflow-id) settings to allow this workflow to call it.",
-   "message": "The sub-workflow (subworkflow-id) cannot be called by this workflow",
- }
+ [TypeError: Cannot assign to read only property 'name' of object 'SubworkflowPolicyDenialError: The sub-workflow (subworkflow-id) cannot be called by this workflow']
    at Object.toMatchObject (.../n8n/node_modules/.pnpm/[email protected]..../expect/build/index.js:218:22)
    at Object.<anonymous> (.../pre-execution-checks/__tests__/subworkflow-policy-checker.test.ts:204:32)
    at Promise.then.completed (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:300:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:233:10)
    at _callCircusTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:315:40)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at _runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:251:3)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:125:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at run (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:70:3)
    at runAndTransformResultsToJestFormat (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:367:16)
    at runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/testWorker.js:106:12)
SubworkflowPolicyChecker error details should contain description for inaccessible team project case
Stack Traces | 0.001s run time
Error: expect(received).rejects.toMatchObject(expected)

- Expected  - 6
+ Received  + 1

- Object {
-   "description": "The sub-workflow you’re trying to execute limits which workflows it can be called by. You will need an admin from the function () {
-         return fn.apply(this, arguments);
-       } project to update the sub-workflow (subworkflow-id) settings to allow this workflow to call it.",
-   "message": "The sub-workflow (subworkflow-id) cannot be called by this workflow",
- }
+ [TypeError: Cannot assign to read only property 'name' of object 'SubworkflowPolicyDenialError: The sub-workflow (subworkflow-id) cannot be called by this workflow']
    at Object.toMatchObject (.../n8n/node_modules/.pnpm/[email protected]..../expect/build/index.js:218:22)
    at Object.<anonymous> (.../pre-execution-checks/__tests__/subworkflow-policy-checker.test.ts:235:32)
    at Promise.then.completed (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:300:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:233:10)
    at _callCircusTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:315:40)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at _runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:251:3)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:125:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at run (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:70:3)
    at runAndTransformResultsToJestFormat (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:367:16)
    at runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/testWorker.js:106:12)
WorkflowExecute Cancellation should set status to canceled when execution timeout is reached
Stack Traces | 0.001s run time
Error: expect(received).toBe(expected) // Object.is equality

Expected: "TimeoutExecutionCancelledError"
Received: undefined
    at Object.<anonymous> (.../execution-engine/__tests__/workflow-execute.test.ts:2637:47)
SubworkflowPolicyChecker error details should contain description for accessible case
Stack Traces | 0.002s run time
Error: expect(received).rejects.toMatchObject(expected)

- Expected  - 4
+ Received  + 1

- Object {
-   "description": "The sub-workflow you’re trying to execute limits which workflows it can be called by. <a href=\"https://n8n.test.com/workflow/subworkflow-id\" target=\"_blank\">Update sub-workflow settings</a> to allow other workflows to call it.",
-   "message": "The sub-workflow (subworkflow-id) cannot be called by this workflow",
- }
+ [TypeError: Cannot assign to read only property 'name' of object 'SubworkflowPolicyDenialError: The sub-workflow (subworkflow-id) cannot be called by this workflow']
    at Object.toMatchObject (.../n8n/node_modules/.pnpm/[email protected]..../expect/build/index.js:218:22)
    at Object.<anonymous> (.../pre-execution-checks/__tests__/subworkflow-policy-checker.test.ts:176:32)
    at Promise.then.completed (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:300:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:233:10)
    at _callCircusTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:315:40)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at _runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:251:3)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:125:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at run (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:70:3)
    at runAndTransformResultsToJestFormat (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:367:16)
    at runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/testWorker.js:106:12)
ExecutionService stop should throw when stopping a not-in-progress execution
Stack Traces | 0.007s run time
Error: expect(received).rejects.toThrowError(expected)

Expected constructor: WorkflowOperationError
Received constructor: TypeError

Received message: "Cannot assign to read only property 'name' of object 'WorkflowOperationError: Only running or waiting executions can be stopped and 123 is currently success'"

      13 | 		super(message, { cause: undefined });
      14 | 		this.level = 'warning';
    > 15 | 		this.name = this.constructor.name;
         | 		         ^
      16 | 		if (description) this.description = description;
      17 | 		this.node = node;
      18 | 		this.timestamp = Date.now();

      at new WorkflowOperationError (...../src/errors/workflow-operation.error.ts:15:12)
      at ExecutionService.assertStoppable (src/executions/execution.service.ts:538:10)
      at ExecutionService.assertStoppable [as stop] (src/executions/execution.service.ts:518:8)
      at Object.<anonymous> (.../executions/__tests__/execution.service.test.ts:165:4)
    at Object.toThrowError (.../n8n/node_modules/.pnpm/[email protected]..../expect/build/index.js:218:22)
    at Object.<anonymous> (.../executions/__tests__/execution.service.test.ts:165:31)
    at Promise.then.completed (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:300:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:233:10)
    at _callCircusTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:315:40)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at _runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:251:3)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:125:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at run (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:70:3)
    at runAndTransformResultsToJestFormat (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:367:16)
    at runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/testWorker.js:106:12)
ExecutionRecoveryService recoverFromLogs if leader, with 1+ messages for running execution, should update `status`, `stoppedAt` and `data` if last node did not finish
Stack Traces | 0.008s run time
TypeError: Cannot assign to read only property 'name' of object 'WorkflowCrashedError: Workflow did not finish, possible out-of-memory issue'
    at new WorkflowOperationError (.../src/errors/workflow-operation.error.ts:15:12)
    at new WorkflowCrashedError (.../src/errors/workflow-crashed.error.ts:5:3)
    at ExecutionRecoveryService.amend (.../src/executions/execution-recovery.service.ts:122:41)
    at ExecutionRecoveryService.recoverFromLogs (.../src/executions/execution-recovery.service.ts:36:28)
    at Object.<anonymous> (.../executions/__tests__/execution-recovery.service.integration.test.ts:276:30)
processRunExecutionData workflow issues throws if workflow contains nodes with missing required properties
Stack Traces | 0.011s run time
Error: expect(received).toThrowError(expected)

Expected message: "The workflow has issues and cannot be executed for that reason. Please fix them first."
Received message: "Cannot assign to read only property 'name' of object 'WorkflowHasIssuesError: The workflow has issues and cannot be executed for that reason. Please fix them first.'"

      13 | 		super(message, { cause: undefined });
      14 | 		this.level = 'warning';
    > 15 | 		this.name = this.constructor.name;
         | 		         ^
      16 | 		if (description) this.description = description;
      17 | 		this.node = node;
      18 | 		this.timestamp = Date.now();

      at new WorkflowOperationError (...../src/errors/workflow-operation.error.ts:15:12)
      at new WorkflowHasIssuesError (src/errors/workflow-has-issues.error.ts:5:3)
      at WorkflowExecute.checkForWorkflowIssues (src/execution-engine/workflow-execute.ts:1382:10)
      at WorkflowExecute.checkForWorkflowIssues [as processRunExecutionData] (src/execution-engine/workflow-execute.ts:1461:8)
      at .../execution-engine/__tests__/workflow-execute-process-process-run-execution-data.test.ts:266:33
      at Object.<anonymous> (../../node_modules/.pnpm/[email protected]..../expect/build/toThrowMatchers.js:74:11)
      at Object.throwingMatcher [as toThrowError] (../../node_modules/.pnpm/[email protected]..../expect/build/index.js:320:21)
      at Object.<anonymous> (.../execution-engine/__tests__/workflow-execute-process-process-run-execution-data.test.ts:266:68)
    at Object.<anonymous> (/home/runner/_work/n8n/n8n/packages/core/.../execution-engine/__tests__/workflow-execute-process-process-run-execution-data.test.ts:266:68)
    at Promise.then.completed (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:300:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:233:10)
    at _callCircusTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:315:40)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at _runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:251:3)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:125:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at run (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:70:3)
    at runAndTransformResultsToJestFormat (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:367:16)
    at runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/testWorker.js:106:12)
SubworkflowPolicyChecker no caller policy should fall back to `N8N_WORKFLOW_CALLER_POLICY_DEFAULT_OPTION`
Stack Traces | 0.015s run time
Error: expect(received).rejects.toThrowError(expected)

Expected constructor: SubworkflowPolicyDenialError
Received constructor: TypeError

Received message: "Cannot assign to read only property 'name' of object 'SubworkflowPolicyDenialError: The sub-workflow (subworkflow-id) cannot be called by this workflow'"

      13 | 		super(message, { cause: undefined });
      14 | 		this.level = 'warning';
    > 15 | 		this.name = this.constructor.name;
         | 		         ^
      16 | 		if (description) this.description = description;
      17 | 		this.node = node;
      18 | 		this.timestamp = Date.now();

      at new WorkflowOperationError (...../src/errors/workflow-operation.error.ts:15:12)
      at new SubworkflowPolicyDenialError (src/errors/subworkflow-policy-denial.error.ts:61:3)
      at SubworkflowPolicyChecker.check (.../executions/pre-execution-checks/subworkflow-policy-checker.ts:52:9)
      at Object.<anonymous> (.../pre-execution-checks/__tests__/subworkflow-policy-checker.test.ts:53:4)
    at Object.toThrowError (.../n8n/node_modules/.pnpm/[email protected]..../expect/build/index.js:218:22)
    at Object.<anonymous> (/home/runner/_work/n8n/n8n/packages/cli/.../pre-execution-checks/__tests__/subworkflow-policy-checker.test.ts:53:32)
    at Promise.then.completed (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:300:28)
    at new Promise (<anonymous>)
    at callAsyncCircusFn (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/utils.js:233:10)
    at _callCircusTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:315:40)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at _runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:251:3)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:125:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at _runTestsForDescribeBlock (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:120:9)
    at run (.../n8n/node_modules/.pnpm/[email protected]..../jest-circus/build/run.js:70:3)
    at runAndTransformResultsToJestFormat (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapterInit.js:122:21)
    at jestAdapter (.../n8n/node_modules/.pnpm/[email protected]..../build/legacy-code-todo-rewrite/jestAdapter.js:79:19)
    at runTestInternal (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:367:16)
    at runTest (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/runTest.js:444:34)
    at Object.worker (.../n8n/node_modules/.pnpm/[email protected]..../jest-runner/build/testWorker.js:106:12)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@blacksmith-sh
Copy link

blacksmith-sh bot commented Oct 31, 2025

Found 11 test failures on Blacksmith runners:

Test View Logs
ExecutionRecoveryService/
ExecutionRecoveryService recoverFromLogs if leader, with 1+ messages for running execut
ion, should update status, stoppedAt and data if last node did not finish
View Logs
ExecutionService/
ExecutionService stop should throw when stopping a not-in-progress execution
View Logs
processRunExecutionData/
processRunExecutionData workflow issues throws if workflow contains nodes with missing
required properties
View Logs
SubworkflowPolicyChecker/
SubworkflowPolicyChecker workflows-from-list caller policy should deny if caller list
does not contain parent workflow ID
View Logs
SubworkflowPolicyChecker/
SubworkflowPolicyChecker workflows-from-same-owner caller policy should deny if the t
wo workflows are owned by different projects
View Logs
SubworkflowPolicyChecker/
SubworkflowPolicyChecker error details should contain description for accessible case
View Logs
SubworkflowPolicyChecker/
SubworkflowPolicyChecker error details should contain description for default (e.g. err
or workflow) case
View Logs
SubworkflowPolicyChecker/
SubworkflowPolicyChecker error details should contain description for inaccessible pers
onal project case
View Logs
SubworkflowPolicyChecker/
SubworkflowPolicyChecker error details should contain description for inaccessible team
project case
View Logs
SubworkflowPolicyChecker/
SubworkflowPolicyChecker no caller policy should fall back to N8N_WORKFLOW_CALLER_POLI</code><br /><code>CY_DEFAULT_OPTION
View Logs
WorkflowExecute/
WorkflowExecute Cancellation should set status to canceled when execution timeout is re
ached
View Logs


Fix in Cursor

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