Skip to content

Conversation

CharlieKolb
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)

@CharlieKolb CharlieKolb changed the base branch from master to ADO-3852 August 5, 2025 13:47
@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team node/new Creation of an entirely new node labels Aug 5, 2025
Copy link

codecov bot commented Aug 5, 2025

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
9548 2 9546 0
View the top 2 failed test(s) by shortest run time
JobProcessor should use manualExecutionService to process a job in "evaluation" mode
Stack Traces | 0.001s run time
TypeError: Cannot read properties of undefined (reading 'manager')
    at new manager (.../modules/data-store/data-store.repository.ts:21:31)
    at ContainerClass.get (.../n8n/n8n/packages/@.../di/src/di.ts:104:16)
    at .../n8n/n8n/packages/@.../di/src/di.ts:97:17
    at Array.map (<anonymous>)
    at ContainerClass.get (.../n8n/n8n/packages/@.../di/src/di.ts:91:36)
    at .../n8n/n8n/packages/@.../di/src/di.ts:97:17
    at Array.map (<anonymous>)
    at ContainerClass.get (.../n8n/n8n/packages/@.../di/src/di.ts:91:36)
    at Object.get [as getBase] (.../n8n/n8n/packages/cli/src/workflow-execute-additional-data.ts:382:29)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at JobProcessor.processJob (.../n8n/n8n/packages/cli/src/scaling/job-processor.ts:120:26)
    at .../n8n/n8n/packages/cli/src/scaling/__tests__/job-processor.service.test.ts:97:4
JobProcessor should use manualExecutionService to process a job in "manual" mode
Stack Traces | 0.002s run time
TypeError: Cannot read properties of undefined (reading 'manager')
    at new manager (.../modules/data-store/data-store.repository.ts:21:31)
    at ContainerClass.get (.../n8n/n8n/packages/@.../di/src/di.ts:104:16)
    at .../n8n/n8n/packages/@.../di/src/di.ts:97:17
    at Array.map (<anonymous>)
    at ContainerClass.get (.../n8n/n8n/packages/@.../di/src/di.ts:91:36)
    at .../n8n/n8n/packages/@.../di/src/di.ts:97:17
    at Array.map (<anonymous>)
    at ContainerClass.get (.../n8n/n8n/packages/@.../di/src/di.ts:91:36)
    at Object.get [as getBase] (.../n8n/n8n/packages/cli/src/workflow-execute-additional-data.ts:382:29)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
    at JobProcessor.processJob (.../n8n/n8n/packages/cli/src/scaling/job-processor.ts:120:26)
    at .../n8n/n8n/packages/cli/src/scaling/__tests__/job-processor.service.test.ts:97:4

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

@dariacodes dariacodes force-pushed the ADO-3852 branch 3 times, most recently from a066188 to 33cb851 Compare August 14, 2025 09:12
Base automatically changed from ADO-3852 to master August 19, 2025 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
n8n team Authored by the n8n team node/new Creation of an entirely new node
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants