From 5d9a0252344855eba7c5925a2b67c5ecd901b16a Mon Sep 17 00:00:00 2001 From: Titani Labaj <39532947+tlabaj@users.noreply.github.com> Date: Fri, 5 Jan 2024 15:03:19 -0500 Subject: [PATCH] chore(build): Fix lint issues (#9938) --- .../components/Modal/__tests__/Modal.test.tsx | 10 +++++++--- .../Modal/__tests__/ModalBoxHeader.test.tsx | 8 ++------ .../Modal/__tests__/ModalBoxTitle.test.tsx | 20 +++++-------------- .../Modal/__tests__/ModalContent.test.tsx | 12 ++--------- .../Modal/__tests__/ModalFooter.test.tsx | 4 +--- 5 files changed, 17 insertions(+), 37 deletions(-) diff --git a/packages/react-core/src/next/components/Modal/__tests__/Modal.test.tsx b/packages/react-core/src/next/components/Modal/__tests__/Modal.test.tsx index 7f819d0d99b..ff0e1e09f34 100644 --- a/packages/react-core/src/next/components/Modal/__tests__/Modal.test.tsx +++ b/packages/react-core/src/next/components/Modal/__tests__/Modal.test.tsx @@ -47,9 +47,9 @@ describe('Modal', () => { test('modal closes with escape', async () => { const user = userEvent.setup(); - render(); + render(); - await user.type(screen.getByLabelText("modal-div"), `{${KeyTypes.Escape}}`); + await user.type(screen.getByLabelText('modal-div'), `{${KeyTypes.Escape}}`); expect(props.onClose).toHaveBeenCalled(); }); @@ -75,7 +75,11 @@ describe('Modal', () => { }); test('modal does not show the close button when onClose not passed', () => { - render(No close button ); + render( + + No close button{' '} + + ); expect(screen.queryByRole('button', { name: 'Close' })).toBeNull(); }); diff --git a/packages/react-core/src/next/components/Modal/__tests__/ModalBoxHeader.test.tsx b/packages/react-core/src/next/components/Modal/__tests__/ModalBoxHeader.test.tsx index d3da940a1f7..5c724c9c1c3 100644 --- a/packages/react-core/src/next/components/Modal/__tests__/ModalBoxHeader.test.tsx +++ b/packages/react-core/src/next/components/Modal/__tests__/ModalBoxHeader.test.tsx @@ -16,10 +16,6 @@ test('ModalHeader help renders', () => { test('Modal Test with custom header', () => { const header = TEST; - const { asFragment } = render( - - {header} - - ); + const { asFragment } = render({header}); expect(asFragment()).toMatchSnapshot(); -}); \ No newline at end of file +}); diff --git a/packages/react-core/src/next/components/Modal/__tests__/ModalBoxTitle.test.tsx b/packages/react-core/src/next/components/Modal/__tests__/ModalBoxTitle.test.tsx index fdda1eac2f8..269d784695d 100644 --- a/packages/react-core/src/next/components/Modal/__tests__/ModalBoxTitle.test.tsx +++ b/packages/react-core/src/next/components/Modal/__tests__/ModalBoxTitle.test.tsx @@ -5,37 +5,27 @@ import { ModalBoxTitle } from '../ModalBoxTitle'; import BullhornIcon from '@patternfly/react-icons/dist/esm/icons/bullhorn-icon'; test('ModalBoxTitle alert variant', () => { - const { asFragment } = render( - - ); + const { asFragment } = render(); expect(asFragment()).toMatchSnapshot(); }); test('ModalBoxTitle info variant', () => { - const { asFragment } = render( - - ); + const { asFragment } = render(); expect(asFragment()).toMatchSnapshot(); }); test('ModalBoxTitle danger variant', () => { - const { asFragment } = render( - - ); + const { asFragment } = render(); expect(asFragment()).toMatchSnapshot(); }); test('ModalBoxTitle custom variant', () => { - const { asFragment } = render( - - ); + const { asFragment } = render(); expect(asFragment()).toMatchSnapshot(); }); test('ModalBoxTitle success variant', () => { - const { asFragment } = render( - - ); + const { asFragment } = render(); expect(asFragment()).toMatchSnapshot(); }); diff --git a/packages/react-core/src/next/components/Modal/__tests__/ModalContent.test.tsx b/packages/react-core/src/next/components/Modal/__tests__/ModalContent.test.tsx index ff5fc0aef21..a3419ee83e0 100644 --- a/packages/react-core/src/next/components/Modal/__tests__/ModalContent.test.tsx +++ b/packages/react-core/src/next/components/Modal/__tests__/ModalContent.test.tsx @@ -29,10 +29,7 @@ test('Modal Content Test isOpen', () => { test('Modal Content Test description', () => { const { asFragment } = render( - + This is a ModalBox header ); @@ -41,12 +38,7 @@ test('Modal Content Test description', () => { test('Modal Content Test with onclose', () => { const { asFragment } = render( - undefined} - isOpen - {...modalContentProps} - > + undefined} isOpen {...modalContentProps}> This is a ModalBox header ); diff --git a/packages/react-core/src/next/components/Modal/__tests__/ModalFooter.test.tsx b/packages/react-core/src/next/components/Modal/__tests__/ModalFooter.test.tsx index 2e0c256b907..09010f6f74e 100644 --- a/packages/react-core/src/next/components/Modal/__tests__/ModalFooter.test.tsx +++ b/packages/react-core/src/next/components/Modal/__tests__/ModalFooter.test.tsx @@ -3,8 +3,6 @@ import { render } from '@testing-library/react'; import { ModalFooter } from '../ModalFooter'; test('ModalFooter Test', () => { - const { asFragment } = render( - This is a ModalBox Footer - ); + const { asFragment } = render(This is a ModalBox Footer); expect(asFragment()).toMatchSnapshot(); });