From c3bcb5185e920ea0e7ed513ad06f7dcc7852f7f0 Mon Sep 17 00:00:00 2001
From: Ahmed Awan <qe66653@umbc.edu>
Date: Wed, 10 Jun 2026 14:50:38 -0500
Subject: [PATCH 1/2] [26.1] Add missing `confirm` prop to `GModal` in
 `RefactorConfirmationModal`

---
 .../components/Workflow/Editor/RefactorConfirmationModal.vue    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/client/src/components/Workflow/Editor/RefactorConfirmationModal.vue b/client/src/components/Workflow/Editor/RefactorConfirmationModal.vue
index 2b0015f5b0f..a96b4aa9b32 100644
--- a/client/src/components/Workflow/Editor/RefactorConfirmationModal.vue
+++ b/client/src/components/Workflow/Editor/RefactorConfirmationModal.vue
@@ -118,7 +118,7 @@ async function executeRefactoring() {
 </script>
 
 <template>
-    <GModal :show.sync="show" :title="title" fixed-height ok-text="Save" @ok="executeRefactoring">
+    <GModal confirm :show.sync="show" :title="title" fixed-height ok-text="Save" @ok="executeRefactoring">
         <div class="workflow-refactor-modal">
             {{ message }}
             <ul>

From f1dbe409cbd1cdc71e147d7678a8a9b0e010fdbf Mon Sep 17 00:00:00 2001
From: Ahmed Awan <qe66653@umbc.edu>
Date: Wed, 10 Jun 2026 15:33:35 -0500
Subject: [PATCH 2/2] add vitest for testing save in
 `RefactorConfirmationModal`

---
 .../Editor/RefactorConfirmationModal.test.ts   | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/client/src/components/Workflow/Editor/RefactorConfirmationModal.test.ts b/client/src/components/Workflow/Editor/RefactorConfirmationModal.test.ts
index 8e79dffe650..bd613e504cc 100644
--- a/client/src/components/Workflow/Editor/RefactorConfirmationModal.test.ts
+++ b/client/src/components/Workflow/Editor/RefactorConfirmationModal.test.ts
@@ -1,5 +1,5 @@
 import { getLocalVue } from "@tests/vitest/helpers";
-import { shallowMount, type Wrapper } from "@vue/test-utils";
+import { mount, type Wrapper } from "@vue/test-utils";
 import flushPromises from "flush-promises";
 import { beforeEach, describe, expect, it, vi } from "vitest";
 
@@ -29,7 +29,7 @@ describe("RefactorConfirmationModal.vue", () => {
     beforeEach(() => {
         vi.clearAllMocks();
         mockConfirm.mockReset();
-        wrapper = shallowMount(RefactorConfirmationModal as object, {
+        wrapper = mount(RefactorConfirmationModal as object, {
             propsData: {
                 refactorActions: [],
                 workflowId: TEST_WORKFLOW_ID,
@@ -87,7 +87,7 @@ describe("RefactorConfirmationModal.vue", () => {
         expect(wrapper.emitted().onRefactor!.length).toBe(1);
     });
 
-    it("should show confirmation dialog if there are messages from the server", async () => {
+    it("should show confirmation modal if there are messages from the server", async () => {
         vi.mocked(refactor).mockReturnValue(
             new Promise((then) => {
                 then({
@@ -121,7 +121,19 @@ describe("RefactorConfirmationModal.vue", () => {
         // but didn't follow up with executing the action because we need to confirm
         expect(vi.mocked(refactor).mock.calls.length).toBe(1);
 
+        // Modal should now be shown with the message from the server
         expect(wrapper.findComponent(GModal).props().show).toBeTruthy();
+        expect(wrapper.find(".workflow-refactor-modal").text()).toContain(
+            "hey, a connection was dropped - better respond",
+        );
+
+        // Simulate confirming the modal
+        const saveButton = wrapper.findComponent(GModal).find(".g-modal-confirm-buttons .g-button.g-blue");
+        await saveButton.trigger("click");
+
+        // On save, refactor should be called again to execute the action (not a dry run this time)
+        expect(vi.mocked(refactor).mock.calls[1]![0]).toEqual(TEST_WORKFLOW_ID);
+        expect(vi.mocked(refactor).mock.calls[1]![3]).toBeFalsy();
     });
 
     it("should show confirm dialog when refactoring a non-latest version", async () => {