test: Fix errors in Rich Text performance test

- Remove unnecessary lint warning disable line caused by erroneous
  placement of `screen` argument.
- Query UI components via user-facing attributes, e.g. a11y label.
- Pass mock function to required `onChange` prop to `RichText`
  component.

packages/block-editor/src/components/rich-text/test/index.perf-test.js

@@ -5,6 +5,7 @@ import {
 	changeTextOfRichText,
 	fireEvent,
 	measurePerformance,
+	screen,
 } from 'test/helpers';
 
 /**
@@ -13,23 +14,19 @@ import {
 import { RichText } from '@wordpress/block-editor';
 
 describe( 'RichText Performance', () => {
-	// eslint-disable-next-line jest/no-done-callback
-	it( 'performance is stable', async ( screen ) => {
+	it( 'performance is stable', async () => {
 		const scenario = async () => {
-			const richTextInput = screen.getByTestId( 'performance' );
+			const richTextInput = screen.getByLabelText( 'Text input. Empty' );
 
 			fireEvent( richTextInput, 'focus' );
 
 			changeTextOfRichText(
 				richTextInput,
 				'<strong>Bold</strong> <em>italic</em> <s>strikethrough</s> text'
 			);
-
-			// Check if the onChange is called and the state is updated.
-			expect( changeTextOfRichText ).toHaveBeenCalledTimes( 1 );
 		};
 
-		await measurePerformance( <RichText testID="performance" />, {
+		await measurePerformance( <RichText onChange={ jest.fn() } />, {
 			scenario,
 		} );
 	} );