Simplifies history navigation tests by removing dropdown option checks

Removes detailed assertions for dropdown options (they moved to the HistoryOptions tests) and their states in history navigation tests. Refactors user data handling to use a partial type for better type safety.

Author: itisAliRH

client/src/components/History/CurrentHistory/HistoryNavigation.test.ts

@@ -2,38 +2,14 @@ import { getLocalVue } from "@tests/jest/helpers";
 import { shallowMount } from "@vue/test-utils";
 import { createPinia } from "pinia";
 
+import type { RegisteredUser } from "@/api";
 import { useUserStore } from "@/stores/userStore";
 
 import HistoryNavigation from "./HistoryNavigation.vue";
 
 const localVue = getLocalVue();
 
-// all options
-const expectedOptions = [
-    "Show Histories Side-by-Side",
-    "Resume Paused Jobs",
-    "Copy this History",
-    "Delete this History",
-    "Export Tool References",
-    "Export History to File",
-    "Archive History",
-    "Extract Workflow",
-    "Show Invocations",
-    "Share & Manage Access",
-];
-
-// options enabled for logged-out users
-const anonymousOptions = [
-    "Resume Paused Jobs",
-    "Delete this History",
-    "Export Tool References",
-    "Export History to File",
-];
-
-// options disabled for logged-out users
-const anonymousDisabledOptions = expectedOptions.filter((option) => !anonymousOptions.includes(option));
-
-async function createWrapper(propsData: object, userData?: any) {
+async function createWrapper(propsData: object, userData?: Partial<RegisteredUser>) {
     const pinia = createPinia();
 
     const wrapper = shallowMount(HistoryNavigation as object, {
@@ -43,7 +19,7 @@ async function createWrapper(propsData: object, userData?: any) {
     });
 
     const userStore = useUserStore();
-    userStore.currentUser = { ...userStore.currentUser, ...userData };
+    userStore.currentUser = { ...(userStore.currentUser as RegisteredUser), ...userData };
 
     return wrapper;
 }
@@ -65,12 +41,6 @@ describe("History Navigation", () => {
         expect(createButton.attributes().disabled).toBeFalsy();
         const switchButton = wrapper.find("*[data-description='switch to another history']");
         expect(switchButton.attributes().disabled).toBeFalsy();
-
-        const dropDown = wrapper.find("*[data-description='history options']");
-        const optionElements = dropDown.findAll("bdropdownitem-stub");
-        const optionTexts = optionElements.wrappers.map((el) => el.text());
-
-        expect(optionTexts).toStrictEqual(expectedOptions);
     });
 
     it("disables options for anonymous users", async () => {
@@ -83,28 +53,5 @@ describe("History Navigation", () => {
         expect(createButton.attributes().disabled).toBeTruthy();
         const switchButton = wrapper.find("*[data-description='switch to another history']");
         expect(switchButton.attributes().disabled).toBeTruthy();
-
-        const dropDown = wrapper.find("*[data-description='history options']");
-        const enabledOptionElements = dropDown.findAll("bdropdownitem-stub:not([disabled])");
-        const enabledOptionTexts = enabledOptionElements.wrappers.map((el) => el.text());
-        expect(enabledOptionTexts).toStrictEqual(anonymousOptions);
-
-        const disabledOptionElements = dropDown.findAll("bdropdownitem-stub[disabled]");
-        const disabledOptionTexts = disabledOptionElements.wrappers.map((el) => el.text());
-        expect(disabledOptionTexts).toStrictEqual(anonymousDisabledOptions);
-    });
-
-    it("prompts anonymous users to log in", async () => {
-        const wrapper = await createWrapper({
-            history: { id: "current_history_id" },
-            histories: [],
-        });
-
-        const dropDown = wrapper.find("*[data-description='history options']");
-        const disabledOptionElements = dropDown.findAll("bdropdownitem-stub[disabled]");
-
-        disabledOptionElements.wrappers.forEach((option) => {
-            expect((option.attributes("title") as string).toLowerCase()).toContain("log in");
-        });
     });
 });