Skip to content

Conversation

@Cadiac
Copy link
Contributor

@Cadiac Cadiac commented Dec 5, 2025

…ngelog)

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)

@n8n-assistant n8n-assistant bot added core Enhancement outside /nodes-base and /editor-ui n8n team Authored by the n8n team labels Dec 5, 2025
@bundlemon
Copy link

bundlemon bot commented Dec 5, 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
11.43MB (+146.57KB +1.27%) -
**/*.css
233.39KB (+12.62KB +5.72%) -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@codecov
Copy link

codecov bot commented Dec 5, 2025

❌ 4 Tests Failed:

Tests completed Failed Passed Skipped
15369 4 15365 0
View the top 3 failed test(s) by shortest run time
Scope Information ensure scopes are defined correctly
Stack Traces | 0.054s run time
Error: expect(received).toMatchSnapshot()

Snapshot name: `Scope Information ensure scopes are defined correctly 1`

- Snapshot  - 0
+ Received  + 1

@@ -99,10 +99,11 @@
    "workflow:share",
    "workflow:execute",
    "workflow:move",
    "workflow:activate",
    "workflow:deactivate",
+   "workflow:chat",
    "workflow:create",
    "workflow:read",
    "workflow:update",
    "workflow:delete",
    "workflow:list",
    at Object.<anonymous> (.../n8n/n8n/packages/@.../src/__tests__/scope-information.test.ts:5:22)
    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 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)
POST /workflows should return scopes on created workflow
Stack Traces | 0.103s run time
Error: expect(received).toEqual(expected) // deep equality

- Expected  - 0
+ Received  + 1

@@ -1,6 +1,7 @@
  Array [
+   "workflow:chat",
    "workflow:delete",
    "workflow:execute",
    "workflow:move",
    "workflow:read",
    "workflow:share",
    at Object.<anonymous> (.../integration/workflows/workflows.controller.test.ts:165:18)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
GET /workflows should return workflows with scopes when ?includeScopes=true
Stack Traces | 0.11s run time
Error: expect(received).toEqual(expected) // deep equality

- Expected  - 0
+ Received  + 1

@@ -1,6 +1,7 @@
  Array [
+   "workflow:chat",
    "workflow:delete",
    "workflow:execute",
    "workflow:move",
    "workflow:read",
    "workflow:update",
    at Object.<anonymous> (.../integration/workflows/workflows.controller.test.ts:809:23)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)
GET /workflows?includeFolders=true should return workflows with scopes and folders when ?includeScopes=true
Stack Traces | 0.119s run time
Error: expect(received).toEqual(expected) // deep equality

- Expected  - 0
+ Received  + 1

@@ -1,6 +1,7 @@
  Array [
+   "workflow:chat",
    "workflow:delete",
    "workflow:execute",
    "workflow:move",
    "workflow:read",
    "workflow:update",
    at Object.<anonymous> (.../integration/workflows/workflows.controller.test.ts:1851:23)
    at processTicksAndRejections (node:internal/process/task_queues:105:5)

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 Dec 5, 2025

Found 4 test failures on Blacksmith runners:

Failures

Test View Logs
POST /workflows/GET /
workflows should return workflows with scopes when ?includeScopes=true
View Logs
POST /workflows/GET /
workflows?includeFolders=true should return workflows with scopes and folders when ?inc
ludeScopes=true
View Logs
POST /workflows/POST /workflows should return scopes on created workflow View Logs
Scope Information/Scope Information ensure scopes are defined correctly View Logs

Fix in Cursor

@currents-bot
Copy link

currents-bot bot commented Dec 5, 2025

E2E Tests: n8n tests passed after 10m 59.5s

🟢 573 · 🔴 0 · ⚪️ 37 · 🟣 2

View Run Details

Run Details

  • Project: n8n

  • Groups: 2

  • Framework: Playwright

  • Run Status: Passed

  • Commit: c444f31

  • Spec files: 122

  • Overall tests: 610

  • Duration: 10m 59.5s

  • Parallelization: 9

Groups

GroupId Results Spec Files Progress
multi-main:ui:isolated 🟢 54 · 🔴 0 · ⚪️ 0 8 / 8
multi-main:ui 🟢 519 · 🔴 0 · ⚪️ 37 · 🟣 2 114 / 114


This message was posted automatically by currents.dev | Integration Settings

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