From adb94ad82ef4ac2e61f7cbf376726ded7b66cd35 Mon Sep 17 00:00:00 2001 From: Andrea Fercia Date: Mon, 22 Apr 2024 09:45:16 +0200 Subject: [PATCH] Adjust test. --- .../src/components/link-control/test/index.js | 45 +++++++++---------- 1 file changed, 21 insertions(+), 24 deletions(-) diff --git a/packages/block-editor/src/components/link-control/test/index.js b/packages/block-editor/src/components/link-control/test/index.js index e416b9b522a53c..dc8291fc9a1933 100644 --- a/packages/block-editor/src/components/link-control/test/index.js +++ b/packages/block-editor/src/components/link-control/test/index.js @@ -373,7 +373,7 @@ describe( 'Basic rendering', () => { /> ); - const linkPreview = screen.getByLabelText( 'Currently selected' ); + const linkPreview = screen.getByLabelText( 'Manage link' ); const isPreviewError = linkPreview.classList.contains( 'is-error' ); expect( isPreviewError ).toBe( true ); @@ -834,7 +834,7 @@ describe( 'Manual link entry', () => { render( ); - let linkPreview = screen.getByLabelText( 'Currently selected' ); + let linkPreview = screen.getByLabelText( 'Manage link' ); expect( linkPreview ).toBeInTheDocument(); @@ -868,7 +868,7 @@ describe( 'Manual link entry', () => { // Cancel the editing process. await user.click( cancelButton ); - linkPreview = screen.getByLabelText( 'Currently selected' ); + linkPreview = screen.getByLabelText( 'Manage link' ); expect( linkPreview ).toBeInTheDocument(); @@ -1076,7 +1076,7 @@ describe( 'Default search suggestions', () => { // Click the "Edit/Change" button and check initial suggestions are not // shown. - const currentLinkUI = screen.getByLabelText( 'Currently selected' ); + const currentLinkUI = screen.getByLabelText( 'Manage link' ); const currentLinkBtn = within( currentLinkUI ).getByRole( 'button', { name: 'Edit link', } ); @@ -1230,8 +1230,7 @@ describe( 'Creating Entities (eg: Posts, Pages)', () => { // Check for loading indicator. const loadingIndicator = screen.getByText( 'Creating…' ); - const currentLinkLabel = - screen.queryByLabelText( 'Currently selected' ); + const currentLinkLabel = screen.queryByLabelText( 'Manage link' ); expect( currentLinkLabel ).not.toBeInTheDocument(); expect( loadingIndicator ).toBeVisible(); @@ -1242,8 +1241,7 @@ describe( 'Creating Entities (eg: Posts, Pages)', () => { // Resolve the `createSuggestion` promise. resolver(); - const currentLink = - await screen.findByLabelText( 'Currently selected' ); + const currentLink = await screen.findByLabelText( 'Manage link' ); expect( currentLink ).toHaveTextContent( entityNameText ); expect( currentLink ).toHaveTextContent( '/?p=123' ); @@ -1291,7 +1289,7 @@ describe( 'Creating Entities (eg: Posts, Pages)', () => { await user.click( createButton ); - const currentLink = screen.getByLabelText( 'Currently selected' ); + const currentLink = screen.getByLabelText( 'Manage link' ); expect( currentLink ).toHaveTextContent( 'Some new page to create' ); expect( currentLink ).toHaveTextContent( '/?p=123' ); @@ -1350,7 +1348,7 @@ describe( 'Creating Entities (eg: Posts, Pages)', () => { triggerEnter( searchInput ); expect( - await screen.findByLabelText( 'Currently selected' ) + await screen.findByLabelText( 'Manage link' ) ).toHaveTextContent( entityNameText ); } ); @@ -1529,7 +1527,7 @@ describe( 'Selecting links', () => { render( ); - const currentLink = screen.getByLabelText( 'Currently selected' ); + const currentLink = screen.getByLabelText( 'Manage link' ); const currentLinkAnchor = screen.getByRole( 'link', { name: `${ selectedLink.title } (opens in a new tab)`, } ); @@ -1559,7 +1557,7 @@ describe( 'Selecting links', () => { render( ); // Required in order to select the button below. - let currentLinkUI = screen.getByLabelText( 'Currently selected' ); + let currentLinkUI = screen.getByLabelText( 'Manage link' ); const currentLinkBtn = within( currentLinkUI ).getByRole( 'button', { name: 'Edit link', } ); @@ -1570,7 +1568,7 @@ describe( 'Selecting links', () => { const searchInput = screen.getByRole( 'combobox', { name: 'Search or type URL', } ); - currentLinkUI = screen.queryByLabelText( 'Currently selected' ); + currentLinkUI = screen.queryByLabelText( 'Manage link' ); // We should be back to showing the search input. expect( searchInput ).toBeVisible(); @@ -1733,8 +1731,7 @@ describe( 'Selecting links', () => { triggerEnter( searchInput ); // Check that the suggestion selected via is now shown as selected. - const currentLink = - screen.getByLabelText( 'Currently selected' ); + const currentLink = screen.getByLabelText( 'Manage link' ); const currentLinkAnchor = screen.getByRole( 'link', { name: `${ selectedLink.title } (opens in a new tab)`, } ); @@ -2127,7 +2124,7 @@ describe( 'Rich link previews', () => { render( ); - const linkPreview = screen.getByLabelText( 'Currently selected' ); + const linkPreview = screen.getByLabelText( 'Manage link' ); const isRichLinkPreview = linkPreview.classList.contains( 'is-rich' ); @@ -2148,7 +2145,7 @@ describe( 'Rich link previews', () => { render( ); - const linkPreview = screen.getByLabelText( 'Currently selected' ); + const linkPreview = screen.getByLabelText( 'Manage link' ); await waitFor( () => expect( linkPreview ).toHaveClass( 'is-rich' ) ); } ); @@ -2165,7 +2162,7 @@ describe( 'Rich link previews', () => { render( ); - const linkPreview = screen.getByLabelText( 'Currently selected' ); + const linkPreview = screen.getByLabelText( 'Manage link' ); await waitFor( () => expect( linkPreview ).toHaveClass( 'is-rich' ) ); @@ -2197,7 +2194,7 @@ describe( 'Rich link previews', () => { render( ); - const linkPreview = screen.getByLabelText( 'Currently selected' ); + const linkPreview = screen.getByLabelText( 'Manage link' ); await waitFor( () => expect( linkPreview ).toHaveClass( 'is-rich' ) ); @@ -2221,7 +2218,7 @@ describe( 'Rich link previews', () => { render( ); - const linkPreview = screen.getByLabelText( 'Currently selected' ); + const linkPreview = screen.getByLabelText( 'Manage link' ); await waitFor( () => expect( linkPreview ).toHaveClass( 'is-rich' ) ); @@ -2256,7 +2253,7 @@ describe( 'Rich link previews', () => { render( ); - const linkPreview = screen.getByLabelText( 'Currently selected' ); + const linkPreview = screen.getByLabelText( 'Manage link' ); await waitFor( () => expect( linkPreview ).toHaveClass( 'is-rich' ) @@ -2281,7 +2278,7 @@ describe( 'Rich link previews', () => { render( ); - const linkPreview = screen.getByLabelText( 'Currently selected' ); + const linkPreview = screen.getByLabelText( 'Manage link' ); expect( linkPreview ).toHaveClass( 'is-fetching' ); @@ -2300,7 +2297,7 @@ describe( 'Rich link previews', () => { render( ); - const linkPreview = screen.getByLabelText( 'Currently selected' ); + const linkPreview = screen.getByLabelText( 'Manage link' ); expect( linkPreview ).toHaveClass( 'is-fetching' ); expect( linkPreview ).not.toHaveClass( 'is-rich' ); @@ -2313,7 +2310,7 @@ describe( 'Rich link previews', () => { render( ); - const linkPreview = screen.getByLabelText( 'Currently selected' ); + const linkPreview = screen.getByLabelText( 'Manage link' ); expect( linkPreview ).toHaveClass( 'is-fetching' );