Restore activated currencies after all E2E test that deactivates the …

…existing ones (#10175)
Automattic · Jan 17, 2025 · 30643ff · 30643ff
1 parent 90442fc
commit 30643ff
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 1 deletion.
diff --git a/changelog/dev-playwright-fix-multi-currency-tests b/changelog/dev-playwright-fix-multi-currency-tests
@@ -0,0 +1,4 @@
+Significance: patch
+Type: dev
+
+Restore activated currencies after all tests that deactivate current ones.
diff --git a/tests/e2e-pw/specs/merchant/multi-currency-on-boarding.spec.ts b/tests/e2e-pw/specs/merchant/multi-currency-on-boarding.spec.ts
@@ -14,6 +14,7 @@ import {
 	disableAllEnabledCurrencies,
 	getActiveThemeSlug,
 	removeCurrency,
+	restoreCurrencies,
 } from '../../utils/merchant';
 import * as navigation from '../../utils/merchant-navigation';
 
@@ -38,7 +39,7 @@ test.describe( 'Multi-currency on-boarding', () => {
 	test.afterAll( async () => {
 		// Restore original theme.
 		await activateTheme( page, activeThemeSlug );
-
+		await restoreCurrencies( page );
 		if ( ! wasMulticurrencyEnabled ) {
 			await deactivateMulticurrency( page );
 		}

diff --git a/tests/e2e-pw/specs/merchant/multi-currency-setup.spec.ts b/tests/e2e-pw/specs/merchant/multi-currency-setup.spec.ts
@@ -12,6 +12,7 @@ import {
 	deactivateMulticurrency,
 	disableAllEnabledCurrencies,
 	removeCurrency,
+	restoreCurrencies,
 	setCurrencyCharmPricing,
 	setCurrencyPriceRounding,
 	setCurrencyRate,
@@ -31,6 +32,8 @@ test.describe( 'Multi-currency setup', () => {
 	} );
 
 	test.afterAll( async () => {
+		await restoreCurrencies( merchantPage );
+
 		if ( ! wasMulticurrencyEnabled ) {
 			await deactivateMulticurrency( merchantPage );
 		}

diff --git a/tests/e2e-pw/specs/merchant/multi-currency.spec.ts b/tests/e2e-pw/specs/merchant/multi-currency.spec.ts
@@ -11,6 +11,7 @@ import {
 	addMulticurrencyWidget,
 	deactivateMulticurrency,
 	disableAllEnabledCurrencies,
+	restoreCurrencies,
 } from '../../utils/merchant';
 import * as navigation from '../../utils/merchant-navigation';
 
@@ -29,6 +30,7 @@ test.describe( 'Multi-currency', () => {
 	} );
 
 	test.afterAll( async () => {
+		await restoreCurrencies( page );
 		if ( ! wasMulticurrencyEnabled ) {
 			await deactivateMulticurrency( page );
 		}

diff --git a/tests/e2e-pw/utils/merchant.ts b/tests/e2e-pw/utils/merchant.ts
@@ -203,6 +203,13 @@ export const addCurrency = async ( page: Page, currencyCode: string ) => {
 	).toBeVisible();
 };
 
+export const restoreCurrencies = async ( page: Page ) => {
+	await disableAllEnabledCurrencies( page );
+	await addCurrency( page, 'USD' );
+	await addCurrency( page, 'EUR' );
+	await addCurrency( page, 'GBP' );
+};
+
 export const removeCurrency = async ( page: Page, currencyCode: string ) => {
 	await navigation.goToMultiCurrencySettings( page );
 	await page