[test] Fix Escape event firing event

mui · Oct 1, 2024 · fea8a19 · fea8a19
1 parent f7feae3
commit fea8a19
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/.../eslint-plugin-material-ui/src/rules/disallow-active-elements-as-key-event-target.test.js b/.../eslint-plugin-material-ui/src/rules/disallow-active-elements-as-key-event-target.test.js
@@ -8,7 +8,7 @@ const ruleTester = new eslint.RuleTester({
 ruleTester.run('disallow-active-element-as-key-event-target', rule, {
   valid: [
     "import { fireEvent } from '@mui/internal-test-utils';\nfireEvent.keyDown(getByRole('button'), { key: ' ' })",
-    "import { fireEvent } from '@mui/internal-test-utils';\nfireEvent.keyDown(document.body, { key: 'Esc' })",
+    "import { fireEvent } from '@mui/internal-test-utils';\nfireEvent.keyDown(document.body, { key: 'Escape' })",
     "import { fireEvent } from '@mui/internal-test-utils';\nfireEvent.keyUp(document.body, { key: 'Tab' })",
   ],
   invalid: [

diff --git a/packages/mui-material/src/Dialog/Dialog.test.js b/packages/mui-material/src/Dialog/Dialog.test.js
@@ -88,7 +88,7 @@ describe('<Dialog />', () => {
 
     // keyDown not targetted at anything specific
     // eslint-disable-next-line material-ui/disallow-active-element-as-key-event-target
-    fireEvent.keyDown(document.activeElement, { key: 'Esc' });
+    fireEvent.keyDown(document.activeElement, { key: 'Escape' });
     expect(onClose.calledOnce).to.equal(true);
 
     clock.tick(100);
@@ -107,10 +107,10 @@ describe('<Dialog />', () => {
 
     // Actual Behavior when "あ" (Japanese) is entered and press the Esc for IME cancellation.
     fireEvent.change(textbox, { target: { value: 'あ' } });
-    fireEvent.keyDown(textbox, { key: 'Esc', keyCode: 229 });
+    fireEvent.keyDown(textbox, { key: 'Escape', keyCode: 229 });
     expect(onClose.callCount).to.equal(0);
 
-    fireEvent.keyDown(textbox, { key: 'Esc' });
+    fireEvent.keyDown(textbox, { key: 'Escape' });
     expect(onClose.callCount).to.equal(1);
   });
 
@@ -143,7 +143,7 @@ describe('<Dialog />', () => {
       dialog.click();
       // keyDown is not targetted at anything specific.
       // eslint-disable-next-line material-ui/disallow-active-element-as-key-event-target
-      fireEvent.keyDown(document.activeElement, { key: 'Esc' });
+      fireEvent.keyDown(document.activeElement, { key: 'Escape' });
     });
 
     expect(onClose.callCount).to.equal(0);