Skip to content

Add a step to set several options at once - #398

Open
schlessera wants to merge 2 commits into
mainfrom
set-options-step
Open

schlessera wants to merge 2 commits into
mainfrom
set-options-step

Conversation

@schlessera

@schlessera schlessera commented Oct 30, 2026 •

Copy link
Copy Markdown
Member

Scenarios often run wp option update or wp site option update only to configure the site before the test, and each of these calls loads WordPress in a new process. Given these options: and Given these site options: set a list of options in a single process instead.

Fixes #380.

Measurements

Each draft pull request below points wp-cli/wp-cli-tests at this branch. Its Testing workflow ran through workflow_dispatch, together with the same workflow on an unchanged copy of the default branch, three times each where possible. Times are the Run Behat step summed over the matrix jobs that ran in every measured run (median over runs, in job minutes). Retried jobs passed only because behat-rerun repeated their failed scenarios.

Repository Jobs Runs Before (min) After (min) Change Median per job Failed jobs Retried jobs
wp-cli/cache-command#136 50 3/3 84.6 86.0 +1.7% +0.4% 0 → 0 0 → 0
wp-cli/entity-command#678 51 3/3 960.1 947.1 -1.4% -0.3% 3 → 3 7 → 7
wp-cli/import-command#113 51 3/3 37.1 36.5 -1.4% +3.7% 0 → 0 0 → 0
wp-cli/server-command#107 51 3/3 19.9 20.1 +1.0% -1.8% 0 → 0 0 → 0
Total 1101.7 1089.7 -1.1%

Failing scenarios

Scenarios that failed at least once in a measured run, including runs where behat-rerun then passed them.

  • wp-cli/entity-command: failing with and without the change: features/post.feature (Filtering by meta, time of day and order) (2/3 runs)

@schlessera schlessera added the scope:testing Related to testing label Oct 30, 2026
@schlessera
schlessera requested a review from a team as a code owner October 30, 2026 18:47
@coderabbitai

coderabbitai Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 51 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

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: 2ce738d9-fbdd-4689-bf61-514dd64c2fd9
Commits

Reviewing files that changed from the base of the PR and between b533a56 and f223215.

Files selected for processing: 2
  • features/behat-steps.feature
  • src/Context/GivenStepDefinitions.php
  • 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

@codecov

codecov Bot commented Oct 30, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 19 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/Context/GivenStepDefinitions.php 0.00% 19 Missing ⚠️

📢 Thoughts on this report? Let us know!

schlessera and others added 2 commits October 31, 2026 04:11
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope:testing Related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Set several options in one step

1 participant