Skip to content
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

ci: Improve CI performance (no-changelog) #7637

Merged
merged 4 commits into from
Nov 7, 2023
Merged

Conversation

netroy
Copy link
Member

@netroy netroy commented Nov 7, 2023

  1. Split backend and frontend tests to avoid them running in parallel
  2. Enable coverage only on master

cstuncsik
cstuncsik previously approved these changes Nov 7, 2023
@n8n-assistant n8n-assistant bot added n8n team Authored by the n8n team ui Enhancement in /editor-ui or /design-system labels Nov 7, 2023
Copy link

cypress bot commented Nov 7, 2023

3 flaky tests on run #2751 ↗︎

0 271 0 0 Flakiness 3

Details:

🌳 🖥️ browsers:node18.12.0-chrome107 🤖 netroy 🗃️ e2e/*
Project: n8n Commit: b3b82694a8
Status: Passed Duration: 07:34 💡
Started: Nov 7, 2023 1:11 PM Ended: Nov 7, 2023 1:19 PM
Flakiness  28-resource-mapper.cy.ts • 1 flaky test

View Output Video

Test Artifacts
Resource Mapper > should correctly delete single field Output Screenshots Video
Flakiness  27-two-factor-authentication.cy.ts • 1 flaky test

View Output Video

Test Artifacts
Two-factor authentication > Should be able to login with MFA token Output Screenshots Video
Flakiness  29-sql-editor.cy.ts • 1 flaky test

View Output Video

Test Artifacts
SQL editors > should preserve changes when opening-closing Postgres node Output Screenshots Video

Review all test suite changes for PR #7637 ↗︎

Copy link
Contributor

github-actions bot commented Nov 7, 2023

✅ All Cypress E2E specs passed

Copy link
Contributor

github-actions bot commented Nov 7, 2023

✅ All Cypress E2E specs passed

@netroy netroy merged commit 2f66568 into master Nov 7, 2023
55 of 56 checks passed
@netroy netroy deleted the split-ci-test-workflow branch November 7, 2023 13:19
netroy added a commit that referenced this pull request Nov 7, 2023
…log) (#7641)

Disabled coverage in #7637, but forgot
to conditionally re-enable them for builds on `master`
MiloradFilipovic added a commit that referenced this pull request Nov 8, 2023
* master: (24 commits)
  fix(Crypto Node): Fix issue with value not appearing for Sign action (#7619)
  feat(NocoDB Node): Add new data apis and workspace support (#7329)
  fix(Facebook Lead Ads Trigger Node): Fix issue with missing scope for business management  (#7616)
  refactor(core): Stop reporting to Sentry missing-node-on-retry error (no-changelog) (#7648)
  fix(editor): Fix workflow history prune time limit (getting hours instead of days) (#7644)
  fix(core): Comply with custom default for workflow saving settings (#7634)
  feat(core): Initial support for two-way communication over websockets (#7570)
  ci: Conditionally re-enable coverage for frontend packages (no-changelog) (#7641)
  ci(core): Load config schema after process.env has been overwritten (no-changelog) (#7550)
  fix(core): Make password-reset urls valid only for single-use (#7622)
  fix: Error handling on forgot password page (no-changelog) (#7633)
  ci: Improve CI performance (no-changelog) (#7637)
  fix(core): Ensure pruning starts only after migrations have completed (#7626)
  feat(core): Coordinate workflow activation in multiple main scenario in internal API (#7566)
  fix(editor): Fix local storage flags defaulting to undefined string (#7603)
  fix(editor): Reset canvas zoom before workspace reset in node view (#7625)
  fix(editor): More dark-mode fixes (no-changelog) (#7624)
  fix(core): Fix accessor error when running partial execution (#7618)
  fix(editor): Fix issue that frontend breaks with unkown nodes (#7596)
  fix(core): Ensure `init` before checking leader or follower in multi-main scenario (#7621)
  ...
@janober
Copy link
Member

janober commented Nov 8, 2023

Got released with n8n@1.16.0

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 Released ui Enhancement in /editor-ui or /design-system
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants