Add test also when saving the default WooCommerce template

woocommerce · Feb 6, 2024 · ef5d508 · ef5d508
1 parent e764212
commit ef5d508
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/...s/woocommerce-blocks/tests/e2e/tests/templates/template-customization.block_theme.spec.ts b/...s/woocommerce-blocks/tests/e2e/tests/templates/template-customization.block_theme.spec.ts
@@ -11,6 +11,8 @@ import { CUSTOMIZABLE_WC_TEMPLATES, WC_TEMPLATES_SLUG } from './constants';
 CUSTOMIZABLE_WC_TEMPLATES.forEach( ( testData ) => {
 	const userText = `Hello World in the ${ testData.templateName } template`;
 	const fallbackTemplateUserText = `Hello World in the fallback ${ testData.templateName } template`;
+	const templateTypeName =
+		testData.templateType === 'wp_template' ? 'template' : 'template part';
 
 	test.describe( `${ testData.templateName } template`, async () => {
 		test( 'can be modified and reverted', async ( {
@@ -31,6 +33,14 @@ CUSTOMIZABLE_WC_TEMPLATES.forEach( ( testData ) => {
 				attributes: { content: userText },
 			} );
 			await editorUtils.saveTemplate();
+			// Verify template name didn't change.
+			// See: https://github.com/woocommerce/woocommerce/issues/42221
+			await expect(
+				page.getByRole( 'heading', {
+					name: `Editing ${ templateTypeName }: ${ testData.templateName }`,
+				} )
+			).toBeVisible();
+
 			await testData.visitPage( { frontendUtils, page } );
 			await expect( page.getByText( userText ).first() ).toBeVisible();
 

diff --git a/...locks/tests/e2e/tests/templates/template-customization.block_theme_with_templates.spec.ts b/...locks/tests/e2e/tests/templates/template-customization.block_theme_with_templates.spec.ts
@@ -15,7 +15,8 @@ CUSTOMIZABLE_WC_TEMPLATES.forEach( ( testData ) => {
 	}
 	const userText = `Hello World in the ${ testData.templateName } template`;
 	const fallbackTemplateUserText = `Hello World in the fallback ${ testData.templateName } template`;
-	const templateTypeName = testData.templateType === 'wp_template' ? 'template' : 'template part';
+	const templateTypeName =
+		testData.templateType === 'wp_template' ? 'template' : 'template part';
 
 	test.describe( `${ testData.templateName } template`, async () => {
 		test( "theme template has priority over WooCommerce's and can be modified", async ( {
@@ -40,7 +41,7 @@ CUSTOMIZABLE_WC_TEMPLATES.forEach( ( testData ) => {
 			// See: https://github.com/woocommerce/woocommerce/issues/42221
 			await expect(
 				page.getByRole( 'heading', {
-					name: `Editing ${templateTypeName}: ${ testData.templateName }`,
+					name: `Editing ${ templateTypeName }: ${ testData.templateName }`,
 				} )
 			).toBeVisible();