fix: find OK button by label in showMessageBox, handle Refactor Preview

On Linux the button order in dialog.showMessageBox differs from Windows,
so response:0 selects 'Show Preview' instead of 'OK'. Fix by scanning
the buttons array for the 'OK' label.

Also handle the Refactor Preview panel (Apply/Discard) as a fallback
in case the dialog still enters preview mode.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: wenytang-ms

test/e2e/fixtures/baseTest.ts

@@ -106,10 +106,26 @@ export const test = base.extend<TestFixtures>({
         // Auto-dismiss Electron native dialogs (e.g. redhat.java refactoring
         // confirmation "wants to make refactoring changes"). These dialogs are
         // outside the renderer DOM and cannot be handled via Playwright Page API.
-        // Monkey-patch dialog.showMessageBox in the main process to auto-click OK.
+        // Monkey-patch dialog.showMessageBox in the main process to find and
+        // click the "OK" button by label, falling back to the first button.
         await electronApp.evaluate(({ dialog }) => {
-            dialog.showMessageBox = async () => ({ response: 0, checkboxChecked: true });
-            dialog.showMessageBoxSync = () => 0;
+            const origShowMessageBox = dialog.showMessageBox;
+            dialog.showMessageBox = async (_win: any, opts: any) => {
+                // opts may be the first arg if called without a window
+                const options = opts || _win;
+                const buttons: string[] = options?.buttons || [];
+                // Find "OK" button index; fall back to first button
+                let idx = buttons.findIndex((b: string) => /^OK$/i.test(b));
+                if (idx < 0) idx = 0;
+                return { response: idx, checkboxChecked: true };
+            };
+            dialog.showMessageBoxSync = (_win: any, opts: any) => {
+                const options = opts || _win;
+                const buttons: string[] = options?.buttons || [];
+                let idx = buttons.findIndex((b: string) => /^OK$/i.test(b));
+                if (idx < 0) idx = 0;
+                return idx;
+            };
         });
 
         // Dismiss any startup notifications/dialogs before handing off to tests

test/e2e/tests/fileOperations.test.ts

@@ -87,6 +87,19 @@ test.describe("File Operations", () => {
             // Dialog may not appear in all cases
         }
 
+        // On Linux, if the refactoring dialog resolved to "Show Preview",
+        // VS Code shows a Refactor Preview panel with "Apply" / "Discard"
+        // buttons. Click "Apply" to complete the rename.
+        try {
+            const applyBtn = page.getByRole(VSCode.BUTTON_ROLE, { name: "Apply" });
+            if (await applyBtn.isVisible({ timeout: 3_000 }).catch(() => false)) {
+                await applyBtn.click();
+                await page.waitForTimeout(Timeout.CLICK);
+            }
+        } catch {
+            // No refactor preview
+        }
+
         // Editor should open with renamed file
         const tabFound = await VscodeOperator.waitForEditorTab(page, "AppRenamed.java");
         expect(tabFound).toBeTruthy();