Skip to content

[codex] Fix E2E workflow crash and PHPUnit alerts #17

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ediamin merged 4 commits into from
Apr 18, 2026
Merged

[codex] Fix E2E workflow crash and PHPUnit alerts #17

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
be7d126
Fix E2E server stability and PHPUnit alerts
ediamin Apr 18, 2026
2a77217
Restore localhost for E2E test host
ediamin Apr 18, 2026
52292ba
Run E2E workflow on wp-env
ediamin Apr 18, 2026
b77cd99
Stabilize wp-env CI and e2e tests
ediamin Apr 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
127 changes: 37 additions & 90 deletions .github/workflows/tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -185,39 +185,18 @@ jobs:
if-no-files-found: warn

# ---------------------------------------------------------------------------
# Playwright E2E tests (no wp-env / Docker)
# Playwright E2E tests via wp-env
# ---------------------------------------------------------------------------
e2e:
name: E2E Tests
runs-on: ubuntu-latest
runs-on: ubuntu-24.04
env:
WP_BASE_URL: http://127.0.0.1:8889

services:
mysql:
image: mysql:8.0
env:
MYSQL_ROOT_PASSWORD: root
MYSQL_DATABASE: wordpress
ports:
- 3306:3306
options: >-
--health-cmd="mysqladmin ping -h 127.0.0.1"
--health-interval=10s
--health-timeout=5s
--health-retries=5
WP_BASE_URL: http://localhost:8889

steps:
- name: Checkout repository
uses: actions/checkout@v4

- name: Set up PHP
uses: shivammathur/setup-php@v2
with:
php-version: '8.4'
coverage: none
tools: composer, wp-cli

- name: Get Composer cache directory
id: composer-cache
run: echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT
Expand Down Expand Up @@ -266,92 +245,56 @@ jobs:
key: playwright-${{ runner.os }}-${{ hashFiles('package-lock.json') }}

- name: Install Playwright browsers and system dependencies
if: steps.playwright-cache.outputs.cache-hit != 'true'
run: npx playwright install --with-deps chromium

- name: Build assets
run: npm run build

- name: Set up WordPress
- name: Patch wp-env Composer install for Composer 2.9
run: |
# Download WordPress core
wp core download --path=/tmp/wordpress

# Configure database connection
wp config create --path=/tmp/wordpress \
--dbname=wordpress \
--dbuser=root \
--dbpass=root \
--dbhost=127.0.0.1

# Install WordPress
wp core install --path=/tmp/wordpress \
--url="$WP_BASE_URL" \
--title="CSS Class Manager Tests" \
--admin_user=admin \
--admin_password=password \
--admin_email=admin@example.org \
--skip-email

# Enable debug mode
wp config set WP_DEBUG true --raw --path=/tmp/wordpress
wp config set WP_DEBUG_LOG true --raw --path=/tmp/wordpress

# Symlink the plugin into the WordPress plugins directory
ln -s "$GITHUB_WORKSPACE" /tmp/wordpress/wp-content/plugins/css-class-manager

# Activate the plugin
wp plugin activate css-class-manager --path=/tmp/wordpress

# Install and activate the theme used by E2E tests
wp theme install twentytwentyfive --activate --path=/tmp/wordpress

# Set up pretty permalinks for REST API support
wp rewrite structure '/%postname%/' --path=/tmp/wordpress
wp rewrite flush --path=/tmp/wordpress

- name: Create PHP router for built-in server
run: |
cat > /tmp/wordpress/router.php << 'ROUTER'
<?php
$uri = urldecode( parse_url( $_SERVER['REQUEST_URI'], PHP_URL_PATH ) );
node <<'NODE'
const fs = require('node:fs');
const target = 'node_modules/@wordpress/env/lib/init-config.js';
const source = fs.readFileSync(target, 'utf8');
const original =
'RUN composer global require --dev phpunit/phpunit:"^5.7.21 || ^6.0 || ^7.0 || ^8.0 || ^9.0 || ^10.0"';
const replacement =
'RUN composer global require --dev --no-security-blocking phpunit/phpunit:"^5.7.21 || ^6.0 || ^7.0 || ^8.0 || ^9.0 || ^10.0"';

if (source.includes(replacement)) {
console.log('wp-env Composer install already patched');
process.exit(0);
}

if ( $uri !== '/' && file_exists( $_SERVER['DOCUMENT_ROOT'] . $uri ) ) {
return false;
if (!source.includes(original)) {
throw new Error('Expected wp-env Composer install command was not found');
}

$_SERVER['PHP_SELF'] = '/index.php';
require $_SERVER['DOCUMENT_ROOT'] . '/index.php';
ROUTER
fs.writeFileSync(target, source.replace(original, replacement));
console.log('Patched wp-env Composer install for Composer 2.9');
NODE

- name: Start WordPress
run: |
PHP_CLI_SERVER_WORKERS=4 php -S 127.0.0.1:8889 -t /tmp/wordpress /tmp/wordpress/router.php > /tmp/wordpress/php-server.log 2>&1 &
echo $! > /tmp/wordpress/php-server.pid
- name: Remove local wp-env overrides
run: rm -f .wp-env.override.json

- name: Wait for WordPress
run: |
for i in $(seq 1 30); do
if curl -sS -o /dev/null "$WP_BASE_URL/wp-login.php" 2>/dev/null; then
echo "WordPress is ready"
exit 0
fi
sleep 1
done
echo "WordPress failed to start"
exit 1
- name: Start WordPress test environment
run: npm run env:start:e2e

- name: Run E2E tests
env:
PLAYWRIGHT_SKIP_BROWSER_DOWNLOAD: 'true'
run: npm run test:e2e

- name: Upload PHP server log
- name: Capture wp-env logs
if: failure()
run: npx wp-env logs tests --no-watch > /tmp/wp-env-tests.log 2>&1 || true

- name: Upload wp-env logs
uses: actions/upload-artifact@v4
if: failure()
with:
name: php-server-log
path: /tmp/wordpress/php-server.log
name: wp-env-tests-log
path: /tmp/wp-env-tests.log
if-no-files-found: warn

- name: Upload Playwright HTML report
Expand All @@ -369,3 +312,7 @@ jobs:
name: e2e-junit-results
path: e2e-results.xml
if-no-files-found: warn

- name: Stop WordPress test environment
if: always()
run: npm run env:stop
8 changes: 2 additions & 6 deletions composer.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
"automattic/vipwpcs": "^3.0",
"dealerdirect/phpcodesniffer-composer-installer": "^1.0",
"phpcompatibility/php-compatibility": "^9.3",
"phpunit/phpunit": "^9.6",
"phpunit/phpunit": "^9.6.33",
"slevomat/coding-standard": "^8.14",
"phpstan/phpstan": "^1.10",
"szepeviktor/phpstan-wordpress": "^1.3",
Expand All @@ -51,11 +51,7 @@
"ignore": {
"PKSA-5jz8-6tcw-pbk4": {
"apply": "block",
"reason": "PHP 7.4 CI support currently depends on PHPUnit 9.x, but we still want audits to report this advisory."
},
"PKSA-z3gr-8qht-p93v": {
"apply": "block",
"reason": "PHP 7.4 CI support currently depends on PHPUnit 9.x, but we still want audits to report this advisory."
"reason": "GitHub/Packagist currently report GHSA-qrr6-mg7r-m243 as affecting all phpunit/phpunit versions <=12.5.21, which incorrectly sweeps in the repo's PHPUnit 9.6.34 test dependency."
}
}
}
Expand Down
Loading
Loading