Merge pull request #4444 from woocommerce/dev/qa/new-autotests-pcp-6459-pcp-6513

[QA] New autotests for variable product, negative fees

Author: mishautkin

.github/workflows/e2e-tests.yml

@@ -21,7 +21,6 @@ on:
           - shard:settings
           - shard:transaction-usa
           - shard:transaction-mexico
-          # - shard:transaction-googlepay - temporarily disabled due to instability in CI
           - shard:refund
           - shard:vaulting
           - shard:subscription
@@ -72,7 +71,6 @@ jobs:
             settings
             transaction-usa
             transaction-mexico
-            # transaction-googlepay - temporarily disabled due to instability in CI
             refund
             vaulting
             subscription

package.json

@@ -48,10 +48,6 @@
     "e2e:test:shard:transaction-mexico:smoke": "npx playwright test --project \"shard:transaction-mexico\" --grep \"@Smoke\"",
     "e2e:test:shard:transaction-mexico:critical": "npx playwright test --project \"shard:transaction-mexico\" --grep \"@Critical\"",
 
-    "e2e:test:shard:transaction-googlepay": "npx playwright test --project \"shard:transaction-googlepay\"",
-    "e2e:test:shard:transaction-googlepay:smoke": "npx playwright test --project \"shard:transaction-googlepay\" --grep \"@Smoke\"",
-    "e2e:test:shard:transaction-googlepay:critical": "npx playwright test --project \"shard:transaction-googlepay\" --grep \"@Critical\"",
-
     "e2e:test:shard:refund": "npx playwright test --project \"shard:refund\"",
     "e2e:test:shard:refund:smoke": "npx playwright test --project \"shard:refund\" --grep \"@Smoke\"",
     "e2e:test:shard:refund:critical": "npx playwright test --project \"shard:refund\" --grep \"@Critical\"",

playwright.config.ts

@@ -167,20 +167,6 @@ export default defineConfig< BaseExtend >( {
 			grep: /setup:transaction:mexico;/,
 			fullyParallel: false,
 		},
-		{
-			name: 'setup-transaction-googlepay',
-			dependencies: [ 'setup-pcp-usa' ],
-			testMatch: /03-pcp\.setup\.ts/,
-			grep: /setup:transaction:googlepay;/,
-			fullyParallel: false,
-		},
-		{
-			name: 'setup-refund',
-			dependencies: [ 'setup-pcp-usa' ],
-			testMatch: /03-pcp\.setup\.ts/,
-			grep: /setup:refund;/,
-			fullyParallel: false,
-		},
 		{
 			name: 'setup-vaulting',
 			dependencies: [ 'setup-pcp-usa' ],
@@ -226,19 +212,14 @@ export default defineConfig< BaseExtend >( {
 			dependencies: [ 'setup-transaction-usa' ],
 			testMatch: /05-transactions\/transaction-usa.*\.spec\.ts/,
 		},
-		{
-			name: 'shard:transaction-googlepay',
-			dependencies: [ 'setup-transaction-googlepay' ],
-			testMatch: /05-transactions\/transaction-googlepay\.spec\.ts/,
-		},
 		{
 			name: 'shard:transaction-mexico',
 			dependencies: [ 'setup-transaction-mexico' ],
 			testMatch: /05-transactions\/transaction-mexico.*\.spec\.ts/,
 		},
 		{
 			name: 'shard:refund',
-			dependencies: [ 'setup-refund' ],
+			dependencies: [ 'setup-transaction-usa' ],
 			testMatch: /06-refund\/.*\.spec\.ts/,
 		},
 		{

tests/qa/resources/e2e-plugins.ts

@@ -35,3 +35,9 @@ export const subscriptionsPlugin = {
 	slug: 'woocommerce-subscriptions',
 	zipFilePath: `${ filePath }/woocommerce-subscriptions.zip`,
 };
+
+export const negative12FeePlugin = {
+	name: 'Negative 12 Fee',
+	slug: 'negative-12-fee',
+	zipFilePath: `${ filePath }/negative-12-fee.zip`,
+};

tests/qa/resources/files/negative-12-fee.zip

@@ -11,6 +11,15 @@ const country = process.env.WC_DEFAULT_COUNTRY || 'usa';
 const currency = process.env.WC_DEFAULT_CURRENCY || 'USD';
 const merchant = merchants[ country ];
 
+orders.negative12Fee = {
+	...orders.default,
+	fees: [ {
+		name: 'negative12Fee',
+		total: '-12.00',
+		type: 'fixed',
+	} ],
+};
+
 for ( const order in orders ) {
 	orders[ order ].merchant = merchant;
 	orders[ order ].currency = currency;

tests/qa/resources/orders.ts

@@ -3,6 +3,78 @@
  */
 import { products as baseProducts } from '@inpsyde/playwright-utils/build';
 
+const variationNotSelected100: WooCommerce.CreateProduct = {
+	name: 'Variable T-Shirt',
+	slug: 'variable-t-shirt',
+	type: 'variable',
+	regular_price: '100.00',
+	description:
+		'A comfortable t-shirt available in multiple colors and sizes.',
+	short_description: 'Variable t-shirt with color and size options.',
+	attributes: [
+		{
+			name: 'Color',
+			variation: true,
+			options: [ 'Gray', 'Red' ],
+		},
+		{
+			name: 'Size',
+			variation: true,
+			options: [ 'M', 'L' ],
+		},
+	],
+	default_attributes: [],
+	variations: [
+		{
+			attributes: [
+				{ name: 'Color', option: 'Gray' },
+				{ name: 'Size', option: 'M' },
+			],
+			regular_price: '100.00',
+			stock_status: 'instock',
+			manage_stock: false,
+		},
+		{
+			attributes: [
+				{ name: 'Color', option: 'Gray' },
+				{ name: 'Size', option: 'L' },
+			],
+			regular_price: '100.00',
+			stock_status: 'instock',
+			manage_stock: false,
+		},
+		{
+			attributes: [
+				{ name: 'Color', option: 'Red' },
+				{ name: 'Size', option: 'M' },
+			],
+			regular_price: '100.00',
+			stock_status: 'instock',
+			manage_stock: false,
+		},
+		{
+			attributes: [
+				{ name: 'Color', option: 'Red' },
+				{ name: 'Size', option: 'L' },
+			],
+			regular_price: '100.00',
+			stock_status: 'instock',
+			manage_stock: false,
+		},
+	],
+	images: [
+		{
+			src: 'https://woocommercecore.mystagingwebsite.com/wp-content/uploads/2017/12/vneck-tee-2.jpg',
+		},
+		{
+			src: 'https://woocommercecore.mystagingwebsite.com/wp-content/uploads/2017/12/tshirt-2.jpg',
+		},
+		{
+			src: 'https://woocommercecore.mystagingwebsite.com/wp-content/uploads/2017/12/t-shirt-with-logo-1.jpg',
+		},
+	],
+};
+
 const subscriptionPayPal: WooCommerce.CreateProduct = {
 	name: 'PayPal Subscription Test Product',
 	slug: 'paypal-subscription-test-product',
@@ -73,6 +145,7 @@ export const products: {
 	[ key: string ]: WooCommerce.CreateProduct;
 } = {
 	...baseProducts,
+	variationNotSelected100,
 	subscriptionPayPal,
 	subscriptionPayPalFreeTrial,
 	simpleWithStock,

tests/qa/resources/pcp-products.ts

@@ -9,6 +9,7 @@ export type ShopConfig = {
 	};
 	customer?: WooCommerce.CreateCustomer; // Registered customer to be created
 	products?: WooCommerce.CreateProduct[]; // Products to be created if not existing
+	isPayNowEnabled?: boolean; // Is Pay Now Experience option enabled in PCP Settings > Payment Methods
 };
 
 export type PayPalAccount = {
@@ -144,6 +145,65 @@ export namespace Pcp {
 			softDescriptor?: string;
 			subtotalAdjustment?: string;
 		};
+
+		export namespace Styling {
+			// Top-level styling keys (camelCase) — addresses each location's config
+			export type LocationPageKey =
+				| 'cart'
+				| 'classicCheckout'
+				| 'expressCheckout'
+				| 'miniCart'
+				| 'product';
+
+			// Value echoed in the `location` field of a styling block (snake_case)
+			export type LocationPageSlug =
+				| 'cart'
+				| 'classic_checkout'
+				| 'express_checkout'
+				| 'mini_cart'
+				| 'product';
+
+			// API value of Admin.Styling.ButtonLayout
+			export type ButtonLayout = 'vertical' | 'horizontal';
+
+			// API value of Admin.Styling.ButtonShape
+			export type ButtonShape = 'rect' | 'pill';
+
+			// API value of Admin.Styling.ButtonLabel
+			// 'paypal' = PayPal, 'checkout' = Checkout, 'buynow' = PayPal Buy Now, 'pay' = Pay with PayPal
+			export type ButtonLabel = 'paypal' | 'checkout' | 'buynow' | 'pay';
+
+			// API value of Admin.Styling.ButtonColor
+			export type ButtonColor =
+				| 'gold'
+				| 'blue'
+				| 'silver'
+				| 'black'
+				| 'white';
+
+			// Per-location styling block (mirrors Admin.Styling.Config)
+			export type LocationPageConfig = {
+				enabled?: boolean;
+				methods?: Extract <
+					GatewayId,
+					| 'ppcp-gateway'
+					| 'pay-later'
+					| 'venmo'
+					| 'ppcp-googlepay'
+					| 'ppcp-applepay'
+				>[];
+				label?: ButtonLabel;
+				shape?: ButtonShape;
+				color?: ButtonColor;
+				location?: LocationPageSlug;
+				layout?: ButtonLayout;
+				tagline?: boolean;
+			};
+
+			export type Settings = Partial <
+				Record < LocationPageKey, LocationPageConfig >
+			>;
+		}
 	}
 
 	export namespace Admin {
@@ -184,7 +244,7 @@ export namespace Pcp {
 		};
 
 		export namespace Styling {
-			export type Location =
+			export type LocationPage =
 				| 'Cart'
 				| 'Classic Checkout'
 				| 'Express Checkout'
@@ -216,8 +276,8 @@ export namespace Pcp {
 				| 'Black'
 				| 'White';
 
-			export type Config = {
-				location?: Location;
+			export type LocationPageConfig = {
+				location?: LocationPage;
 				enablePaymentMethodsInLocation?: boolean;
 				paymentMethods?: PaymentMethods;
 				buttonLayout?: ButtonLayout;
@@ -294,8 +354,17 @@ export namespace Pcp {
 			merchant?: Merchant;
 			paymentMethods?: Gateway[];
 			settings?: Settings;
-			styling?: Styling.Config;
+			styling?: Styling.LocationPageConfig;
 			payLaterMessaging?: Plm.Config;
 		};
 	}
 }
+
+export type PayPalPaymentDetails = {
+	transactionId?: string;
+	currency?: string;
+	amount?: string;
+	grossAmount?: string;
+	payPalFee?: string;
+	netAmount?: string;
+};

tests/qa/resources/types.ts

@@ -14,7 +14,8 @@ test.beforeAll( async ( { utils, pcpApi } ) => {
 	);
 } );
 
-test( 'PCP-0000 | Settings - Pay Later Messaging - Default UI', async ( {
+// https://inpsyde.atlassian.net/browse/PCP-4336
+test( 'PCP-4336 | Settings - Pay Later Messaging - Default UI', async ( {
 	utils,
 	payPalUi,
 	pcpPayLaterMessaging,
@@ -107,7 +108,8 @@ test( 'PCP-0000 | Settings - Pay Later Messaging - Default UI', async ( {
 	).not.toBeVisible();
 } );
 
-test( 'PCP-0000 | Settings - Pay Later Messaging - Disabled on all pages', async ( {
+// https://inpsyde.atlassian.net/browse/PCP-4337
+test( 'PCP-4337 | Settings - Pay Later Messaging - Disabled on all pages', async ( {
 	utils,
 	payPalUi,
 	pcpPayLaterMessaging,

tests/qa/tests/03-plugin-settings/pay-later-messaging-ui.spec.ts

@@ -22,7 +22,8 @@ test.beforeAll( async ( { utils, pcpApi } ) => {
 	);
 } );
 
-test( 'PCP-0000 | Settings - Styling - Default UI', async ( {
+// https://inpsyde.atlassian.net/browse/PCP-4747
+test( 'PCP-4747 | Settings - US- Styling - Default UI', async ( {
 	utils,
 	pcpStyling,
 	product,