NXT-4467: add tests and fix lint issue

NXT-4467 (Close version history panel when entering a shared component)

Author: hriverahdez

org.knime.ui.js/src/store/application/__tests__/lifecycle.test.ts

@@ -15,6 +15,7 @@ import { createRouter, createWebHistory } from "vue-router";
 import { setupHints } from "@knime/components";
 import { CURRENT_STATE_VERSION } from "@knime/hub-features/versions";
 
+import { WorkflowInfo } from "@/api/gateway-api/generated-api";
 import { isBrowser, isDesktop, runInEnvironment } from "@/environment";
 import { APP_ROUTES } from "@/router/appRoutes";
 import { router, routes } from "@/router/router";
@@ -324,6 +325,54 @@ describe("application::lifecycle", () => {
       expect(mockedAPI.event.subscribeEvent).not.toHaveBeenCalled();
     });
 
+    it("handles afterSetActivateWorkflow when a workflow is loaded", async () => {
+      const { lifecycleStore, componentInteractionsStore } = loadStore();
+      const loadedWF = createWorkflow({
+        info: { containerId: "root" },
+        nodes: Object.create({}),
+      });
+
+      mockedAPI.workflow.getWorkflow.mockResolvedValue({
+        workflow: loadedWF,
+        snapshotId: "snap",
+      });
+      await lifecycleStore.loadWorkflow({
+        projectId: "wf1",
+        workflowId: "root:0:12",
+      });
+
+      expect(
+        componentInteractionsStore.checkForLinkedComponentUpdates,
+      ).toHaveBeenCalledWith({ auto: true });
+    });
+
+    it("closes version panel when entering a linked component", async () => {
+      const { lifecycleStore, workflowVersionsStore } = loadStore();
+
+      // @ts-expect-error - mock getter
+      workflowVersionsStore.activeProjectVersionsModeStatus = "active";
+
+      const loadedWF = createWorkflow({
+        info: {
+          containerId: "root",
+          linked: true,
+          containerType: WorkflowInfo.ContainerTypeEnum.Component,
+        },
+        nodes: Object.create({}),
+      });
+
+      mockedAPI.workflow.getWorkflow.mockResolvedValue({
+        workflow: loadedWF,
+        snapshotId: "snap",
+      });
+      await lifecycleStore.loadWorkflow({
+        projectId: "wf1",
+        workflowId: "root:0:12",
+      });
+
+      expect(workflowVersionsStore.deactivateVersionsMode).toHaveBeenCalled();
+    });
+
     it("unloads workflow when another one is loaded", async () => {
       const { lifecycleStore, workflowStore, selectionStore } = loadStore();
       const loadedWF = createWorkflow({

org.knime.ui.js/src/store/application/lifecycle.ts

@@ -10,8 +10,8 @@ import { sleep } from "@knime/utils";
 import {
   AppState,
   type Workflow,
-  type WorkflowSnapshot,
   WorkflowInfo,
+  type WorkflowSnapshot,
 } from "@/api/gateway-api/generated-api";
 import { fetchUiStrings as kaiFetchUiStrings } from "@/components/kai/useKaiServer";
 import { isDesktop, runInEnvironment } from "@/environment";
@@ -23,8 +23,8 @@ import { useSelectionStore } from "@/store/selection";
 import { ratioToZoomLevel, useSettingsStore } from "@/store/settings";
 import { useAnnotationInteractionsStore } from "@/store/workflow/annotationInteractions";
 import { useComponentInteractionsStore } from "@/store/workflow/componentInteractions";
-import { useWorkflowVersionsStore } from "@/store/workflow/workflowVersions";
 import { useWorkflowStore } from "@/store/workflow/workflow";
+import { useWorkflowVersionsStore } from "@/store/workflow/workflowVersions";
 import { encodeString } from "@/util/encodeString";
 import { geometry } from "@/util/geometry";
 import { setProjectActiveOrThrow } from "@/util/projectUtil";
@@ -597,10 +597,12 @@ export const useLifecycleStore = defineStore("lifecycle", {
           WorkflowInfo.ContainerTypeEnum.Component
       ) {
         const versionsStore = useWorkflowVersionsStore();
+
         if (versionsStore.activeProjectVersionsModeStatus !== "inactive") {
-          void versionsStore.deactivateVersionsMode();
+          versionsStore.deactivateVersionsMode();
         }
       }
+
       useComponentInteractionsStore().checkForLinkedComponentUpdates({
         auto: true,
       });