From 82eccd402ad8d87c28a30ca5f4713200b7bf5175 Mon Sep 17 00:00:00 2001 From: Marin Atanasov <8436925+tyxla@users.noreply.github.com> Date: Mon, 10 Oct 2022 16:51:41 +0200 Subject: [PATCH] Components: Refactor TreeGrid RovingTabIndexItem tests to RTL (#44821) --- .../roving-tab-index-item.js.snap | 32 ++++++++++--------- .../tree-grid/test/roving-tab-index-item.js | 16 +++++----- 2 files changed, 25 insertions(+), 23 deletions(-) diff --git a/packages/components/src/tree-grid/test/__snapshots__/roving-tab-index-item.js.snap b/packages/components/src/tree-grid/test/__snapshots__/roving-tab-index-item.js.snap index 73aa1411cffc1..5e803bdd95c85 100644 --- a/packages/components/src/tree-grid/test/__snapshots__/roving-tab-index-item.js.snap +++ b/packages/components/src/tree-grid/test/__snapshots__/roving-tab-index-item.js.snap @@ -1,25 +1,27 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP exports[`RovingTabIndexItem allows another component to be specified as the rendered component using the \`as\` prop 1`] = ` - +
+ +
`; exports[`RovingTabIndexItem forwards props to the \`as\` component 1`] = ` - +
+ +
`; diff --git a/packages/components/src/tree-grid/test/roving-tab-index-item.js b/packages/components/src/tree-grid/test/roving-tab-index-item.js index 74e071a71881e..0f2c6bd58c7c6 100644 --- a/packages/components/src/tree-grid/test/roving-tab-index-item.js +++ b/packages/components/src/tree-grid/test/roving-tab-index-item.js @@ -1,7 +1,7 @@ /** * External dependencies */ -import TestRenderer from 'react-test-renderer'; +import { render } from '@testing-library/react'; /** * WordPress dependencies @@ -21,23 +21,23 @@ const TestButton = forwardRef( ( { ...props }, ref ) => ( describe( 'RovingTabIndexItem', () => { it( 'requires RovingTabIndex to be declared as a parent component somewhere in the component hierarchy', () => { expect( () => - TestRenderer.create( ) + render( ) ).toThrow(); expect( console ).toHaveErrored(); } ); it( 'allows another component to be specified as the rendered component using the `as` prop', () => { - const renderer = TestRenderer.create( + const { container } = render( ); - expect( renderer.toJSON() ).toMatchSnapshot(); + expect( container ).toMatchSnapshot(); } ); it( 'forwards props to the `as` component', () => { - const renderer = TestRenderer.create( + const { container } = render( Click Me! @@ -45,11 +45,11 @@ describe( 'RovingTabIndexItem', () => { ); - expect( renderer.toJSON() ).toMatchSnapshot(); + expect( container ).toMatchSnapshot(); } ); it( 'allows children to be declared using a child render function as an alternative to `as`', () => { - const renderer = TestRenderer.create( + const { container } = render( { ( props ) => ( @@ -61,6 +61,6 @@ describe( 'RovingTabIndexItem', () => { ); - expect( renderer.toJSON() ).toMatchSnapshot(); + expect( container ).toMatchSnapshot(); } ); } );