perf(core): Cache roles

[ivov]

https://linear.app/n8n/issue/PAY-666/cache-roles-queries

Latest e2e run: https://github.com/n8n-io/n8n/actions/runs/5740482744

[github-actions]

Great PR! Please pay attention to the following items before merging:

Files matching packages/**:

• If fixing bug, added test to cover scenario.
• If addressing forum or Github issue, added link to description.

Files matching packages/**/*.ts:

• Added unit tests to cover new or updated functionality.

Make sure to check off this list before asking for review.

[codecov]

Codecov Report

Patch coverage: 67.74% and no project coverage change.

Comparison is base (659ca26) 24.76% compared to head (ed1cdc8) 24.77%.
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #6803   +/-   ##
=======================================
  Coverage   24.76%   24.77%           
=======================================
  Files        3131     3130    -1     
  Lines      190809   190825   +16     
  Branches    21013    21020    +7     
=======================================
+ Hits        47262    47269    +7     
- Misses     142591   142598    +7     
- Partials      956      958    +2     

Files Changed	Coverage Δ
packages/cli/src/Server.ts	0.00% <0.00%> (ø)
packages/cli/src/commands/import/credentials.ts	0.00% <0.00%> (ø)
packages/cli/src/controllers/e2e.controller.ts	0.00% <0.00%> (ø)
...ts/sourceControl/sourceControlImport.service.ee.ts	9.31% <12.50%> (+0.36%)	⬆️
packages/cli/src/commands/user-management/reset.ts	27.77% <25.00%> (ø)
packages/cli/src/InternalHooks.ts	5.75% <33.33%> (ø)
...ges/cli/src/UserManagement/UserManagementHelper.ts	79.34% <50.00%> (+0.18%)	⬆️
packages/cli/src/sso/saml/samlHelpers.ts	38.38% <50.00%> (ø)
packages/cli/src/WorkflowHelpers.ts	49.18% <60.00%> (+0.27%)	⬆️
packages/cli/src/services/ownership.service.ts	72.22% <75.00%> (+1.63%)	⬆️
... and 15 more

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[krynble]

LGTM

Code looks nice and functionality too. Loved it 💯

[github-actions]

⚠️ Some Cypress E2E specs are failing, please fix them before merging

[cypress]

1 flaky tests on run #1759 ↗︎

0

221

0

0

1

Details:

🌳 pay-666-cache-roles 🖥️ browsers:node18.12.0-chrome107 🤖 ivov 🗃️ e2e/*
Project: n8n	Commit: ed1cdc8661
Status: Passed	Duration: 07:06 💡
Started: Aug 2, 2023 4:33 PM	Ended: Aug 2, 2023 4:40 PM

  cypress/e2e/2-credentials.cy.ts • 1 flaky test

View Output Video

Test		Artifacts
Credentials > should create a new credential using Add Credential button		Output Screenshots Video

_{This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.}

[github-actions]

⚠️ Some Cypress E2E specs are failing, please fix them before merging

[github-actions]

⚠️ Some Cypress E2E specs are failing, please fix them before merging

[github-actions]

⚠️ Some Cypress E2E specs are failing, please fix them before merging

[github-actions]

⚠️ Some Cypress E2E specs are failing, please fix them before merging

[github-actions]

⚠️ Some Cypress E2E specs are failing, please fix them before merging

[github-actions]

⚠️ Some Cypress E2E specs are failing, please fix them before merging

[netroy]

@ivov are the e2e tests passing for you locally on this branch?

[ivov]

@ivov are the e2e tests passing for you locally on this branch?

e2e tests keep failing randomly elsewhere so I've been defaulting to rerunning them without a closer look, I'll check locally.

[github-actions]

✅ All Cypress E2E specs passed

[janober]

Got released with [email protected]