n8n-editor-ui 1.75.1 → 1.76.0

package/dist/assets/{TestDefinitionListView-Dgq9dXJL.js → TestDefinitionListView-D3eg62pM.js}

@@ -1,4 +1,4 @@
-import { d as defineComponent, m as resolveComponent, c as openBlock, h as createElementBlock, j as createBaseVNode, t as toDisplayString, l as unref, n as normalizeClass, i as createVNode, g as useI18n, _ as _export_sfc, q as computed, w as withCtx, k as createTextVNode, F as Fragment, fA as _sfc_main$4, f as createCommentVNode, A as renderList, e as createBlock, b1 as resolveDynamicComponent, aR as _sfc_main$5, I as withModifiers, b0 as mergeProps, b as useRouter, hE as useAnnotationTagsStore, hM as useTestDefinitionStore, r as ref, a as useToast, o as onMounted, V as VIEWS } from "./index-CWcTt6jd.js";
+import { d as defineComponent, m as resolveComponent, c as openBlock, h as createElementBlock, j as createBaseVNode, t as toDisplayString, l as unref, n as normalizeClass, i as createVNode, g as useI18n, _ as _export_sfc, q as computed, w as withCtx, k as createTextVNode, F as Fragment, fx as _sfc_main$4, f as createCommentVNode, A as renderList, e as createBlock, a$ as resolveDynamicComponent, aP as _sfc_main$5, I as withModifiers, b as useRouter, hI as useAnnotationTagsStore, hU as useTestDefinitionStore, r as ref, a as useToast, o as onMounted, V as VIEWS, ae as MODAL_CONFIRM, ak as useMessage } from "./index-DvkbRfZN.js";
 const _sfc_main$3 = /* @__PURE__ */ defineComponent({
   __name: "EmptyState",
   emits: ["create-test"],
@@ -55,48 +55,16 @@ const _hoisted_2 = { key: 1 };
 const _sfc_main$2 = /* @__PURE__ */ defineComponent({
   __name: "TestItem",
   props: {
-    test: {}
+    test: {},
+    actions: {}
   },
-  emits: ["run-test", "view-details", "edit-test", "delete-test", "cancel-test-run"],
-  setup(__props, { emit: __emit }) {
+  emits: ["view-details"],
+  setup(__props) {
     const props = __props;
     const locale = useI18n();
-    const emit = __emit;
-    const actions2 = [
-      {
-        icon: "play",
-        id: "run",
-        event: () => emit("run-test", props.test.id),
-        tooltip: locale.baseText("testDefinition.runTest"),
-        show: () => props.test.execution.status !== "running"
-      },
-      {
-        icon: "stop",
-        id: "cancel",
-        event: () => emit("cancel-test-run", props.test.id, props.test.execution.id),
-        tooltip: locale.baseText("testDefinition.cancelTestRun"),
-        show: () => props.test.execution.status === "running"
-      },
-      {
-        icon: "list",
-        id: "view",
-        event: () => emit("view-details", props.test.id),
-        tooltip: locale.baseText("testDefinition.viewDetails")
-      },
-      {
-        icon: "pen",
-        id: "edit",
-        event: () => emit("edit-test", props.test.id),
-        tooltip: locale.baseText("testDefinition.editTest")
-      },
-      {
-        icon: "trash",
-        id: "delete",
-        event: () => emit("delete-test", props.test.id),
-        tooltip: locale.baseText("testDefinition.deleteTest")
-      }
-    ];
-    const visibleActions = computed(() => actions2.filter((action) => action.show?.() ?? true));
+    const visibleActions = computed(
+      () => props.actions.filter((action) => action.show?.(props.test.id) ?? true)
+    );
     return (_ctx, _cache) => {
       const _component_n8n_text = resolveComponent("n8n-text");
       const _component_n8n_spinner = resolveComponent("n8n-spinner");
@@ -154,22 +122,21 @@ const _sfc_main$2 = /* @__PURE__ */ defineComponent({
             return openBlock(), createBlock(_component_n8n_tooltip, {
               key: action.icon,
               placement: "top",
-              "show-after": 1e3
+              "show-after": 1e3,
+              content: action.tooltip(_ctx.test.id)
             }, {
-              content: withCtx(() => [
-                createTextVNode(toDisplayString(action.tooltip), 1)
-              ]),
               default: withCtx(() => [
                 (openBlock(), createBlock(resolveDynamicComponent(unref(_sfc_main$5)), {
                   icon: action.icon,
                   "data-test-id": `${action.id}-test-button-${_ctx.test.id}`,
                   type: "tertiary",
                   size: "mini",
-                  onClick: withModifiers(action.event, ["stop"])
-                }, null, 8, ["icon", "data-test-id", "onClick"]))
+                  disabled: action?.disabled ? action.disabled(_ctx.test.id) : false,
+                  onClick: withModifiers(($event) => action.event(_ctx.test.id), ["stop"])
+                }, null, 8, ["icon", "data-test-id", "disabled", "onClick"]))
               ]),
               _: 2
-            }, 1024);
+            }, 1032, ["content"]);
           }), 128))
         ], 2)
       ], 10, _hoisted_1);
@@ -199,9 +166,10 @@ const TestItem = /* @__PURE__ */ _export_sfc(_sfc_main$2, [["__cssModules", cssM
 const _sfc_main$1 = /* @__PURE__ */ defineComponent({
   __name: "TestsList",
   props: {
-    tests: {}
+    tests: {},
+    actions: {}
   },
-  emits: ["create-test"],
+  emits: ["create-test", "view-details"],
   setup(__props) {
     const locale = useI18n();
     return (_ctx, _cache) => {
@@ -219,11 +187,12 @@ const _sfc_main$1 = /* @__PURE__ */ defineComponent({
           }, null, 8, ["label"])
         ], 2),
         (openBlock(true), createElementBlock(Fragment, null, renderList(_ctx.tests, (test) => {
-          return openBlock(), createBlock(TestItem, mergeProps({
+          return openBlock(), createBlock(TestItem, {
             key: test.id,
             test,
-            ref_for: true
-          }, _ctx.$attrs), null, 16, ["test"]);
+            actions: _ctx.actions,
+            onViewDetails: ($event) => _ctx.$emit("view-details", test.id)
+          }, null, 8, ["test", "actions", "onViewDetails"]);
         }), 128))
       ], 2);
     };
@@ -248,13 +217,50 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
     const isLoading = ref(false);
     const toast = useToast();
     const locale = useI18n();
+    const { confirm } = useMessage();
+    const actions2 = computed(() => [
+      {
+        icon: "play",
+        id: "run",
+        event: onRunTest,
+        disabled: isRunDisabled,
+        show: (testId) => !isTestRunning(testId),
+        tooltip: (testId) => isRunDisabled(testId) ? getDisabledRunTooltip(testId) : locale.baseText("testDefinition.runTest")
+      },
+      {
+        icon: "stop",
+        id: "cancel",
+        event: onCancelTestRun,
+        tooltip: () => locale.baseText("testDefinition.cancelTestRun"),
+        show: (testId) => isTestRunning(testId)
+      },
+      {
+        icon: "list",
+        id: "view",
+        event: onViewDetails,
+        tooltip: () => locale.baseText("testDefinition.viewDetails")
+      },
+      {
+        icon: "pen",
+        id: "edit",
+        event: onEditTest,
+        tooltip: () => locale.baseText("testDefinition.editTest")
+      },
+      {
+        icon: "trash",
+        id: "delete",
+        event: onDeleteTest,
+        tooltip: () => locale.baseText("testDefinition.deleteTest")
+      }
+    ]);
     const tests = computed(() => {
       return (testDefinitionStore.allTestDefinitionsByWorkflowId[router.currentRoute.value.params.name] ?? []).filter((test) => test.id !== void 0).sort((a, b) => new Date(b?.updatedAt ?? "").getTime() - new Date(a?.updatedAt ?? "").getTime()).map((test) => ({
         id: test.id,
         name: test.name ?? "",
         tagName: test.annotationTagId ? getTagName(test.annotationTagId) : "",
         testCases: testDefinitionStore.testRunsByTestId[test.id]?.length ?? 0,
-        execution: getTestExecution(test.id)
+        execution: getTestExecution(test.id),
+        fieldsIssues: testDefinitionStore.getFieldIssues(test.id)
       }));
     });
     const hasTests = computed(() => tests.value.length > 0);
@@ -263,6 +269,10 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
       const matchingTag = allTags.value.find((t) => t.id === tagId);
       return matchingTag?.name ?? "";
     }
+    function getDisabledRunTooltip(testId) {
+      const issues = testDefinitionStore.getFieldIssues(testId)?.map((i) => i.message).join("<br />- ");
+      return `${locale.baseText("testDefinition.completeConfig")} <br /> - ${issues}`;
+    }
     function getTestExecution(testId) {
       const lastRun = testDefinitionStore.lastRunByTestId[testId];
       if (!lastRun) {
@@ -283,6 +293,12 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
       };
       return mockExecutions;
     }
+    function isTestRunning(testId) {
+      return testDefinitionStore.lastRunByTestId[testId]?.status === "running";
+    }
+    function isRunDisabled(testId) {
+      return testDefinitionStore.getFieldIssues(testId)?.length > 0;
+    }
     function onCreateTest() {
       void router.push({ name: VIEWS.NEW_TEST_DEFINITION });
     }
@@ -302,8 +318,9 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
         toast.showError(error, locale.baseText("testDefinition.list.testStartError"));
       }
     }
-    async function onCancelTestRun(testId, testRunId) {
+    async function onCancelTestRun(testId) {
       try {
+        const testRunId = testDefinitionStore.lastRunByTestId[testId]?.id;
         if (!testRunId) {
           throw new Error("Failed to cancel test run");
         }
@@ -328,6 +345,19 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
       void router.push({ name: VIEWS.TEST_DEFINITION_EDIT, params: { testId } });
     }
     async function onDeleteTest(testId) {
+      const deleteConfirmed = await confirm(
+        locale.baseText("testDefinition.deleteTest.warning"),
+        locale.baseText("testDefinition.deleteTest"),
+        {
+          type: "warning",
+          confirmButtonText: locale.baseText("generic.delete"),
+          cancelButtonText: locale.baseText("generic.cancel"),
+          closeOnClickModal: true
+        }
+      );
+      if (deleteConfirmed !== MODAL_CONFIRM) {
+        return;
+      }
       await testDefinitionStore.deleteById(testId);
       toast.showMessage({
         title: locale.baseText("testDefinition.list.testDeleted"),
@@ -338,12 +368,9 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
       if (!isLoading.value) {
         isLoading.value = true;
         try {
-          await Promise.all([
-            tagsStore.fetchAll(),
-            testDefinitionStore.fetchAll({
-              workflowId: router.currentRoute.value.params.name
-            })
-          ]);
+          await testDefinitionStore.fetchAll({
+            workflowId: router.currentRoute.value.params.name
+          });
           isLoading.value = false;
         } catch (error) {
           toast.showError(error, locale.baseText("testDefinition.list.loadError"));
@@ -352,7 +379,7 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
         }
       }
     }
-    onMounted(() => {
+    onMounted(async () => {
       if (!testDefinitionStore.isFeatureEnabled) {
         toast.showMessage({
           title: locale.baseText("testDefinition.notImplemented"),
@@ -364,7 +391,8 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
         });
         return;
       }
-      void loadInitialData();
+      await loadInitialData();
+      tests.value.forEach((test) => testDefinitionStore.updateRunFieldIssues(test.id));
     });
     return (_ctx, _cache) => {
       const _component_n8n_loading = resolveComponent("n8n-loading");
@@ -387,13 +415,10 @@ const _sfc_main = /* @__PURE__ */ defineComponent({
           })) : (openBlock(), createBlock(TestsList, {
             key: 1,
             tests: tests.value,
-            onCreateTest,
-            onRunTest,
+            actions: actions2.value,
             onViewDetails,
-            onEditTest,
-            onDeleteTest,
-            onCancelTestRun
-          }, null, 8, ["tests"]))
+            onCreateTest
+          }, null, 8, ["tests", "actions"]))
         ], 64))
       ], 2);
     };

package/dist/assets/{TestDefinitionRootView-DU-JMBSH.js → TestDefinitionRootView-DJdsaL4M.js}

@@ -1,4 +1,4 @@
-import { d as defineComponent, b as useRouter, a4 as useWorkflowHelpers, T as useWorkflowsStore, o as onMounted, c as openBlock, h as createElementBlock, i as createVNode, n as normalizeClass, m as resolveComponent, _ as _export_sfc } from "./index-CWcTt6jd.js";
+import { d as defineComponent, b as useRouter, a4 as useWorkflowHelpers, T as useWorkflowsStore, o as onMounted, c as openBlock, h as createElementBlock, i as createVNode, n as normalizeClass, m as resolveComponent, _ as _export_sfc } from "./index-DvkbRfZN.js";
 const _sfc_main = /* @__PURE__ */ defineComponent({
   __name: "TestDefinitionRootView",
   setup(__props) {