@patternfly/react-core 4.206.4 → 4.207.0

package/src/components/Modal/__tests__/__snapshots__/ModalContent.test.tsx.snap

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`Modal Content Test description 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-backdrop"
   >
@@ -48,7 +48,6 @@ exports[`Modal Content Test description 1`] = `
             class="pf-c-modal-box__title"
             id="labelId"
           >
-            
             <span
               class="pf-c-modal-box__title-text"
             >
@@ -70,11 +69,11 @@ exports[`Modal Content Test description 1`] = `
       </div>
     </div>
   </div>
-</div>
+</DocumentFragment>
 `;
 
 exports[`Modal Content Test isOpen 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-backdrop"
   >
@@ -121,7 +120,6 @@ exports[`Modal Content Test isOpen 1`] = `
             class="pf-c-modal-box__title"
             id="labelId"
           >
-            
             <span
               class="pf-c-modal-box__title-text"
             >
@@ -138,11 +136,11 @@ exports[`Modal Content Test isOpen 1`] = `
       </div>
     </div>
   </div>
-</div>
+</DocumentFragment>
 `;
 
 exports[`Modal Content Test only body 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-backdrop"
   >
@@ -189,7 +187,6 @@ exports[`Modal Content Test only body 1`] = `
             class="pf-c-modal-box__title"
             id="labelId"
           >
-            
             <span
               class="pf-c-modal-box__title-text"
             >
@@ -206,11 +203,11 @@ exports[`Modal Content Test only body 1`] = `
       </div>
     </div>
   </div>
-</div>
+</DocumentFragment>
 `;
 
 exports[`Modal Content Test with footer 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-backdrop"
   >
@@ -257,7 +254,6 @@ exports[`Modal Content Test with footer 1`] = `
             class="pf-c-modal-box__title"
             id="labelId"
           >
-            
             <span
               class="pf-c-modal-box__title-text"
             >
@@ -279,11 +275,11 @@ exports[`Modal Content Test with footer 1`] = `
       </div>
     </div>
   </div>
-</div>
+</DocumentFragment>
 `;
 
 exports[`Modal Content Test with onclose 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-backdrop"
   >
@@ -330,7 +326,6 @@ exports[`Modal Content Test with onclose 1`] = `
             class="pf-c-modal-box__title"
             id="labelId"
           >
-            
             <span
               class="pf-c-modal-box__title-text"
             >
@@ -352,11 +347,11 @@ exports[`Modal Content Test with onclose 1`] = `
       </div>
     </div>
   </div>
-</div>
+</DocumentFragment>
 `;
 
 exports[`Modal Content test without footer 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-backdrop"
   >
@@ -403,7 +398,6 @@ exports[`Modal Content test without footer 1`] = `
             class="pf-c-modal-box__title"
             id="labelId"
           >
-            
             <span
               class="pf-c-modal-box__title-text"
             >
@@ -420,11 +414,11 @@ exports[`Modal Content test without footer 1`] = `
       </div>
     </div>
   </div>
-</div>
+</DocumentFragment>
 `;
 
 exports[`Modal Test with custom footer 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-backdrop"
   >
@@ -471,7 +465,6 @@ exports[`Modal Test with custom footer 1`] = `
             class="pf-c-modal-box__title"
             id="labelId"
           >
-            
             <span
               class="pf-c-modal-box__title-text"
             >
@@ -497,11 +490,11 @@ exports[`Modal Test with custom footer 1`] = `
       </div>
     </div>
   </div>
-</div>
+</DocumentFragment>
 `;
 
 exports[`Modal Test with custom header 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-backdrop"
   >
@@ -564,5 +557,5 @@ exports[`Modal Test with custom header 1`] = `
       </div>
     </div>
   </div>
-</div>
+</DocumentFragment>
 `;

package/src/components/MultipleFileUpload/__tests__/MultipleFileUploadTitleIcon.test.tsx

@@ -4,12 +4,12 @@ import { MultipleFileUploadTitleIcon } from '../MultipleFileUploadTitleIcon';
 
 describe('MultipleFileUploadTitleIcon', () => {
   test('renders with expected class names', () => {
-    const { asFragment } = render(<MultipleFileUploadTitleIcon data-testid="test-id" />);
+    const { asFragment } = render(<MultipleFileUploadTitleIcon />);
     expect(asFragment()).toMatchSnapshot();
   });
 
   test('renders custom class names', () => {
-    const { asFragment } = render(<MultipleFileUploadTitleIcon className="test" data-testid="test-id" />);
+    const { asFragment } = render(<MultipleFileUploadTitleIcon className="test" />);
     expect(asFragment()).toMatchSnapshot();
   });
 });

package/src/components/MultipleFileUpload/__tests__/__snapshots__/MultipleFileUploadTitleIcon.test.tsx.snap

@@ -4,7 +4,6 @@ exports[`MultipleFileUploadTitleIcon renders custom class names 1`] = `
 <DocumentFragment>
   <div
     class="pf-c-multiple-file-upload__title-icon test"
-    data-testid="test-id"
   />
 </DocumentFragment>
 `;
@@ -13,7 +12,6 @@ exports[`MultipleFileUploadTitleIcon renders with expected class names 1`] = `
 <DocumentFragment>
   <div
     class="pf-c-multiple-file-upload__title-icon"
-    data-testid="test-id"
   />
 </DocumentFragment>
 `;

package/src/components/Nav/__tests__/Generated/NavExpandable.test.tsx

@@ -8,7 +8,7 @@ import { NavExpandable } from '../../NavExpandable';
 import {} from '../..';
 
 it('NavExpandable should match snapshot (auto-generated)', () => {
-  const view = render(
+  const { asFragment } = render(
     <NavExpandable
       title={'string'}
       srText={"''"}
@@ -18,8 +18,8 @@ it('NavExpandable should match snapshot (auto-generated)', () => {
       groupId={null}
       isActive={false}
       id={"''"}
-      onExpand={(e: React.MouseEvent<HTMLLIElement, MouseEvent>, val: boolean) => undefined as void}
+      onExpand={() => undefined}
     />
   );
-  expect(view.container).toMatchSnapshot();
+  expect(asFragment()).toMatchSnapshot();
 });

package/src/components/Nav/__tests__/Generated/NavGroup.test.tsx

@@ -8,6 +8,6 @@ import { NavGroup } from '../../NavGroup';
 import {} from '../..';
 
 it('NavGroup should match snapshot (auto-generated)', () => {
-  const view = render(<NavGroup title={'string'} children={<>ReactNode</>} className={"''"} id={'string'} />);
-  expect(view.container).toMatchSnapshot();
+  const { asFragment } = render(<NavGroup title={'string'} children={<>ReactNode</>} className={"''"} id={'string'} />);
+  expect(asFragment()).toMatchSnapshot();
 });

package/src/components/Nav/__tests__/Generated/NavItem.test.tsx

@@ -8,7 +8,7 @@ import { NavItem } from '../../NavItem';
 import {} from '../..';
 
 it('NavItem should match snapshot (auto-generated)', () => {
-  const view = render(
+  const { asFragment } = render(
     <NavItem
       children={<>ReactNode</>}
       className={"''"}
@@ -21,5 +21,5 @@ it('NavItem should match snapshot (auto-generated)', () => {
       component={'a'}
     />
   );
-  expect(view.container).toMatchSnapshot();
+  expect(asFragment()).toMatchSnapshot();
 });

package/src/components/Nav/__tests__/Generated/NavList.test.tsx

@@ -12,7 +12,7 @@ describe('NavList', () => {
   });
 
   it('should match snapshot', () => {
-    const view = render(
+    const { asFragment } = render(
       <NavContext.Provider
         value={{
           onSelect: jest.fn(),
@@ -26,6 +26,6 @@ describe('NavList', () => {
         <NavList children={<>ReactNode</>} className="" ariaLeftScroll="Scroll left" ariaRightScroll="Scroll right" />
       </NavContext.Provider>
     );
-    expect(view.container).toMatchSnapshot();
+    expect(asFragment()).toMatchSnapshot();
   });
 });

package/src/components/Nav/__tests__/Generated/__snapshots__/NavExpandable.test.tsx.snap

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`NavExpandable should match snapshot (auto-generated) 1`] = `
-<div>
+<DocumentFragment>
   <li
     class="pf-c-nav__item pf-m-expandable ''"
     data-ouia-component-id="OUIA-Generated-NavExpandable-1"
@@ -51,5 +51,5 @@ exports[`NavExpandable should match snapshot (auto-generated) 1`] = `
       />
     </section>
   </li>
-</div>
+</DocumentFragment>
 `;

package/src/components/Nav/__tests__/Generated/__snapshots__/NavGroup.test.tsx.snap

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`NavGroup should match snapshot (auto-generated) 1`] = `
-<div>
+<DocumentFragment>
   <section
     aria-labelledby="string"
     class="pf-c-nav__section ''"
@@ -18,5 +18,5 @@ exports[`NavGroup should match snapshot (auto-generated) 1`] = `
       ReactNode
     </ul>
   </section>
-</div>
+</DocumentFragment>
 `;

package/src/components/Nav/__tests__/Generated/__snapshots__/NavItem.test.tsx.snap

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`NavItem should match snapshot (auto-generated) 1`] = `
-<div>
+<DocumentFragment>
   <li
     class="pf-c-nav__item ''"
     data-ouia-component-id="OUIA-Generated-NavItem-1"
@@ -10,5 +10,5 @@ exports[`NavItem should match snapshot (auto-generated) 1`] = `
   >
     ReactNode
   </li>
-</div>
+</DocumentFragment>
 `;

package/src/components/Nav/__tests__/Generated/__snapshots__/NavList.test.tsx.snap

@@ -1,11 +1,11 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`NavList should match snapshot 1`] = `
-<div>
+<DocumentFragment>
   <ul
     class="pf-c-nav__list"
   >
     ReactNode
   </ul>
-</div>
+</DocumentFragment>
 `;

package/src/components/Nav/__tests__/Nav.test.tsx

@@ -108,7 +108,7 @@ describe('Nav', () => {
     );
 
     userEvent.click(screen.getByRole('button'));
-    expect(onExpand).toBeCalled();
+    expect(onExpand).toHaveBeenCalled();
   });
 
   test('Expandable Nav List - Trigger toggle', () => {

package/src/components/NotificationBadge/__tests__/NotificationBadge.test.tsx

@@ -4,19 +4,23 @@ import { render } from '@testing-library/react';
 
 Object.values([true, false]).forEach(isRead => {
   test(`${isRead} NotificationBadge`, () => {
-    const view = render(<NotificationBadge isRead={isRead} />);
-    expect(view.container).toMatchSnapshot();
+    const { asFragment } = render(<NotificationBadge isRead={isRead} />);
+    expect(asFragment()).toMatchSnapshot();
   });
 });
 
 Object.values([true, false]).forEach(attentionVariant => {
   test(`${attentionVariant} NotificationBadge needs attention`, () => {
-    const view = render(<NotificationBadge variant='attention'>{attentionVariant ? 'needs attention' : 'does not need attention'} Badge</NotificationBadge>);
-    expect(view.container).toMatchSnapshot();
+    const { asFragment } = render(
+      <NotificationBadge variant="attention">
+        {attentionVariant ? 'needs attention' : 'does not need attention'} Badge
+      </NotificationBadge>
+    );
+    expect(asFragment()).toMatchSnapshot();
   });
 });
 
 test(`NotificationBadge count`, () => {
-  const view = render(<NotificationBadge variant='read' count={3} />);
-  expect(view.container).toMatchSnapshot();
+  const { asFragment } = render(<NotificationBadge variant="read" count={3} />);
+  expect(asFragment()).toMatchSnapshot();
 });

package/src/components/NotificationBadge/__tests__/__snapshots__/NotificationBadge.test.tsx.snap

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`NotificationBadge count 1`] = `
-<div>
+<DocumentFragment>
   <button
     aria-disabled="false"
     class="pf-c-button pf-m-plain"
@@ -33,11 +33,11 @@ exports[`NotificationBadge count 1`] = `
       </span>
     </span>
   </button>
-</div>
+</DocumentFragment>
 `;
 
 exports[`false NotificationBadge 1`] = `
-<div>
+<DocumentFragment>
   <button
     aria-disabled="false"
     class="pf-c-button pf-m-plain"
@@ -64,11 +64,11 @@ exports[`false NotificationBadge 1`] = `
       </svg>
     </span>
   </button>
-</div>
+</DocumentFragment>
 `;
 
 exports[`false NotificationBadge needs attention 1`] = `
-<div>
+<DocumentFragment>
   <button
     aria-disabled="false"
     class="pf-c-button pf-m-plain"
@@ -80,15 +80,14 @@ exports[`false NotificationBadge needs attention 1`] = `
     <span
       class="pf-c-notification-badge pf-m-attention"
     >
-      does not need attention
-       Badge
+      does not need attention Badge
     </span>
   </button>
-</div>
+</DocumentFragment>
 `;
 
 exports[`true NotificationBadge 1`] = `
-<div>
+<DocumentFragment>
   <button
     aria-disabled="false"
     class="pf-c-button pf-m-plain"
@@ -115,11 +114,11 @@ exports[`true NotificationBadge 1`] = `
       </svg>
     </span>
   </button>
-</div>
+</DocumentFragment>
 `;
 
 exports[`true NotificationBadge needs attention 1`] = `
-<div>
+<DocumentFragment>
   <button
     aria-disabled="false"
     class="pf-c-button pf-m-plain"
@@ -131,9 +130,8 @@ exports[`true NotificationBadge needs attention 1`] = `
     <span
       class="pf-c-notification-badge pf-m-attention"
     >
-      needs attention
-       Badge
+      needs attention Badge
     </span>
   </button>
-</div>
+</DocumentFragment>
 `;

package/src/components/NotificationDrawer/__tests__/NotificationDrawer.test.tsx

@@ -1,7 +1,7 @@
 import React from 'react';
 
 import { render, screen } from '@testing-library/react';
-import "@testing-library/jest-dom";
+import '@testing-library/jest-dom';
 
 import { NotificationDrawer } from '../NotificationDrawer';
 

package/src/components/NotificationDrawer/__tests__/NotificationDrawerBody.test.tsx

@@ -1,7 +1,7 @@
 import React from 'react';
 
 import { render, screen } from '@testing-library/react';
-import "@testing-library/jest-dom";
+import '@testing-library/jest-dom';
 
 import { NotificationDrawerBody } from '../NotificationDrawerBody';
 

package/src/components/NotificationDrawer/__tests__/NotificationDrawerGroup.test.tsx

@@ -1,7 +1,7 @@
 import React from 'react';
 
 import { render, screen } from '@testing-library/react';
-import "@testing-library/jest-dom";
+import '@testing-library/jest-dom';
 
 import { NotificationDrawerGroup } from '../NotificationDrawerGroup';
 
@@ -12,7 +12,15 @@ describe('NotificationDrawerGroup', () => {
   });
 
   test('className is added to the root element', () => {
-    render(<NotificationDrawerGroup count={2} isExpanded={false} title="Critical Alerts" className="extra-class" data-testid='test-id' />);
+    render(
+      <NotificationDrawerGroup
+        count={2}
+        isExpanded={false}
+        title="Critical Alerts"
+        className="extra-class"
+        data-testid="test-id"
+      />
+    );
     expect(screen.getByTestId('test-id')).toHaveClass('extra-class');
   });
 
@@ -22,7 +30,9 @@ describe('NotificationDrawerGroup', () => {
   });
 
   test('drawer group with isRead applied ', () => {
-    const { asFragment } = render(<NotificationDrawerGroup count={2} isExpanded={false} isRead={true} title="Critical Alerts" />);
+    const { asFragment } = render(
+      <NotificationDrawerGroup count={2} isExpanded={false} isRead={true} title="Critical Alerts" />
+    );
     expect(asFragment()).toMatchSnapshot();
   });
 });

package/src/components/NotificationDrawer/__tests__/NotificationDrawerGroupList.test.tsx

@@ -1,7 +1,7 @@
 import React from 'react';
 
 import { render, screen } from '@testing-library/react';
-import "@testing-library/jest-dom";
+import '@testing-library/jest-dom';
 
 import { NotificationDrawerGroupList } from '../NotificationDrawerGroupList';
 

package/src/components/NotificationDrawer/__tests__/NotificationDrawerHeader.test.tsx

@@ -1,7 +1,7 @@
 import React from 'react';
 
 import { render, screen } from '@testing-library/react';
-import "@testing-library/jest-dom";
+import '@testing-library/jest-dom';
 
 import { NotificationDrawerHeader } from '../NotificationDrawerHeader';
 

package/src/components/NotificationDrawer/__tests__/NotificationDrawerList.test.tsx

@@ -1,7 +1,7 @@
 import React from 'react';
 
 import { render, screen } from '@testing-library/react';
-import "@testing-library/jest-dom";
+import '@testing-library/jest-dom';
 
 import { NotificationDrawerList } from '../NotificationDrawerList';
 

package/src/components/NotificationDrawer/__tests__/NotificationDrawerListItem.test.tsx

@@ -1,7 +1,7 @@
 import React from 'react';
 
 import { render, screen } from '@testing-library/react';
-import "@testing-library/jest-dom";
+import '@testing-library/jest-dom';
 
 import { NotificationDrawerListItem } from '../NotificationDrawerListItem';
 

package/src/components/NotificationDrawer/__tests__/NotificationDrawerListItemBody.test.tsx

@@ -1,7 +1,7 @@
 import React from 'react';
 
 import { render, screen } from '@testing-library/react';
-import "@testing-library/jest-dom";
+import '@testing-library/jest-dom';
 
 import { NotificationDrawerListItemBody } from '../NotificationDrawerListItemBody';
 

package/src/components/NotificationDrawer/__tests__/NotificationDrawerListItemHeader.test.tsx

@@ -1,7 +1,7 @@
 import React from 'react';
 
 import { render, screen } from '@testing-library/react';
-import "@testing-library/jest-dom";
+import '@testing-library/jest-dom';
 
 import BellIcon from '@patternfly/react-icons/dist/esm/icons/bell-icon';
 import { NotificationDrawerListItemHeader } from '../NotificationDrawerListItemHeader';
@@ -13,17 +13,23 @@ describe('NotificationDrawerListItemHeader', () => {
   });
 
   test('className is added to the root element', () => {
-    render(<NotificationDrawerListItemHeader title="Pod quit unexpectedly" className="extra-class" data-testid='test-id' />);
+    render(
+      <NotificationDrawerListItemHeader title="Pod quit unexpectedly" className="extra-class" data-testid="test-id" />
+    );
     expect(screen.getByTestId('test-id')).toHaveClass('extra-class');
   });
 
   test('list item header with custom icon applied ', () => {
-    const { asFragment } = render(<NotificationDrawerListItemHeader title="Pod quit unexpectedly" icon={<BellIcon />} />);
+    const { asFragment } = render(
+      <NotificationDrawerListItemHeader title="Pod quit unexpectedly" icon={<BellIcon />} />
+    );
     expect(asFragment()).toMatchSnapshot();
   });
 
   test('list item header with srTitle applied ', () => {
-    const { asFragment } = render(<NotificationDrawerListItemHeader title="Pod quit unexpectedly" srTitle="screen reader title" />);
+    const { asFragment } = render(
+      <NotificationDrawerListItemHeader title="Pod quit unexpectedly" srTitle="screen reader title" />
+    );
     expect(asFragment()).toMatchSnapshot();
   });