From fea8a19be42b49e5b9d29fac49acd7b0dce3299d Mon Sep 17 00:00:00 2001 From: Olivier Tassinari Date: Wed, 2 Oct 2024 00:56:19 +0200 Subject: [PATCH] [test] Fix Escape event firing event --- .../disallow-active-elements-as-key-event-target.test.js | 2 +- packages/mui-material/src/Dialog/Dialog.test.js | 8 ++++---- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/packages/eslint-plugin-material-ui/src/rules/disallow-active-elements-as-key-event-target.test.js b/packages/eslint-plugin-material-ui/src/rules/disallow-active-elements-as-key-event-target.test.js index 6e90f6c1898800..8ac6281bbea7a9 100644 --- a/packages/eslint-plugin-material-ui/src/rules/disallow-active-elements-as-key-event-target.test.js +++ b/packages/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 index dd954f5b4f0b27..a37e5dbd4b15c6 100644 --- a/packages/mui-material/src/Dialog/Dialog.test.js +++ b/packages/mui-material/src/Dialog/Dialog.test.js @@ -88,7 +88,7 @@ describe('', () => { // 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('', () => { // 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.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);