@wordpress/e2e-tests 4.6.0 → 4.7.0

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@wordpress/e2e-tests",
-	"version": "4.6.0",
+	"version": "4.7.0",
 	"description": "End-To-End (E2E) tests for WordPress.",
 	"author": "The WordPress Contributors",
 	"license": "GPL-2.0-or-later",
@@ -23,11 +23,11 @@
 		"node": ">=12"
 	},
 	"dependencies": {
-		"@wordpress/e2e-test-utils": "^7.8.0",
-		"@wordpress/jest-console": "^5.3.0",
+		"@wordpress/e2e-test-utils": "^7.9.0",
+		"@wordpress/jest-console": "^5.4.0",
 		"@wordpress/jest-puppeteer-axe": "^4.0.2",
-		"@wordpress/scripts": "^23.4.0",
-		"@wordpress/url": "^3.13.0",
+		"@wordpress/scripts": "^23.5.0",
+		"@wordpress/url": "^3.14.0",
 		"chalk": "^4.0.0",
 		"expect-puppeteer": "^4.4.0",
 		"filenamify": "^4.2.0",
@@ -46,5 +46,5 @@
 	"publishConfig": {
 		"access": "public"
 	},
-	"gitHead": "a80eeb62ec7cb1418b9915c277e084a29d6665e3"
+	"gitHead": "9d9d33bbdf317a4381b8ca1713e43bb50df653b3"
 }

package/specs/editor/blocks/gallery.test.js

@@ -103,7 +103,7 @@ describe( 'Gallery', () => {
 		// This xpath selects the anchor node for the block which has a child span which contains the text
 		// label of the block and then selects the expander span for that node.
 		const galleryExpander = await page.waitForXPath(
-			`//a[span[text()='Gallery']]/span[contains(@class, 'block-editor-list-view__expander')]`
+			`//a[.//span[text()='Gallery']]/span[contains(@class, 'block-editor-list-view__expander')]`
 		);
 
 		await galleryExpander.click();

package/specs/editor/blocks/navigation.test.js

@@ -1047,7 +1047,7 @@ Expected mock function not to be called but it was called with: ["POST", "http:/
 			await openListView();
 
 			const navExpander = await page.waitForXPath(
-				`//a[span[text()='Navigation']]/span[contains(@class, 'block-editor-list-view__expander')]`
+				`//a[.//span[text()='Navigation']]/span[contains(@class, 'block-editor-list-view__expander')]`
 			);
 
 			await navExpander.click();
@@ -1317,42 +1317,6 @@ Expected mock function not to be called but it was called with: ["POST", "http:/
 			await switchUserToAdmin();
 		} );
 
-		it( 'shows a warning if user does not have permission to edit or update navigation menus', async () => {
-			await createNewPost();
-			await insertBlock( 'Navigation' );
-
-			const startEmptyButton = await page.waitForXPath(
-				START_EMPTY_XPATH
-			);
-
-			// This creates an empty Navigation post type entity.
-			await startEmptyButton.click();
-
-			// Publishing the Post ensures the Navigation entity is saved.
-			// The Post itself is irrelevant.
-			await publishPost();
-
-			// Switch to a Contributor role user - they should not have
-			// permission to update Navigation menus.
-			await loginUser( contributorUsername, contributorPassword );
-
-			await createNewPost();
-
-			// At this point the block will automatically pick the first Navigation Menu
-			// which will be the one created by the Admin User.
-			await insertBlock( 'Navigation' );
-
-			// Make sure the snackbar error shows up.
-			await page.waitForXPath(
-				`//*[contains(@class, 'components-snackbar__content')][ text()="You do not have permission to edit this Menu. Any changes made will not be saved." ]`
-			);
-
-			// Expect a console 403 for requests to:
-			// * /wp/v2/settings?_locale=user
-			// * /wp/v2/templates?context=edit&post_type=post&per_page=100&_locale=user
-			expect( console ).toHaveErrored();
-		} );
-
 		it( 'shows a warning if user does not have permission to create navigation menus', async () => {
 			const noticeText =
 				'You do not have permission to create Navigation Menus.';

package/specs/editor/plugins/meta-boxes.test.js

@@ -59,7 +59,7 @@ describe( 'Meta boxes', () => {
 		await viewPostLinks[ 0 ].click();
 		await page.waitForNavigation();
 
-		// Check the the dynamic block appears.
+		// Check the dynamic block appears.
 		const latestPostsBlock = await page.waitForSelector(
 			'.wp-block-latest-posts'
 		);

package/specs/editor/various/compatibility-classic-editor.test.js

@@ -28,7 +28,7 @@ describe( 'Compatibility with classic editor', () => {
 		await viewPostLinks[ 0 ].click();
 		await page.waitForNavigation();
 
-		// Check the the content doesn't contain <p> tags.
+		// Check the content doesn't contain <p> tags.
 		await page.waitForSelector( '.entry-content' );
 		const content = await page.$eval( '.entry-content', ( element ) =>
 			element.innerHTML.trim()

package/specs/editor/various/links.test.js

@@ -392,7 +392,7 @@ describe( 'Links', () => {
 		await page.waitForSelector( '.block-editor-link-control__search-item' );
 		await page.keyboard.press( 'ArrowDown' );
 
-		// Expect the the escape key to dismiss the popover when the autocomplete suggestion list is open.
+		// Expect the escape key to dismiss the popover when the autocomplete suggestion list is open.
 		await page.keyboard.press( 'Escape' );
 		expect(
 			await page.$(
@@ -417,7 +417,7 @@ describe( 'Links', () => {
 			)
 		).not.toBeNull();
 
-		// Expect the the escape key to dismiss the popover normally.
+		// Expect the escape key to dismiss the popover normally.
 		await page.keyboard.press( 'Escape' );
 		expect(
 			await page.$(
@@ -440,7 +440,7 @@ describe( 'Links', () => {
 		await page.keyboard.press( 'Tab' );
 		await page.keyboard.press( 'Tab' );
 
-		// Expect the the escape key to dismiss the popover normally.
+		// Expect the escape key to dismiss the popover normally.
 		await page.keyboard.press( 'Escape' );
 		expect(
 			await page.$(

package/specs/editor/various/multi-block-selection.test.js

@@ -108,7 +108,6 @@ describe( 'Multi-block selection', () => {
 		await pressKeyWithModifier( 'primary', 'a' );
 		await pressKeyWithModifier( 'primary', 'a' );
 
-		await testNativeSelection();
 		expect( await getSelectedFlatIndices() ).toEqual( [ 1, 2, 3 ] );
 
 		// TODO: It would be great to do this test by spying on `wp.a11y.speak`,
@@ -420,7 +419,6 @@ describe( 'Multi-block selection', () => {
 		await page.mouse.move( coord2.x, coord2.y, { steps: 10 } );
 		await page.mouse.up();
 
-		await testNativeSelection();
 		expect( await getSelectedFlatIndices() ).toEqual( [ 1, 2 ] );
 	} );
 

package/specs/editor/various/post-visibility.test.js

@@ -19,7 +19,7 @@ describe( 'Post visibility', () => {
 
 			await openDocumentSettingsSidebar();
 
-			await page.click( '.edit-post-post-visibility__toggle' );
+			await page.click( '*[aria-label^="Select visibility"]' );
 
 			const [ privateLabel ] = await page.$x(
 				'//label[text()="Private"]'
@@ -57,7 +57,7 @@ describe( 'Post visibility', () => {
 					.getEditedPostAttribute( 'status' );
 			} );
 
-			await page.click( '.edit-post-post-visibility__toggle' );
+			await page.click( '*[aria-label^="Select visibility"]' );
 
 			const [ privateLabel ] = await page.$x(
 				'//label[text()="Private"]'
@@ -90,7 +90,7 @@ describe( 'Post visibility', () => {
 		await openDocumentSettingsSidebar();
 
 		// Set a publish date for the next month.
-		await page.click( '.edit-post-post-schedule__toggle' );
+		await page.click( '*[aria-label^="Change date"]' );
 		await page.click(
 			'*[aria-label="Move forward to switch to the next month."]'
 		);
@@ -100,7 +100,7 @@ describe( 'Post visibility', () => {
 			)
 		 )[ 0 ].click();
 
-		await page.click( '.edit-post-post-visibility__toggle' );
+		await page.click( '*[aria-label^="Select visibility"]' );
 
 		const [ privateLabel ] = await page.$x( '//label[text()="Private"]' );
 		await privateLabel.click();

package/specs/editor/various/scheduling.test.js

@@ -39,7 +39,7 @@ describe( 'Scheduling', () => {
 				expect( await getPublishButtonText() ).toBe( 'Publish' );
 
 				// Open the datepicker.
-				await page.click( '.edit-post-post-schedule__toggle' );
+				await page.click( '*[aria-label^="Change date"]' );
 
 				// Change the publishing date to a year in the future.
 				await page.click( '.components-datetime__time-field-year' );
@@ -56,7 +56,7 @@ describe( 'Scheduling', () => {
 	it( 'Should keep date time UI focused when the previous and next month buttons are clicked', async () => {
 		await createNewPost();
 
-		await page.click( '.edit-post-post-schedule__toggle' );
+		await page.click( '*[aria-label^="Change date"]' );
 		await page.click(
 			'*[aria-label="Move backward to switch to the previous month."]'
 		);

package/specs/editor/various/{sidebar-permalink-panel.test.js → sidebar-permalink.test.js}

@@ -5,17 +5,16 @@ import {
 	activatePlugin,
 	createNewPost,
 	deactivatePlugin,
-	findSidebarPanelWithTitle,
 	publishPost,
 } from '@wordpress/e2e-test-utils';
 
-const permalinkPanelXPath = `//div[contains(@class, "edit-post-sidebar")]//button[contains(@class, "components-panel__body-toggle") and contains(text(),"Permalink")]`;
+const urlButtonSelector = '*[aria-label^="Change URL"]';
 
 // This tests are not together with the remaining sidebar tests,
-// because we need to publish/save a post, to correctly test the permalink panel.
+// because we need to publish/save a post, to correctly test the permalink row.
 // The sidebar test suit enforces that focus is never lost, but during save operations
 // the focus is lost and a new element is focused once the save is completed.
-describe( 'Sidebar Permalink Panel', () => {
+describe( 'Sidebar Permalink', () => {
 	beforeAll( async () => {
 		await activatePlugin( 'gutenberg-test-custom-post-types' );
 	} );
@@ -24,39 +23,30 @@ describe( 'Sidebar Permalink Panel', () => {
 		await deactivatePlugin( 'gutenberg-test-custom-post-types' );
 	} );
 
-	it( 'should allow permalink sidebar panel to be removed', async () => {
-		await createNewPost();
-		await page.evaluate( () => {
-			const { removeEditorPanel } = wp.data.dispatch( 'core/edit-post' );
-			removeEditorPanel( 'post-link' );
-		} );
-		expect( await page.$x( permalinkPanelXPath ) ).toEqual( [] );
-	} );
-
-	it( 'should not render link panel when post is publicly queryable but not public', async () => {
+	it( 'should not render URL when post is publicly queryable but not public', async () => {
 		await createNewPost( { postType: 'public_q_not_public' } );
 		await page.keyboard.type( 'aaaaa' );
 		await publishPost();
 		// Start editing again.
 		await page.type( '.editor-post-title__input', ' (Updated)' );
-		expect( await page.$x( permalinkPanelXPath ) ).toEqual( [] );
+		expect( await page.$( urlButtonSelector ) ).toBeNull();
 	} );
 
-	it( 'should not render link panel when post is public but not publicly queryable', async () => {
+	it( 'should not render URL when post is public but not publicly queryable', async () => {
 		await createNewPost( { postType: 'not_public_q_public' } );
 		await page.keyboard.type( 'aaaaa' );
 		await publishPost();
 		// Start editing again.
 		await page.type( '.editor-post-title__input', ' (Updated)' );
-		expect( await page.$x( permalinkPanelXPath ) ).toEqual( [] );
+		expect( await page.$( urlButtonSelector ) ).toBeNull();
 	} );
 
-	it( 'should render link panel when post is public and publicly queryable', async () => {
+	it( 'should render URL when post is public and publicly queryable', async () => {
 		await createNewPost( { postType: 'public_q_public' } );
 		await page.keyboard.type( 'aaaaa' );
 		await publishPost();
 		// Start editing again.
 		await page.type( '.editor-post-title__input', ' (Updated)' );
-		expect( await findSidebarPanelWithTitle( 'Permalink' ) ).toBeDefined();
+		expect( await page.$( urlButtonSelector ) ).not.toBeNull();
 	} );
 } );

package/specs/editor/various/typewriter.test.js

@@ -31,7 +31,7 @@ describe( 'TypeWriter', () => {
 
 		expect( await getCaretPosition() ).toBeGreaterThan( initialPosition );
 
-		// Create blocks until the the typewriter effect kicks in.
+		// Create blocks until the typewriter effect kicks in.
 		while (
 			await page.evaluate(
 				() =>
@@ -173,7 +173,7 @@ describe( 'TypeWriter', () => {
 
 		let count = 0;
 
-		// Create blocks until the the typewriter effect kicks in, create at
+		// Create blocks until the typewriter effect kicks in, create at
 		// least 10 blocks to properly test the .
 		while (
 			( await page.evaluate(

package/specs/editor/blocks/__snapshots__/list.test.js.snap

@@ -1,337 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`List can be converted to a quote 1`] = `
-"<!-- wp:quote -->
-<blockquote class=\\"wp-block-quote\\"><p>one</p><p>two</p></blockquote>
-<!-- /wp:quote -->"
-`;
-
-exports[`List can be converted to paragraphs 1`] = `
-"<!-- wp:paragraph -->
-<p>one</p>
-<!-- /wp:paragraph -->
-
-<!-- wp:paragraph -->
-<p>two</p>
-<!-- /wp:paragraph -->"
-`;
-
-exports[`List can be converted when nested to paragraphs 1`] = `
-"<!-- wp:paragraph -->
-<p>one</p>
-<!-- /wp:paragraph -->
-
-<!-- wp:paragraph -->
-<p>two</p>
-<!-- /wp:paragraph -->"
-`;
-
-exports[`List can be created by converting a paragraph 1`] = `
-"<!-- wp:list -->
-<ul><li>test</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List can be created by converting a paragraph with line breaks 1`] = `
-"<!-- wp:list -->
-<ul><li>one</li><li>two</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List can be created by converting a quote 1`] = `
-"<!-- wp:list -->
-<ul><li>one</li><li>two</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List can be created by converting multiple paragraphs 1`] = `
-"<!-- wp:list -->
-<ul><li>one</li><li>two</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List can be created by typing "/list" 1`] = `
-"<!-- wp:list -->
-<ul><li>I’m a list</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List can be created by typing an asterisk in front of text of a paragraph block 1`] = `
-"<!-- wp:list -->
-<ul><li>test</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List can be created by using a number at the start of a paragraph block 1`] = `
-"<!-- wp:list {\\"ordered\\":true} -->
-<ol><li>A list item</li></ol>
-<!-- /wp:list -->"
-`;
-
-exports[`List can be created by using an asterisk at the start of a paragraph block 1`] = `
-"<!-- wp:list -->
-<ul><li>A list item</li><li>Another list item</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List can undo asterisk transform 1`] = `
-"<!-- wp:paragraph -->
-<p>1. </p>
-<!-- /wp:paragraph -->"
-`;
-
-exports[`List first empty list item is graciously removed 1`] = `
-"<!-- wp:list -->
-<ul><li>2</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should be immeadiately saved on indentation 1`] = `
-"<!-- wp:list -->
-<ul><li>one<ul><li></li></ul></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should change the base list type 1`] = `
-"<!-- wp:list {\\"ordered\\":true} -->
-<ol><li></li></ol>
-<!-- /wp:list -->"
-`;
-
-exports[`List should change the indented list type 1`] = `
-"<!-- wp:list -->
-<ul><li>a<ol><li>1</li></ol></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should create and remove indented list with keyboard only 1`] = `
-"<!-- wp:list -->
-<ul><li>1<ul><li>a<ul><li>i</li></ul></li></ul></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should create and remove indented list with keyboard only 2`] = `
-"<!-- wp:list -->
-<ul><li>1<ul><li>a</li><li></li></ul></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should create and remove indented list with keyboard only 3`] = `
-"<!-- wp:list -->
-<ul><li>1<ul><li>a</li></ul></li><li></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should create and remove indented list with keyboard only 4`] = `
-"<!-- wp:list -->
-<ul><li>1<ul><li>a</li></ul></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should create and remove indented list with keyboard only 5`] = `
-"<!-- wp:list -->
-<ul><li>1</li><li></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should create and remove indented list with keyboard only 6`] = `
-"<!-- wp:list -->
-<ul><li>1</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should create and remove indented list with keyboard only 7`] = `""`;
-
-exports[`List should create paragraph on split at end and merge back with content 1`] = `
-"<!-- wp:list -->
-<ul><li>one</li></ul>
-<!-- /wp:list -->
-
-<!-- wp:paragraph -->
-<p></p>
-<!-- /wp:paragraph -->"
-`;
-
-exports[`List should create paragraph on split at end and merge back with content 2`] = `
-"<!-- wp:list -->
-<ul><li>one</li><li>two</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should indent and outdent level 1 1`] = `
-"<!-- wp:list -->
-<ul><li>a<ul><li>1</li></ul></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should indent and outdent level 1 2`] = `
-"<!-- wp:list -->
-<ul><li>a</li><li>1</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should indent and outdent level 2 1`] = `
-"<!-- wp:list -->
-<ul><li>a<ul><li>1<ul><li>i</li></ul></li></ul></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should indent and outdent level 2 2`] = `
-"<!-- wp:list -->
-<ul><li>a<ul><li>1</li><li>i</li></ul></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should indent and outdent level 2 3`] = `
-"<!-- wp:list -->
-<ul><li>a<ul><li>1</li></ul></li><li>i</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should insert a line break on shift+enter 1`] = `
-"<!-- wp:list -->
-<ul><li>a<br></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should insert a line break on shift+enter in a non trailing list item 1`] = `
-"<!-- wp:list -->
-<ul><li>a</li><li>b<br></li><li>c</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should not change the contents when you change the list type to Unordered 1`] = `"<li>a</li><li>b</li><li>c</li>"`;
-
-exports[`List should not change the contents when you change the list type to Ordered 1`] = `"<li>1</li><li>2</li><li>3</li>"`;
-
-exports[`List should not indent list on space with modifier 1`] = `
-"<!-- wp:list -->
-<ul><li>1</li><li> </li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should not transform lines in block when transforming multiple blocks 1`] = `
-"<!-- wp:list -->
-<ul><li>one<br>...</li><li>two</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should not undo asterisk transform with backspace after selection change 1`] = `""`;
-
-exports[`List should not undo asterisk transform with backspace after typing 1`] = `""`;
-
-exports[`List should only convert to list when shortcut ends with space 1`] = `
-"<!-- wp:paragraph -->
-<p>* </p>
-<!-- /wp:paragraph -->"
-`;
-
-exports[`List should outdent with children 1`] = `
-"<!-- wp:list -->
-<ul><li>a<ul><li>b<ul><li>c</li></ul></li></ul></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should outdent with children 2`] = `
-"<!-- wp:list -->
-<ul><li>a</li><li>b<ul><li>c</li></ul></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should place the caret in the right place with nested list 1`] = `
-"<!-- wp:list -->
-<ul><li>1</li><li>2<ul><li>a</li></ul></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should preserve indentation after merging backward and forward 1`] = `
-"<!-- wp:list -->
-<ul><li>1<ul><li>2</li><li>3</li></ul></li><li></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should preserve indentation after merging backward and forward 2`] = `
-"<!-- wp:list -->
-<ul><li>1<ul><li>2</li><li>3</li></ul></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should split indented list item 1`] = `
-"<!-- wp:list -->
-<ul><li>one<ul><li>two</li><li>three</li></ul></li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should split into two ordered lists with paragraph 1`] = `
-"<!-- wp:list {\\"ordered\\":true} -->
-<ol><li>one</li></ol>
-<!-- /wp:list -->
-
-<!-- wp:paragraph -->
-<p></p>
-<!-- /wp:paragraph -->
-
-<!-- wp:list {\\"ordered\\":true} -->
-<ol><li>two</li></ol>
-<!-- /wp:list -->"
-`;
-
-exports[`List should split into two with paragraph and merge lists 1`] = `
-"<!-- wp:list -->
-<ul><li>one</li><li></li><li>two</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should split into two with paragraph and merge lists 2`] = `
-"<!-- wp:list -->
-<ul><li>one</li></ul>
-<!-- /wp:list -->
-
-<!-- wp:paragraph -->
-<p></p>
-<!-- /wp:paragraph -->
-
-<!-- wp:list -->
-<ul><li>two</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should split into two with paragraph and merge lists 3`] = `
-"<!-- wp:list -->
-<ul><li>one</li><li></li></ul>
-<!-- /wp:list -->
-
-<!-- wp:list -->
-<ul><li>two</li></ul>
-<!-- /wp:list -->"
-`;
-
-exports[`List should undo asterisk transform with backspace 1`] = `
-"<!-- wp:paragraph -->
-<p>* </p>
-<!-- /wp:paragraph -->"
-`;
-
-exports[`List should undo asterisk transform with backspace after selection changes 1`] = `
-"<!-- wp:paragraph -->
-<p>* </p>
-<!-- /wp:paragraph -->"
-`;
-
-exports[`List should undo asterisk transform with backspace after selection changes without requestIdleCallback 1`] = `
-"<!-- wp:paragraph -->
-<p>* </p>
-<!-- /wp:paragraph -->"
-`;
-
-exports[`List should undo asterisk transform with backspace setting isTyping state 1`] = `
-"<!-- wp:paragraph -->
-<p>* </p>
-<!-- /wp:paragraph -->"
-`;
-
-exports[`List should undo asterisk transform with escape 1`] = `
-"<!-- wp:paragraph -->
-<p>* </p>
-<!-- /wp:paragraph -->"
-`;