@patternfly/react-core 4.206.5 → 4.206.6

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

@@ -6,70 +6,70 @@ import InProgressIcon from '@patternfly/react-icons/dist/esm/icons/in-progress-i
 
 describe('ProgressStepper', () => {
   test('renders content', () => {
-    const view = render(
+    const { asFragment } = render(
       <ProgressStepper>
         <ProgressStep>First</ProgressStep>
         <ProgressStep>Second</ProgressStep>
         <ProgressStep>Third</ProgressStep>
       </ProgressStepper>
     );
-    expect(view.container).toMatchSnapshot();
+    expect(asFragment()).toMatchSnapshot();
   });
 
   test('gets custom class and id', () => {
-    const view = render(
+    const { asFragment } = render(
       <ProgressStepper className="custom-class" id="test-id">
         <ProgressStep>First</ProgressStep>
         <ProgressStep>Second</ProgressStep>
         <ProgressStep>Third</ProgressStep>
       </ProgressStepper>
     );
-    expect(view.container).toMatchSnapshot();
+    expect(asFragment()).toMatchSnapshot();
   });
 
   test('renders vertically', () => {
-    const view = render(
+    const { asFragment } = render(
       <ProgressStepper isVertical>
         <ProgressStep>First</ProgressStep>
         <ProgressStep>Second</ProgressStep>
         <ProgressStep>Third</ProgressStep>
       </ProgressStepper>
     );
-    expect(view.container).toMatchSnapshot();
+    expect(asFragment()).toMatchSnapshot();
   });
 
   test('renders compact', () => {
-    const view = render(
+    const { asFragment } = render(
       <ProgressStepper isCompact>
         <ProgressStep>First</ProgressStep>
         <ProgressStep>Second</ProgressStep>
         <ProgressStep>Third</ProgressStep>
       </ProgressStepper>
     );
-    expect(view.container).toMatchSnapshot();
+    expect(asFragment()).toMatchSnapshot();
   });
 
   test('renders center aligned', () => {
-    const view = render(
+    const { asFragment } = render(
       <ProgressStepper isCenterAligned>
         <ProgressStep>First</ProgressStep>
         <ProgressStep>Second</ProgressStep>
         <ProgressStep>Third</ProgressStep>
       </ProgressStepper>
     );
-    expect(view.container).toMatchSnapshot();
+    expect(asFragment()).toMatchSnapshot();
   });
 });
 
 describe('ProgressStep', () => {
   test('renders content', () => {
-    const view = render(<ProgressStep>Title</ProgressStep>);
-    expect(view.container).toMatchSnapshot();
+    const { asFragment } = render(<ProgressStep>Title</ProgressStep>);
+    expect(asFragment()).toMatchSnapshot();
   });
 
   Object.values(['default', 'success', 'info', 'pending', 'warning', 'danger']).forEach(variant => {
     test(`renders ${variant} variant`, () => {
-      const view = render(
+      const { asFragment } = render(
         <ProgressStep
           variant={variant as 'default' | 'success' | 'info' | 'pending' | 'warning' | 'danger'}
           aria-label={variant}
@@ -77,36 +77,36 @@ describe('ProgressStep', () => {
           {variant} step
         </ProgressStep>
       );
-      expect(view.container).toMatchSnapshot();
+      expect(asFragment()).toMatchSnapshot();
     });
   });
 
   test('renders current', () => {
-    const view = render(<ProgressStep isCurrent>Title</ProgressStep>);
-    expect(view.container).toMatchSnapshot();
+    const { asFragment } = render(<ProgressStep isCurrent>Title</ProgressStep>);
+    expect(asFragment()).toMatchSnapshot();
   });
 
   test('renders help text styling', () => {
-    const view = render(<ProgressStep popoverRender={({ stepRef }) => (<div></div>)}>Title</ProgressStep>);
-    expect(view.container).toMatchSnapshot();
+    const { asFragment } = render(<ProgressStep popoverRender={() => <div></div>}>Title</ProgressStep>);
+    expect(asFragment()).toMatchSnapshot();
   });
 
   test('renders custom icon', () => {
-    const view = render(<ProgressStep icon={<InProgressIcon />}>Title</ProgressStep>);
-    expect(view.container).toMatchSnapshot();
+    const { asFragment } = render(<ProgressStep icon={<InProgressIcon />}>Title</ProgressStep>);
+    expect(asFragment()).toMatchSnapshot();
   });
 
   test('renders custom null icon - removing default from variant', () => {
-    const view = render(
+    const { asFragment } = render(
       <ProgressStep icon={null} variant="success">
         Title
       </ProgressStep>
     );
-    expect(view.container).toMatchSnapshot();
+    expect(asFragment()).toMatchSnapshot();
   });
 
   test('renders description', () => {
-    const view = render(<ProgressStep description="This is a description">Title</ProgressStep>);
-    expect(view.container).toMatchSnapshot();
+    const { asFragment } = render(<ProgressStep description="This is a description">Title</ProgressStep>);
+    expect(asFragment()).toMatchSnapshot();
   });
 });

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

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`ProgressStep renders content 1`] = `
-<div>
+<DocumentFragment>
   <li
     class="pf-c-progress-stepper__step"
   >
@@ -22,11 +22,11 @@ exports[`ProgressStep renders content 1`] = `
       </div>
     </div>
   </li>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStep renders current 1`] = `
-<div>
+<DocumentFragment>
   <li
     aria-current="step"
     class="pf-c-progress-stepper__step pf-m-current"
@@ -48,11 +48,11 @@ exports[`ProgressStep renders current 1`] = `
       </div>
     </div>
   </li>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStep renders custom icon 1`] = `
-<div>
+<DocumentFragment>
   <li
     class="pf-c-progress-stepper__step"
   >
@@ -87,11 +87,11 @@ exports[`ProgressStep renders custom icon 1`] = `
       </div>
     </div>
   </li>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStep renders custom null icon - removing default from variant 1`] = `
-<div>
+<DocumentFragment>
   <li
     class="pf-c-progress-stepper__step pf-m-success"
   >
@@ -112,11 +112,11 @@ exports[`ProgressStep renders custom null icon - removing default from variant 1
       </div>
     </div>
   </li>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStep renders danger variant 1`] = `
-<div>
+<DocumentFragment>
   <li
     aria-label="danger"
     class="pf-c-progress-stepper__step pf-m-danger"
@@ -148,16 +148,15 @@ exports[`ProgressStep renders danger variant 1`] = `
       <div
         class="pf-c-progress-stepper__step-title"
       >
-        danger
-         step
+        danger step
       </div>
     </div>
   </li>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStep renders default variant 1`] = `
-<div>
+<DocumentFragment>
   <li
     aria-label="default"
     class="pf-c-progress-stepper__step"
@@ -175,16 +174,15 @@ exports[`ProgressStep renders default variant 1`] = `
       <div
         class="pf-c-progress-stepper__step-title"
       >
-        default
-         step
+        default step
       </div>
     </div>
   </li>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStep renders description 1`] = `
-<div>
+<DocumentFragment>
   <li
     class="pf-c-progress-stepper__step"
   >
@@ -210,11 +208,11 @@ exports[`ProgressStep renders description 1`] = `
       </div>
     </div>
   </li>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStep renders help text styling 1`] = `
-<div>
+<DocumentFragment>
   <li
     class="pf-c-progress-stepper__step"
   >
@@ -239,11 +237,11 @@ exports[`ProgressStep renders help text styling 1`] = `
       </span>
     </div>
   </li>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStep renders info variant 1`] = `
-<div>
+<DocumentFragment>
   <li
     aria-label="info"
     class="pf-c-progress-stepper__step pf-m-info"
@@ -275,16 +273,15 @@ exports[`ProgressStep renders info variant 1`] = `
       <div
         class="pf-c-progress-stepper__step-title"
       >
-        info
-         step
+        info step
       </div>
     </div>
   </li>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStep renders pending variant 1`] = `
-<div>
+<DocumentFragment>
   <li
     aria-label="pending"
     class="pf-c-progress-stepper__step pf-m-pending"
@@ -302,16 +299,15 @@ exports[`ProgressStep renders pending variant 1`] = `
       <div
         class="pf-c-progress-stepper__step-title"
       >
-        pending
-         step
+        pending step
       </div>
     </div>
   </li>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStep renders success variant 1`] = `
-<div>
+<DocumentFragment>
   <li
     aria-label="success"
     class="pf-c-progress-stepper__step pf-m-success"
@@ -343,16 +339,15 @@ exports[`ProgressStep renders success variant 1`] = `
       <div
         class="pf-c-progress-stepper__step-title"
       >
-        success
-         step
+        success step
       </div>
     </div>
   </li>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStep renders warning variant 1`] = `
-<div>
+<DocumentFragment>
   <li
     aria-label="warning"
     class="pf-c-progress-stepper__step pf-m-warning"
@@ -384,16 +379,15 @@ exports[`ProgressStep renders warning variant 1`] = `
       <div
         class="pf-c-progress-stepper__step-title"
       >
-        warning
-         step
+        warning step
       </div>
     </div>
   </li>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStepper gets custom class and id 1`] = `
-<div>
+<DocumentFragment>
   <ol
     class="pf-c-progress-stepper custom-class"
     id="test-id"
@@ -459,11 +453,11 @@ exports[`ProgressStepper gets custom class and id 1`] = `
       </div>
     </li>
   </ol>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStepper renders center aligned 1`] = `
-<div>
+<DocumentFragment>
   <ol
     class="pf-c-progress-stepper pf-m-center"
   >
@@ -528,11 +522,11 @@ exports[`ProgressStepper renders center aligned 1`] = `
       </div>
     </li>
   </ol>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStepper renders compact 1`] = `
-<div>
+<DocumentFragment>
   <ol
     class="pf-c-progress-stepper pf-m-compact"
   >
@@ -597,11 +591,11 @@ exports[`ProgressStepper renders compact 1`] = `
       </div>
     </li>
   </ol>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStepper renders content 1`] = `
-<div>
+<DocumentFragment>
   <ol
     class="pf-c-progress-stepper"
   >
@@ -666,11 +660,11 @@ exports[`ProgressStepper renders content 1`] = `
       </div>
     </li>
   </ol>
-</div>
+</DocumentFragment>
 `;
 
 exports[`ProgressStepper renders vertically 1`] = `
-<div>
+<DocumentFragment>
   <ol
     class="pf-c-progress-stepper pf-m-vertical"
   >
@@ -735,5 +729,5 @@ exports[`ProgressStepper renders vertically 1`] = `
       </div>
     </li>
   </ol>
-</div>
+</DocumentFragment>
 `;

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

@@ -2,7 +2,6 @@ import React from 'react';
 
 import { render, screen } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
-import '@testing-library/jest-dom';
 
 import { Radio } from '../Radio';
 
@@ -27,12 +26,16 @@ describe('Radio', () => {
   });
 
   test('isLabelBeforeButton', () => {
-    const { asFragment } = render(<Radio id="check" isLabelBeforeButton label="Radio label" aria-label="check" name="check" />);
+    const { asFragment } = render(
+      <Radio id="check" isLabelBeforeButton label="Radio label" aria-label="check" name="check" />
+    );
     expect(asFragment()).toMatchSnapshot();
   });
 
   test('isLabelWrapped', () => {
-    const { asFragment } = render(<Radio id="check" isLabelWrapped label="Radio label" aria-label="check" name="check" />);
+    const { asFragment } = render(
+      <Radio id="check" isLabelWrapped label="Radio label" aria-label="check" name="check" />
+    );
     expect(asFragment()).toMatchSnapshot();
   });
 
@@ -43,22 +46,30 @@ describe('Radio', () => {
 
   test('label is function', () => {
     const functionLabel = () => <h1>Header</h1>;
-    const { asFragment } = render(<Radio label={functionLabel()} id="check" isChecked aria-label="check" name="check" />);
+    const { asFragment } = render(
+      <Radio label={functionLabel()} id="check" isChecked aria-label="check" name="check" />
+    );
     expect(asFragment()).toMatchSnapshot();
   });
 
   test('label is node', () => {
-    const { asFragment } = render(<Radio label={<h1>Header</h1>} id="check" isChecked aria-label="check" name="check" />);
+    const { asFragment } = render(
+      <Radio label={<h1>Header</h1>} id="check" isChecked aria-label="check" name="check" />
+    );
     expect(asFragment()).toMatchSnapshot();
   });
 
   test('passing class', () => {
-    const { asFragment } = render(<Radio label="label" className="class-123" id="check" isChecked aria-label="check" name="check" />);
+    const { asFragment } = render(
+      <Radio label="label" className="class-123" id="check" isChecked aria-label="check" name="check" />
+    );
     expect(asFragment()).toMatchSnapshot();
   });
 
   test('passing HTML attribute', () => {
-    const { asFragment } = render(<Radio label="label" aria-labelledby="labelId" id="check" isChecked aria-label="check" name="check" />);
+    const { asFragment } = render(
+      <Radio label="label" aria-labelledby="labelId" id="check" isChecked aria-label="check" name="check" />
+    );
     expect(asFragment()).toMatchSnapshot();
   });
 
@@ -85,6 +96,6 @@ describe('Radio', () => {
 
     render(<Radio id={undefined} name="check" aria-label="check" description="Text description..." />);
 
-    expect(myMock).toBeCalled();
+    expect(myMock).toHaveBeenCalled();
   });
 });

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

@@ -2,7 +2,6 @@ import React from 'react';
 
 import { render, screen } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
-import '@testing-library/jest-dom';
 
 import { SearchInput } from '../SearchInput';
 import { FormGroup } from '../../Form';
@@ -42,13 +41,13 @@ describe('SearchInput', () => {
     expect(input.querySelector('.pf-c-badge')).toBeInTheDocument();
 
     userEvent.click(screen.getByRole('button', { name: 'Previous' }));
-    expect(props.onPreviousClick).toBeCalled();
+    expect(props.onPreviousClick).toHaveBeenCalled();
 
     userEvent.click(screen.getByRole('button', { name: 'Next' }));
-    expect(props.onNextClick).toBeCalled();
+    expect(props.onNextClick).toHaveBeenCalled();
 
     userEvent.click(screen.getByRole('button', { name: 'Reset' }));
-    expect(props.onClear).toBeCalled();
+    expect(props.onClear).toHaveBeenCalled();
   });
 
   test('hide clear button', () => {
@@ -75,7 +74,7 @@ describe('SearchInput', () => {
 
     userEvent.click(screen.getByRole('button', { name: 'Search' }));
 
-    expect(props.onSearch).toBeCalled();
+    expect(props.onSearch).toHaveBeenCalled();
     expect(asFragment()).toMatchSnapshot();
   });
 
@@ -103,7 +102,7 @@ describe('SearchInput', () => {
 
     userEvent.click(screen.getByRole('button', { name: 'Search' }));
 
-    expect(props.onSearch).toBeCalled();
+    expect(props.onSearch).toHaveBeenCalled();
     expect(asFragment()).toMatchSnapshot();
   });
 });

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

@@ -2,7 +2,6 @@ import * as React from 'react';
 
 import { configure, render, screen } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
-import '@testing-library/jest-dom';
 
 import { Select } from '../Select';
 import { SelectOption, SelectOptionObject } from '../SelectOption';
@@ -419,7 +418,7 @@ describe('API', () => {
       </Select>
     );
 
-    expect(myMock).not.toBeCalled();
+    expect(myMock).not.toHaveBeenCalled();
   });
 });
 

package/src/components/Select/__tests__/SelectGroup.test.tsx

@@ -1,21 +1,28 @@
 import React from 'react';
 
-import { render, screen } from '@testing-library/react';
+import { render } from '@testing-library/react';
 
 import { SelectGroup } from '../SelectGroup';
 import { SelectProvider } from '../selectConstants';
 
 describe('SelectGroup', () => {
   test('renders with children successfully', () => {
-    render(
+    const { asFragment } = render(
       <SelectProvider
-        value={{ onSelect: () => {}, onFavorite: () => {}, onClose: () => {}, variant: 'single', inputIdPrefix: '' }}
+        value={{
+          onSelect: () => {},
+          onFavorite: () => {},
+          onClose: () => {},
+          variant: 'single',
+          inputIdPrefix: '',
+          shouldResetOnSelect: true
+        }}
       >
-        <SelectGroup label="test" data-testid="test-id">
+        <SelectGroup label="test">
           <div>child</div>
         </SelectGroup>
       </SelectProvider>
     );
-    expect(screen.getByTestId('test-id').outerHTML).toMatchSnapshot();
+    expect(asFragment()).toMatchSnapshot();
   });
 });