Skip to content

Run the scenarios of each package in parallel workers - #82

Merged
schlessera merged 1 commit into
mainfrom
parallel-workers
Nov 2, 2026
Merged

schlessera merged 1 commit into
mainfrom
parallel-workers

Conversation

@schlessera

Copy link
Copy Markdown
Member

Uses the parallel runner of wp-cli/wp-cli-tests (5.5) for the source and Phar suites:

  • bin/test-source and bin/test-phar now call vendor/bin/run-behat-tests --suite <package> instead of vendor/bin/behat, with BEHAT_FEATURES_FOLDER pointing at the package's features. The runner derives the environment tags from there, as before. Calling it also enables WP_CLI_TEST_PARALLEL, and a rerun after a parallel run covers exactly the scenarios that failed.
  • The workflow sets WP_CLI_TEST_PARALLEL to 4 workers, one per core of the runner. composer prepare-tests already lets the test user create the per-worker databases.
  • The workers don't run in Docker containers (WP_CLI_TEST_DOCKER). The containers don't receive WP_CLI_BIN_DIR or the temporary directory that holds the downloaded Phar, so the Phar legs would quietly test the source.
  • Fixes wp-cli-/wp-cli in test-source, which made TEST_PACKAGE=all skip the framework suite.

Tested locally on cron-command, from source and with the nightly Phar: 27 scenarios across 4 workers, all passing.

🤖 Generated with Claude Code

Run every suite through the run-behat-tests runner of wp-cli/wp-cli-tests
instead of calling Behat directly, so that WP_CLI_TEST_PARALLEL spreads
its scenarios over several workers and a rerun covers exactly the
scenarios that failed. Also fixes the name of the framework suite in
test-source, which made TEST_PACKAGE=all skip it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 2, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: d51c1723-eed1-4309-9996-d60be7d009fe
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@schlessera
schlessera merged commit 649a14c into main Nov 2, 2026
136 of 156 checks passed
@schlessera
schlessera deleted the parallel-workers branch November 2, 2026 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant