From fa118ba3e40b5b03361f58220a2cb670cb84ef3b Mon Sep 17 00:00:00 2001
From: Ahmed Awan <qe66653@umbc.edu>
Date: Mon, 4 Nov 2024 16:41:15 -0600
Subject: [PATCH] adjust `WorkflowRunButton` jest for router push

The `BButton :to`, was replaced with a `BButton @click` because the styling in `WorkflowNavigationTitle` worked better with a non `<a>` BButton (using `:to` turns the `<button>` into an `<a>`)
---
 .../Workflow/WorkflowRunButton.test.ts        | 20 +++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/client/src/components/Workflow/WorkflowRunButton.test.ts b/client/src/components/Workflow/WorkflowRunButton.test.ts
index 11c02dc95549..dd9c9530abbb 100644
--- a/client/src/components/Workflow/WorkflowRunButton.test.ts
+++ b/client/src/components/Workflow/WorkflowRunButton.test.ts
@@ -1,4 +1,5 @@
 import { mount } from "@vue/test-utils";
+import flushPromises from "flush-promises";
 import { getLocalVue } from "tests/jest/helpers";
 
 import { generateRandomString } from "./testUtils";
@@ -11,12 +12,19 @@ const WORKFLOW_ID = generateRandomString();
 const WORKFLOW_RUN_BUTTON_SELECTOR = `[data-workflow-run="${WORKFLOW_ID}"]`;
 
 async function mountWorkflowRunButton(props?: { id: string; full?: boolean }) {
+    const mockRouter = {
+        push: jest.fn(),
+    };
+
     const wrapper = mount(WorkflowRunButton as object, {
         propsData: { ...props },
         localVue,
+        mocks: {
+            $router: mockRouter,
+        },
     });
 
-    return { wrapper };
+    return { wrapper, mockRouter };
 }
 
 describe("WorkflowRunButton.vue", () => {
@@ -29,10 +37,14 @@ describe("WorkflowRunButton.vue", () => {
     });
 
     it("should redirect to workflow run page", async () => {
-        const { wrapper } = await mountWorkflowRunButton({ id: WORKFLOW_ID });
+        const { wrapper, mockRouter } = await mountWorkflowRunButton({ id: WORKFLOW_ID });
+
+        const runButton = wrapper.find(WORKFLOW_RUN_BUTTON_SELECTOR);
 
-        const runButton = await wrapper.find(WORKFLOW_RUN_BUTTON_SELECTOR);
+        await runButton.trigger("click");
+        await flushPromises();
 
-        expect(runButton.attributes("href")).toBe(`/workflows/run?id=${WORKFLOW_ID}`);
+        expect(mockRouter.push).toHaveBeenCalledTimes(1);
+        expect(mockRouter.push).toHaveBeenCalledWith(`/workflows/run?id=${WORKFLOW_ID}`);
     });
 });