refactor: Update new tests to match updated testing approach

The tests were authored prior to these new practices were established.
WordPress · Sep 14, 2022 · f804f04 · f804f04
1 parent bd255ac
commit f804f04
Showing 1 changed file with 12 additions and 17 deletions.
diff --git a/packages/data/src/components/use-select/test/index.js b/packages/data/src/components/use-select/test/index.js
@@ -439,7 +439,7 @@ describe( 'useSelect', () => {
 			registry.registerStore( 'store-2', counterStore );
 			registry.registerStore( 'store-3', counterStore );
 
-			let renderer, childShouldDispatch, setChildShouldDispatch;
+			let childShouldDispatch, setChildShouldDispatch;
 			const selectCount1AndDep = jest.fn();
 
 			class ChildComponent extends Component {
@@ -459,9 +459,8 @@ describe( 'useSelect', () => {
 			}
 
 			const TestComponent = jest.fn( () => {
-				[ childShouldDispatch, setChildShouldDispatch ] = useState(
-					false
-				);
+				[ childShouldDispatch, setChildShouldDispatch ] =
+					useState( false );
 				const state = useSelect(
 					( select ) =>
 						selectCount1AndDep() || {
@@ -472,32 +471,28 @@ describe( 'useSelect', () => {
 
 				return (
 					<>
-						<div data={ state } />
+						<div role="status">count1:{ state.count1 }</div>
 						<ChildComponent
 							childShouldDispatch={ childShouldDispatch }
 						/>
 					</>
 				);
 			} );
 
-			act( () => {
-				renderer = TestRenderer.create(
-					<RegistryProvider value={ registry }>
-						<TestComponent />
-					</RegistryProvider>
-				);
-			} );
+			const rendered = render(
+				<RegistryProvider value={ registry }>
+					<TestComponent />
+				</RegistryProvider>
+			);
 
 			act( () => {
 				setChildShouldDispatch( true );
 			} );
 
-			const testInstance = renderer.root;
-
 			expect( selectCount1AndDep ).toHaveBeenCalledTimes( 3 );
-			expect( testInstance.findByType( 'div' ).props.data ).toEqual( {
-				count1: 1,
-			} );
+			expect( rendered.getByRole( 'status' ) ).toHaveTextContent(
+				'count1:1'
+			);
 		} );
 
 		it( 'handles registry selectors', () => {