-
Notifications
You must be signed in to change notification settings - Fork 51.1k
chore(core): Add "test:unit" scripts to most of the backend packages #22091
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(core): Add "test:unit" scripts to most of the backend packages #22091
Conversation
BundleMonUnchanged files (2)
No change in files bundle size Groups updated (2)
Final result: ✅ View report in BundleMon website ➡️ |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
3403952 to
9786826
Compare
There was a problem hiding this 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
ivov
left a comment
There was a problem hiding this 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
|
E2E Tests: n8n tests passed after 11m 46.6s Run Details
Groups
This message was posted automatically by
currents.dev | Integration Settings
|
|
Got released with |
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):
Vitest-based packages (6 packages):
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
release/backport(if the PR is an urgent fix that needs to be backported)