From b2293b7ad5fcd951f9cbd62944891e1a794532ed Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ra=C3=BAl=20G=C3=B3mez=20Morales?= Date: Fri, 21 Feb 2025 12:58:28 +0100 Subject: [PATCH] feat(editor): Evaluation feature Phase one readiness (no-changelog) (#13383) Co-authored-by: Oleg Ivaniv --- cypress/pages/ndv.ts | 5 +- .../N8nActionToggle/ActionToggle.vue | 14 + .../src/components/N8nTag/Tag.vue | 3 +- .../src/components/N8nText/Text.vue | 10 +- packages/design-system/src/types/text.ts | 3 + .../editor-ui/src/api/testDefinition.ee.ts | 5 +- .../src/components/InlineNameEdit.vue | 59 ++- .../src/components/MainHeader/MainHeader.vue | 21 +- .../Projects/ProjectCreateResource.vue | 17 +- .../ResourceLocatorDropdown.vue | 2 +- .../EditDefinition/BlockArrow.vue | 127 ++++- .../EditDefinition/EvaluationStep.vue | 176 +++++-- .../EditDefinition/MetricsInput.vue | 20 +- .../EditDefinition/NodesPinning.vue | 95 ++-- .../EditDefinition/WorkflowSelector.vue | 101 +++- .../EditDefinition/sections/ConfigSection.vue | 447 ++++++++++++------ .../EditDefinition/sections/HeaderSection.vue | 134 ------ .../EditDefinition/sections/RunsSection.vue | 73 ++- .../ListDefinition/EmptyState.vue | 128 +++-- .../ListDefinition/TestItem.vue | 254 +++++----- .../ListDefinition/TestsList.vue | 43 -- .../TestDefinition/ListRuns/MetricsChart.vue | 10 +- .../TestDefinition/ListRuns/TestRunsTable.vue | 113 ++--- .../composables/useTestDefinitionForm.ts | 13 +- .../TestDefinition/shared/TestTableBase.vue | 160 +++++-- .../TestDefinition/tests/MetricsInput.test.ts | 13 +- .../tests/useTestDefinitionForm.test.ts | 10 +- .../src/components/TestDefinition/types.ts | 29 +- .../WorkflowSelectorParameterInput.vue | 6 + .../workflow/WorkflowExecutionsCard.vue | 1 + .../workflow/WorkflowExecutionsList.vue | 29 +- .../WorkflowExecutionsPreview.test.ts | 34 +- .../workflow/WorkflowExecutionsPreview.vue | 128 ++++- packages/editor-ui/src/constants.ts | 1 - packages/editor-ui/src/constants.workflows.ts | 115 ++--- .../src/plugins/i18n/locales/en.json | 88 ++-- packages/editor-ui/src/plugins/icons/index.ts | 6 + packages/editor-ui/src/router.ts | 49 +- packages/editor-ui/src/stores/tags.store.ts | 17 +- .../stores/testDefinition.store.ee.test.ts | 20 +- .../src/stores/testDefinition.store.ee.ts | 21 +- .../TestDefinition/TestDefinitionEditView.vue | 395 ++++++++-------- .../TestDefinition/TestDefinitionListView.vue | 327 +++++++------ .../TestDefinition/TestDefinitionNewView.vue | 78 +++ .../TestDefinition/TestDefinitionRootView.vue | 21 +- .../TestDefinitionRunDetailView.vue | 292 ++++++++---- .../TestDefinitionRunsListView.vue | 151 ------ .../tests/TestDefinitionEditView.test.ts | 289 +++-------- .../tests/TestDefinitionListView.test.ts | 253 +++++----- .../tests/TestDefinitionRootView.test.ts | 55 +-- .../tests/TestDefinitionRunDetailView.test.ts | 245 ---------- .../src/views/WorkflowExecutionsView.vue | 1 + 52 files changed, 2452 insertions(+), 2255 deletions(-) delete mode 100644 packages/editor-ui/src/components/TestDefinition/EditDefinition/sections/HeaderSection.vue delete mode 100644 packages/editor-ui/src/components/TestDefinition/ListDefinition/TestsList.vue create mode 100644 packages/editor-ui/src/views/TestDefinition/TestDefinitionNewView.vue delete mode 100644 packages/editor-ui/src/views/TestDefinition/TestDefinitionRunsListView.vue delete mode 100644 packages/editor-ui/src/views/TestDefinition/tests/TestDefinitionRunDetailView.test.ts diff --git a/cypress/pages/ndv.ts b/cypress/pages/ndv.ts index 275d80593d..d5b09c15dd 100644 --- a/cypress/pages/ndv.ts +++ b/cypress/pages/ndv.ts @@ -92,7 +92,10 @@ export class NDV extends BasePage { resourceLocatorModeSelector: (paramName: string) => this.getters.resourceLocator(paramName).find('[data-test-id="rlc-mode-selector"]'), resourceLocatorSearch: (paramName: string) => - this.getters.resourceLocator(paramName).findChildByTestId('rlc-search'), + this.getters + .resourceLocator(paramName) + .find('[aria-describedby]') + .then(($el) => cy.get(`#${$el.attr('aria-describedby')}`).findChildByTestId('rlc-search')), resourceMapperFieldsContainer: () => cy.getByTestId('mapping-fields-container'), resourceMapperSelectColumn: () => cy.getByTestId('matching-column-select'), resourceMapperRemoveFieldButton: (fieldName: string) => diff --git a/packages/design-system/src/components/N8nActionToggle/ActionToggle.vue b/packages/design-system/src/components/N8nActionToggle/ActionToggle.vue index 8e0a081220..57dd20bf0d 100644 --- a/packages/design-system/src/components/N8nActionToggle/ActionToggle.vue +++ b/packages/design-system/src/components/N8nActionToggle/ActionToggle.vue @@ -1,5 +1,6 @@