@patternfly/react-core 4.202.17 → 4.202.18

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

@@ -1,19 +1,18 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`className is added to the root element 1`] = `"pf-c-notification-drawer__list extra-class"`;
-
-exports[`drawer list with hidden applied  1`] = `
-<div>
+exports[`NotificationDrawerList drawer list with hidden applied  1`] = `
+<DocumentFragment>
   <ul
     class="pf-c-notification-drawer__list"
+    hidden=""
   />
-</div>
+</DocumentFragment>
 `;
 
-exports[`renders with PatternFly Core styles 1`] = `
-<div>
+exports[`NotificationDrawerList renders with PatternFly Core styles 1`] = `
+<DocumentFragment>
   <ul
     class="pf-c-notification-drawer__list"
   />
-</div>
+</DocumentFragment>
 `;

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

@@ -1,12 +1,10 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`className is added to the root element 1`] = `"pf-c-notification-drawer__list-item pf-m-hoverable pf-m-default extra-class"`;
-
-exports[`renders with PatternFly Core styles 1`] = `
-<div>
+exports[`NotificationDrawerListItem renders with PatternFly Core styles 1`] = `
+<DocumentFragment>
   <li
     class="pf-c-notification-drawer__list-item pf-m-hoverable pf-m-default"
     tabindex="0"
   />
-</div>
+</DocumentFragment>
 `;

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

@@ -1,9 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`className is added to the root element 1`] = `undefined`;
-
-exports[`list item body with timestamp property applied  1`] = `
-<div>
+exports[`NotificationDrawerListItemBody list item body with timestamp property applied 1`] = `
+<DocumentFragment>
   <div
     class="pf-c-notification-drawer__list-item-description"
   />
@@ -12,13 +10,13 @@ exports[`list item body with timestamp property applied  1`] = `
   >
     5 minutes ago
   </div>
-</div>
+</DocumentFragment>
 `;
 
-exports[`renders with PatternFly Core styles 1`] = `
-<div>
+exports[`NotificationDrawerListItemBody renders with PatternFly Core styles 1`] = `
+<DocumentFragment>
   <div
     class="pf-c-notification-drawer__list-item-description"
   />
-</div>
+</DocumentFragment>
 `;

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

@@ -1,9 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`className is added to the root element 1`] = `"extra-class"`;
-
-exports[`list item header with custom icon applied  1`] = `
-<div>
+exports[`NotificationDrawerListItemHeader list item header with custom icon applied  1`] = `
+<DocumentFragment>
   <div
     class="pf-c-notification-drawer__list-item-header"
   >
@@ -30,11 +28,11 @@ exports[`list item header with custom icon applied  1`] = `
       Pod quit unexpectedly
     </h2>
   </div>
-</div>
+</DocumentFragment>
 `;
 
-exports[`list item header with srTitle applied  1`] = `
-<div>
+exports[`NotificationDrawerListItemHeader list item header with srTitle applied  1`] = `
+<DocumentFragment>
   <div
     class="pf-c-notification-drawer__list-item-header"
   >
@@ -66,11 +64,11 @@ exports[`list item header with srTitle applied  1`] = `
       Pod quit unexpectedly
     </h2>
   </div>
-</div>
+</DocumentFragment>
 `;
 
-exports[`list item header with variant applied  1`] = `
-<div>
+exports[`NotificationDrawerListItemHeader list item header with variant applied  1`] = `
+<DocumentFragment>
   <div
     class="pf-c-notification-drawer__list-item-header"
   >
@@ -97,11 +95,11 @@ exports[`list item header with variant applied  1`] = `
       Pod quit unexpectedly
     </h2>
   </div>
-</div>
+</DocumentFragment>
 `;
 
-exports[`renders with PatternFly Core styles 1`] = `
-<div>
+exports[`NotificationDrawerListItemHeader renders with PatternFly Core styles 1`] = `
+<DocumentFragment>
   <div
     class="pf-c-notification-drawer__list-item-header"
   >
@@ -128,5 +126,5 @@ exports[`renders with PatternFly Core styles 1`] = `
       Pod quit unexpectedly
     </h2>
   </div>
-</div>
+</DocumentFragment>
 `;

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

@@ -1,43 +1,46 @@
 import * as React from 'react';
-import { render } from '@testing-library/react';
-import { mount } from 'enzyme';
+
+import { render, screen } from '@testing-library/react';
+import '@testing-library/jest-dom';
+
 import styles from '@patternfly/react-styles/css/components/OverflowMenu/overflow-menu';
 import { OverflowMenu } from '../OverflowMenu';
 
 describe('OverflowMenu', () => {
   test('md', () => {
-    const view = mount(<OverflowMenu breakpoint="md" />);
-    expect(view.find(`.${styles.overflowMenu}`).length).toBe(1);
-    expect(view).toMatchSnapshot();
+    render(<OverflowMenu breakpoint="md" data-testid='test-id' />);
+    expect(screen.getByTestId('test-id')).toHaveClass(styles.overflowMenu);
   });
 
   test('lg', () => {
-    const view = render(<OverflowMenu breakpoint="lg" />);
-    expect(view.container).toMatchSnapshot();
+    const { asFragment } = render(<OverflowMenu breakpoint="lg" />);
+    expect(asFragment()).toMatchSnapshot();
   });
 
   test('xl', () => {
-    const view = render(<OverflowMenu breakpoint="xl" />);
-    expect(view.container).toMatchSnapshot();
+    const { asFragment } = render(<OverflowMenu breakpoint="xl" />);
+    expect(asFragment()).toMatchSnapshot();
   });
 
   test('basic', () => {
-    const view = render(
+    const { asFragment } = render(
       <OverflowMenu breakpoint="md">
         <div>BASIC</div>
       </OverflowMenu>
     );
-    expect(view.container).toMatchSnapshot();
+    expect(asFragment()).toMatchSnapshot();
   });
 
   test('should warn on bad props', () => {
     const myMock = jest.fn() as any;
     global.console = { error: myMock } as any;
+
     render(
-      <OverflowMenu breakpoint={undefined as "md"}>
+      <OverflowMenu breakpoint={undefined as 'md'}>
         <div>BASIC</div>
       </OverflowMenu>
     );
+
     expect(myMock).toBeCalled();
   });
 });

package/src/components/OverflowMenu/__tests__/OverflowMenuControl.test.tsx

@@ -1,23 +1,24 @@
 import * as React from 'react';
-import { render } from '@testing-library/react';
-import { mount } from 'enzyme';
+
+import { render, screen } from '@testing-library/react';
+import '@testing-library/jest-dom';
+
 import styles from '@patternfly/react-styles/css/components/OverflowMenu/overflow-menu';
 import { OverflowMenuControl } from '../OverflowMenuControl';
 import { OverflowMenuContext } from '../OverflowMenuContext';
 
 describe('OverflowMenuControl', () => {
   test('basic', () => {
-    const view = mount(
+    render(
       <OverflowMenuContext.Provider value={{ isBelowBreakpoint: true }}>
-        <OverflowMenuControl />
+        <OverflowMenuControl data-testid="test-id" />
       </OverflowMenuContext.Provider>
     );
-    expect(view.find(`.${styles.overflowMenuControl}`).length).toBe(1);
-    expect(view).toMatchSnapshot();
+    expect(screen.getByTestId('test-id')).toHaveClass(styles.overflowMenuControl);
   });
 
   test('Additional Options', () => {
-    const view = render(<OverflowMenuControl hasAdditionalOptions />);
-    expect(view.container).toMatchSnapshot();
+    const { asFragment } = render(<OverflowMenuControl hasAdditionalOptions />);
+    expect(asFragment()).toMatchSnapshot();
   });
 });

package/src/components/OverflowMenu/__tests__/OverflowMenuGroup.test.tsx

@@ -1,47 +1,46 @@
-import * as React from 'react';
-import { mount } from 'enzyme';
+import React from 'react';
+
+import { render, screen } from '@testing-library/react';
+import '@testing-library/jest-dom';
+
 import styles from '@patternfly/react-styles/css/components/OverflowMenu/overflow-menu';
 import { OverflowMenuGroup } from '../OverflowMenuGroup';
 import { OverflowMenuContext } from '../OverflowMenuContext';
 
 describe('OverflowMenuGroup', () => {
   test('isPersistent and below breakpoint should still show', () => {
-    const view = mount(
+    render(
       <OverflowMenuContext.Provider value={{ isBelowBreakpoint: true }}>
-        <OverflowMenuGroup isPersistent />
+        <OverflowMenuGroup isPersistent data-testid="test-id" />
       </OverflowMenuContext.Provider>
     );
-    expect(view.find(`.${styles.overflowMenuGroup}`).length).toBe(1);
-    expect(view).toMatchSnapshot();
+    expect(screen.getByTestId('test-id')).toHaveClass(styles.overflowMenuGroup);
   });
 
   test('Below breakpoint but not isPersistent should not show', () => {
-    const view = mount(
+    render(
       <OverflowMenuContext.Provider value={{ isBelowBreakpoint: true }}>
-        <OverflowMenuGroup />
+        <OverflowMenuGroup data-testid="test-id" />
       </OverflowMenuContext.Provider>
     );
-    expect(view.find(`.${styles.overflowMenuGroup}`).length).toBe(0);
-    expect(view).toMatchSnapshot();
+    expect(screen.queryByTestId('test-id')).toBeNull();
   });
 
   test('Button group', () => {
-    const view = mount(
+    render(
       <OverflowMenuContext.Provider value={{ isBelowBreakpoint: false }}>
-        <OverflowMenuGroup groupType="button" />
+        <OverflowMenuGroup groupType="button" data-testid="test-id" />
       </OverflowMenuContext.Provider>
     );
-    expect(view.find(`.${styles.modifiers.buttonGroup}`).length).toBe(1);
-    expect(view).toMatchSnapshot();
+    expect(screen.getByTestId('test-id')).toHaveClass(styles.modifiers.buttonGroup);
   });
 
   test('Icon group', () => {
-    const view = mount(
+    render(
       <OverflowMenuContext.Provider value={{ isBelowBreakpoint: false }}>
-        <OverflowMenuGroup groupType="icon" />
+        <OverflowMenuGroup groupType="icon" data-testid="test-id" />
       </OverflowMenuContext.Provider>
     );
-    expect(view.find(`.${styles.modifiers.iconButtonGroup}`).length).toBe(1);
-    expect(view).toMatchSnapshot();
+    expect(screen.getByTestId('test-id')).toHaveClass(styles.modifiers.iconButtonGroup);
   });
 });

package/src/components/OverflowMenu/__tests__/OverflowMenuItem.test.tsx

@@ -1,27 +1,28 @@
 import * as React from 'react';
-import { render } from '@testing-library/react';
-import { mount } from 'enzyme';
+
+import { render, screen } from '@testing-library/react';
+import '@testing-library/jest-dom';
+
 import styles from '@patternfly/react-styles/css/components/OverflowMenu/overflow-menu';
 import { OverflowMenuItem } from '../OverflowMenuItem';
 import { OverflowMenuContext } from '../OverflowMenuContext';
 
 describe('OverflowMenuItem', () => {
   test('isPersistent and below breakpoint should still show', () => {
-    const view = mount(
+    render(
       <OverflowMenuContext.Provider value={{ isBelowBreakpoint: false }}>
-        <OverflowMenuItem isPersistent />
+        <OverflowMenuItem isPersistent>Some item value</OverflowMenuItem>
       </OverflowMenuContext.Provider>
     );
-    expect(view.find(`.${styles.overflowMenuItem}`).length).toBe(1);
-    expect(view).toMatchSnapshot();
+    expect(screen.getByText('Some item value')).toHaveClass(styles.overflowMenuItem);
   });
 
   test('Below breakpoint and not isPersistent should not show', () => {
-    const view = render(
+    const { asFragment } = render(
       <OverflowMenuContext.Provider value={{ isBelowBreakpoint: false }}>
-        <OverflowMenuItem />
+        <OverflowMenuItem>Some item value</OverflowMenuItem>
       </OverflowMenuContext.Provider>
     );
-    expect(view.container).toMatchSnapshot();
+    expect(asFragment()).toMatchSnapshot();
   });
 });

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

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`OverflowMenu basic 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-overflow-menu"
   >
@@ -9,31 +9,21 @@ exports[`OverflowMenu basic 1`] = `
       BASIC
     </div>
   </div>
-</div>
+</DocumentFragment>
 `;
 
 exports[`OverflowMenu lg 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-overflow-menu"
   />
-</div>
-`;
-
-exports[`OverflowMenu md 1`] = `
-<OverflowMenu
-  breakpoint="md"
->
-  <div
-    className="pf-c-overflow-menu"
-  />
-</OverflowMenu>
+</DocumentFragment>
 `;
 
 exports[`OverflowMenu xl 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-overflow-menu"
   />
-</div>
+</DocumentFragment>
 `;

package/src/components/OverflowMenu/__tests__/__snapshots__/OverflowMenuControl.test.tsx.snap

@@ -1,23 +1,11 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`OverflowMenuControl Additional Options 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-overflow-menu__control"
   >
-     
-     
+      
   </div>
-</div>
-`;
-
-exports[`OverflowMenuControl basic 1`] = `
-<OverflowMenuControl>
-  <div
-    className="pf-c-overflow-menu__control"
-  >
-     
-     
-  </div>
-</OverflowMenuControl>
+</DocumentFragment>
 `;

package/src/components/OverflowMenu/__tests__/__snapshots__/OverflowMenuItem.test.tsx.snap

@@ -1,25 +1,11 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`OverflowMenuItem Below breakpoint and not isPersistent should not show 1`] = `
-<div>
+<DocumentFragment>
   <div
     class="pf-c-overflow-menu__item"
   >
-     
-     
+     Some item value 
   </div>
-</div>
-`;
-
-exports[`OverflowMenuItem isPersistent and below breakpoint should still show 1`] = `
-<OverflowMenuItem
-  isPersistent={true}
->
-  <div
-    className="pf-c-overflow-menu__item"
-  >
-     
-     
-  </div>
-</OverflowMenuItem>
+</DocumentFragment>
 `;

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

@@ -1,16 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`ModalBoxCloseButton Test 1`] = `
-<Button
-  aria-label="Close"
-  className="test-box-close-button-class"
-  onClick={[MockFunction]}
-  variant="plain"
->
-  <TimesIcon
-    color="currentColor"
-    noVerticalAlign={false}
-    size="sm"
-  />
-</Button>
-`;

package/src/components/OverflowMenu/__tests__/__snapshots__/OverflowMenuGroup.test.tsx.snap

@@ -1,33 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`OverflowMenuGroup Below breakpoint but not isPersistent should not show 1`] = `<OverflowMenuGroup />`;
-
-exports[`OverflowMenuGroup Button group 1`] = `
-<OverflowMenuGroup
-  groupType="button"
->
-  <div
-    className="pf-c-overflow-menu__group pf-m-button-group"
-  />
-</OverflowMenuGroup>
-`;
-
-exports[`OverflowMenuGroup Icon group 1`] = `
-<OverflowMenuGroup
-  groupType="icon"
->
-  <div
-    className="pf-c-overflow-menu__group pf-m-icon-button-group"
-  />
-</OverflowMenuGroup>
-`;
-
-exports[`OverflowMenuGroup isPersistent and below breakpoint should still show 1`] = `
-<OverflowMenuGroup
-  isPersistent={true}
->
-  <div
-    className="pf-c-overflow-menu__group"
-  />
-</OverflowMenuGroup>
-`;