Skip to content

Conversation

@afitzek
Copy link
Contributor

@afitzek afitzek commented Nov 20, 2025

Summary

#21438 split the test execution in the CI to speed it up. Some backend packages where missing the necessary npm script "test:unit". This reintroduces it.

These tests were missing and not executed in the CI:

Added test:unit scripts to 18 packages with passing tests:

Jest-based packages (12 packages):

  • @n8n/api-types (658 tests)
  • @n8n/backend-common (81 tests)
  • @n8n/client-oauth2 (32 tests)
  • @n8n/config (30 tests)
  • @n8n/db (154 tests)
  • @n8n/di (25 tests)
  • @n8n/json-schema-to-zod (76 tests)
  • @n8n/permissions (85 tests)
  • @n8n/stylelint-config (85 tests)
  • @n8n/task-runner (257 tests)
  • @n8n/codemirror-lang (41 tests)
  • @n8n/codemirror-lang-sql (42 tests)

Vitest-based packages (6 packages):

  • n8n-workflow (1,498 tests)
  • @n8n/eslint-config (49 tests)
  • @n8n/eslint-plugin-community-nodes (179 tests)
  • @n8n/imap (25 tests)
  • @n8n/node-cli (99 tests)
  • @n8n/utils (39 tests)

One package (@n8n/nodes-langchain) still has failing tests and was not updated.

The package n8n-nodes-base is now tested in its own check in the CI (#21836)

Related Linear tickets, Github issues, and Community forum posts

closes https://linear.app/n8n/issue/PAY-4155/check-all-backend-packages-for-the-npm-script-testunit

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 Nov 20, 2025
@afitzek afitzek changed the title chore(ci): Add "test:unit" scripts to most of the backend packages chore(core): Add "test:unit" scripts to most of the backend packages Nov 20, 2025
@bundlemon
Copy link

bundlemon bot commented Nov 20, 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.34MB (+46.52KB +0.4%) -
**/*.css
227.49KB (+6.72KB +3.04%) -

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@codecov
Copy link

codecov bot commented Nov 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@afitzek afitzek marked this pull request as ready for review November 20, 2025 13:01
@afitzek afitzek requested review from alexgrozav and ivov November 20, 2025 13:02
@afitzek afitzek force-pushed the pay-4155-check-all-backend-packages-for-the-npm-script-testunit branch from 3403952 to 9786826 Compare November 20, 2025 13:15
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 20 files

Copy link
Member

@ivov ivov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this

@currents-bot
Copy link

currents-bot bot commented Nov 20, 2025

E2E Tests: n8n tests passed after 11m 46.6s

🟢 583 · 🔴 0 · ⚪️ 12 · 🟣 2

View Run Details

Run Details

  • Project: n8n

  • Groups: 2

  • Framework: Playwright

  • Run Status: Passed

  • Commit: 9786826

  • Spec files: 95

  • Overall tests: 595

  • Duration: 11m 46.6s

  • Parallelization: 8

Groups

GroupId Results Spec Files Progress
ui 🟢 528 · 🔴 0 · ⚪️ 12 · 🟣 2 88 / 88
ui:isolated 🟢 55 · 🔴 0 · ⚪️ 0 7 / 7


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

@afitzek afitzek merged commit 8894383 into master Nov 20, 2025
69 of 71 checks passed
@afitzek afitzek deleted the pay-4155-check-all-backend-packages-for-the-npm-script-testunit branch November 20, 2025 14:28
@n8n-assistant
Copy link

n8n-assistant bot commented Nov 25, 2025

Got released with [email protected]

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 Released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants