fix tests

packages/itwinui-react/src/core/Dialog/Dialog.test.tsx

@@ -3,7 +3,7 @@
  * See LICENSE.md in the project root for license terms and full copyright notice.
  *--------------------------------------------------------------------------------------------*/
 import * as React from 'react';
-import { render, act, screen, fireEvent } from '@testing-library/react';
+import { render, act, screen } from '@testing-library/react';
 import { Dialog } from './Dialog.js';
 import { Button } from '../Buttons/Button.js';
 import { userEvent } from '@testing-library/user-event';
@@ -212,13 +212,4 @@ it('should expose show() and close() methods', () => {
   act(() => dialog.close());
   act(() => vi.runAllTimers());
   expect(dialogElement).not.toBeVisible();
-
-  act(() => dialog.show());
-
-  // Built-in close should still work
-  act(() => {
-    fireEvent.keyDown(dialogElement, { key: 'Escape' });
-  });
-
-  expect(dialogElement).not.toBeVisible();
 });

testing/e2e/app/routes/Dialog/route.tsx

@@ -0,0 +1,16 @@
+import * as React from 'react';
+import { Dialog } from '@itwin/itwinui-react';
+
+export default function DialogTest() {
+  const dialog = Dialog.useInstance();
+
+  React.useEffect(() => {
+    dialog.show();
+  }, [dialog]);
+
+  return (
+    <Dialog setFocus instance={dialog}>
+      <Dialog.Main>Hello</Dialog.Main>
+    </Dialog>
+  );
+}

testing/e2e/app/routes/Dialog/spec.ts

@@ -0,0 +1,13 @@
+import { test, expect } from '@playwright/test';
+
+test.describe('Dialog triggers', () => {
+  test('should close dialog when pressing Esc', async ({ page }) => {
+    await page.goto('/Dialog');
+
+    const dialog = page.getByRole('dialog');
+    await expect(dialog).toBeVisible();
+
+    await page.keyboard.press('Escape');
+    await expect(dialog).not.toBeVisible();
+  });
+});