Merge pull request #21230 from backstage/renovate/eslint-plugin-testing-library-6.x

chore(deps): update dependency eslint-plugin-testing-library to v6

.eslintrc.js

@@ -45,13 +45,12 @@ module.exports = {
           "CallExpression[arguments.length=0] > MemberExpression[property.name='toUpperCase']",
       },
     ],
-    'testing-library/await-async-query': 'error',
+    'testing-library/await-async-queries': 'error',
     'testing-library/await-async-utils': 'error',
-    'testing-library/no-await-sync-query': 'error',
-    'testing-library/prefer-wait-for': 'error',
+    'testing-library/no-await-sync-queries': 'error',
     'testing-library/no-dom-import': 'error',
     'testing-library/no-wait-for-side-effects': 'error',
-    'testing-library/no-wait-for-empty-callback': 'error',
+    'testing-library/await-async-events': 'error',
     'no-restricted-globals': [
       'error',
       [

package.json

@@ -82,7 +82,7 @@
     "eslint": "^8.6.0",
     "eslint-plugin-notice": "^0.9.10",
     "eslint-plugin-react": "^7.28.0",
-    "eslint-plugin-testing-library": "^5.9.1",
+    "eslint-plugin-testing-library": "^6.0.0",
     "fs-extra": "10.1.0",
     "husky": "^8.0.0",
     "lint-staged": "^15.0.0",

plugins/catalog-react/src/components/EntityPeekAheadPopover/EntityPeekAheadPopover.test.tsx

@@ -66,12 +66,12 @@ describe('<EntityPeekAheadPopover/>', () => {
     );
     expect(screen.getByText('s1')).toBeInTheDocument();
     expect(screen.queryByText('service1')).toBeNull();
-    user.hover(screen.getByTestId('popover1'));
+    await user.hover(screen.getByTestId('popover1'));
     expect(await screen.findByText('service1')).toBeInTheDocument();
 
     expect(screen.getByText('s2')).toBeInTheDocument();
     expect(screen.queryByText('service2')).toBeNull();
-    user.hover(screen.getByTestId('popover2'));
+    await user.hover(screen.getByTestId('popover2'));
     expect(
       await screen.findByText('Error: component:default/service2 not found'),
     ).toBeInTheDocument();

plugins/scaffolder-backend/src/scaffolder/actions/builtin/github/helpers.ts

@@ -134,7 +134,7 @@ export async function createGithubRepoWithCollaboratorsAndTopics(
   secrets: { [key: string]: string } | undefined,
   logger: Logger,
 ) {
-  // eslint-disable-next-line testing-library/no-await-sync-query
+  // eslint-disable-next-line testing-library/no-await-sync-queries
   const user = await client.rest.users.getByUsername({
     username: owner,
   });
@@ -409,7 +409,7 @@ async function validateAccessTeam(client: Octokit, access: string) {
   try {
     // Below rule disabled because of a 'getByName' check for a different library
     // incorrectly triggers here.
-    // eslint-disable-next-line testing-library/no-await-sync-query
+    // eslint-disable-next-line testing-library/no-await-sync-queries
     await client.rest.teams.getByName({
       org,
       team_slug,

yarn.lock

@@ -27074,14 +27074,14 @@ __metadata:
   languageName: node
   linkType: hard
 
-"eslint-plugin-testing-library@npm:^5.9.1":
-  version: 5.11.1
-  resolution: "eslint-plugin-testing-library@npm:5.11.1"
+"eslint-plugin-testing-library@npm:^6.0.0":
+  version: 6.1.2
+  resolution: "eslint-plugin-testing-library@npm:6.1.2"
   dependencies:
     "@typescript-eslint/utils": ^5.58.0
   peerDependencies:
     eslint: ^7.5.0 || ^8.0.0
-  checksum: 9f3fc68ef9f13016a4381b33ab5dbffcc189e5de3eaeba184bcf7d2771faa7f54e59c04b652162fb1c0f83fb52428dd909db5450a25508b94be59eba69fcc990
+  checksum: 74e6b1bdff52e9a3937be1ae0e3ddb6cfedcd97044ac5275be6e7472c77cf8ae0f6e29fd7d3b90aac1318e22850086e73ee8dde7de8d9034be59cdfed9ba7bd7
   languageName: node
   linkType: hard
 
@@ -41348,7 +41348,7 @@ __metadata:
     eslint: ^8.6.0
     eslint-plugin-notice: ^0.9.10
     eslint-plugin-react: ^7.28.0
-    eslint-plugin-testing-library: ^5.9.1
+    eslint-plugin-testing-library: ^6.0.0
     fs-extra: 10.1.0
     husky: ^8.0.0
     lint-staged: ^15.0.0