fix: correct Playwright server launch command in Docker tests

[munezaclovis]

Fixes the connection failures in both pest-on-host and pest-in-container Docker test workflows.

Changes

• Add --mode launchServer flag required by Pest's WebSocket client
• Reorder parameters to match expected format (--host then --port)
• Remove unnecessary -y flag from npx (playwright is already installed)

This ensures the Playwright server launches in the correct mode for Pest's WebSocket client to connect.

Fixes #1

Generated with Claude Code

[claude]

No issues found. Checked for bugs and CLAUDE.md compliance.

[munezaclovis]

@claude I checked the logs for the worflow that failed. and it seems one of the issues is file permissions in Docker.
Please edit the Dockerfile to allow full access to the directory that we're mounting the code in.

"Warning: mkdir(): Permission denied in /app/vendor/pestphp/pest-plugin-mutate/src/Cache/FileStore.php on line 23"

Also, Please make sure the tests are run in parallel for maximum speed

[claude]

Claude finished @munezaclovis's task in 2m 26s —— View job

---

Tasks Completed

Changes Made

1. Fixed Docker Permissions ✓

File: docker.test/Dockerfile.pest

Added proper permissions for the /app directory to fix the "Permission denied" error:

# Set proper permissions for the /app directory
RUN mkdir -p /app && chmod -R 777 /app

This ensures that when the code is mounted into the container running as user 1000:1000, it can create cache files and directories without permission errors. The error you saw (Warning: mkdir(): Permission denied in /app/vendor/pestphp/pest-plugin-mutate/src/Cache/FileStore.php on line 23) should now be resolved.

2. Parallel Test Execution

Status: Partially complete - manual action required

I cannot modify workflow files due to GitHub App permissions restrictions. However, here's what needs to be added for maximum test speed:

In .github/workflows/tests.yml, add the --parallel flag to:

Line 90 (pest-on-host job):

- name: Tests
  run: ./vendor/bin/pest --parallel --http-host=host.docker.internal --http-bind=0.0.0.0 --playwright-host=127.0.0.1 --playwright-port=3000

Line 140 (pest-in-container job):

- name: Run tests in Pest container
  run: |
    docker compose -f docker.test/docker-compose.yml run --rm pest -c "./vendor/bin/pest --parallel --http-bind=0.0.0.0 --http-host=pest --playwright-port=3000 --playwright-host=playwright"

Note: The regular composer test command already uses --parallel (configured in composer.json line 73), so the main CI job already runs tests in parallel.

Summary

• ✅ Fixed Docker permission issues
• ⚠️ Parallel tests: Already enabled for main CI job; Docker jobs need manual workflow update (see above)
• ✅ Changes committed and pushed

--- |