Add E2E test credentials and enhance WooPayments setup scripts

Author: mgascam

includes/wp-cli/class-wp-cli-qit-dev-command.php

@@ -1,6 +1,6 @@
 <?php
 /**
- * QIT Dev Tools command - Foundation (PR 1)
+ * QIT Dev Tools command
  *
  * @package WooCommerce\Payments
  */

tests/qit/config/default.env

@@ -7,3 +7,11 @@
 QIT_CI_USER=your_qit_username
 QIT_CI_SECRET=your_qit_application_password
 
+# ===========================================
+# E2E TEST CREDENTIALS (optional - for basic connectivity testing)
+# ===========================================
+# These provide basic WooPayments plugin connectivity for E2E tests
+E2E_JP_SITE_ID=your_site_id_here
+E2E_JP_BLOG_TOKEN=your_blog_token_here
+E2E_JP_USER_TOKEN=your_user_token_here
+

tests/qit/e2e-runner.sh

@@ -22,18 +22,61 @@ echo "Running E2E tests..."
 # Change to project root directory to build plugin
 cd "$WCP_ROOT"
 
-# Foundation version: Simplified build process for easier testing
-# For this foundation PR, we'll always build to avoid complex signature computation issues
+# Foundation version: Simplified build process - just ensure we have a zip
+# Compute a signature of sources relevant to the release build and
+# skip rebuilding if nothing has changed since the last build.
+compute_build_signature() {
+    # Hash tracked files that affect the release artifact. This includes
+    # sources packaged in the zip and build/config files that affect the output.
+    git ls-files -z -- \
+        assets \
+        i18n \
+        includes \
+        languages \
+        lib \
+        src \
+        templates \
+        client \
+        tasks/release.js \
+        webpack \
+        webpack.config.js \
+        babel.config.js \
+        package.json \
+        package-lock.json \
+        composer.json \
+        composer.lock \
+        woocommerce-payments.php \
+        changelog.txt \
+        readme.txt \
+        SECURITY.md \
+        2>/dev/null \
+    | xargs -0 shasum -a 256 2>/dev/null \
+    | shasum -a 256 \
+    | awk '{print $1}'
+}
 
 BUILD_HASH_FILE="$WCP_ROOT/woocommerce-payments.zip.hash"
 
-# For this foundation PR, always build if zip doesn't exist or if forced
-if [[ -n "${WCP_FORCE_BUILD:-}" ]] || [[ ! -f "woocommerce-payments.zip" ]]; then
-    echo "Building WooPayments plugin..."
+CURRENT_SIG="$(compute_build_signature)"
+
+# If WCP_FORCE_BUILD is set, always rebuild
+if [[ -n "${WCP_FORCE_BUILD:-}" ]]; then
+    echo "WCP_FORCE_BUILD set; forcing build of WooPayments plugin..."
     npm run build:release
-    echo "foundation-build-$(date +%s)" > "$BUILD_HASH_FILE"
+    echo "$CURRENT_SIG" > "$BUILD_HASH_FILE"
+elif [[ -f "woocommerce-payments.zip" && -f "$BUILD_HASH_FILE" ]]; then
+    LAST_SIG="$(cat "$BUILD_HASH_FILE" 2>/dev/null || true)"
+    if [[ "$CURRENT_SIG" == "$LAST_SIG" && -n "$CURRENT_SIG" ]]; then
+        echo "No relevant changes detected since last build; skipping build."
+    else
+        echo "Changes detected; rebuilding WooPayments plugin..."
+        npm run build:release
+        echo "$CURRENT_SIG" > "$BUILD_HASH_FILE"
+    fi
 else
-    echo "Using existing woocommerce-payments.zip"
+    echo "Building WooPayments plugin..."
+    npm run build:release
+    echo "$CURRENT_SIG" > "$BUILD_HASH_FILE"
 fi
 
 # Change to QIT directory so qit.yml is automatically found

tests/qit/e2e/basic.spec.js

@@ -15,15 +15,13 @@ test( 'Load home page', async ( { page } ) => {
 } );
 
 /**
- * Test WooCommerce Payments onboarding flow access
- * Since we're running in development mode without Jetpack connection,
- * we expect to always land on the onboarding flow.
+ * Test admin authentication and WooPayments plugin access
  */
-test( 'Access WooCommerce Payments onboarding as admin', async ( { page } ) => {
+test( 'Access WooPayments as admin', async ( { page } ) => {
 	// Use QIT helper to login as admin
 	await qit.loginAsAdmin( page );
 
-	// Navigate to WooCommerce Payments settings
+	// Navigate to WooPayments settings
 	await page.goto(
 		'/wp-admin/admin.php?page=wc-admin&path=%2Fpayments%2Foverview'
 	);
@@ -33,13 +31,21 @@ test( 'Access WooCommerce Payments onboarding as admin', async ( { page } ) => {
 		page.locator( 'h1:not(.screen-reader-text)' ).first()
 	).toContainText( /Settings|Payments|Overview/, { timeout: 15000 } );
 
-	// In development mode without Jetpack connection, we should be on onboarding
-	expect( page.url() ).toContain( 'onboarding' );
+	// Check that we can successfully load the WooPayments interface
+	// Either we get the overview (if fully connected) OR the onboarding.
+	const isOnboarding = page.url().includes( 'onboarding' );
+	const isOverview = page.url().includes( 'payments' );
 
-	// The onboarding page should load without errors
-	await expect( page.locator( 'body' ) ).not.toHaveText(
-		/500|404|Fatal error/
-	);
+	// We should be on either the onboarding or overview page (both indicate success)
+	expect( isOnboarding || isOverview ).toBe( true );
+
+	// If we're on onboarding, it should be functional (not errored)
+	if ( isOnboarding ) {
+		// The onboarding page should load without errors
+		await expect( page.locator( 'body' ) ).not.toHaveText(
+			/500|404|Fatal error/
+		);
+	}
 } );
 
 /**

tests/qit/e2e/bootstrap/setup.sh

@@ -59,7 +59,7 @@ if [ -n "${E2E_JP_SITE_ID:-}" ] && [ -n "${E2E_JP_BLOG_TOKEN:-}" ] && [ -n "${E2
     echo "✅ WooPayments connection configured - account data fetched from server"
 
 else
-    echo "No Jetpack credentials configured - using default development mode"
+    echo "No Jetpack credentials configured - WooPayments will show Connect screen"
     echo "WooPayments will show Connect screen"
     echo ""
     echo "For basic connectivity testing, set in tests/qit/config/local.env:"