Adjust multisite network cron Behat expectation for new WordPress privacy cleanup event#146
Merged
Merged
Conversation
…cenario Co-authored-by: swissspidy <841956+swissspidy@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix tests for WordPress 7.1
Adjust multisite network cron Behat expectation for new WordPress privacy cleanup event
Jul 9, 2026
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Behat acceptance test for multisite wp cron event run --due-now --network to stay aligned with WordPress core’s default scheduled events by excluding the newly-added privacy cleanup hook, preventing the scenario from counting an extra executed event.
Changes:
- Extend the multisite
--network --due-nowscenario’s--exclude=list to includewp_privacy_personal_data_cleanup_requests.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
WordPress 7.1 adds a scheduled
wp_privacy_personal_data_cleanup_requestscron event, which increases events executed in the multisite--network --due-nowBehat scenario. This caused the scenario to fail because the test’s exclusion list no longer matched current core behavior.Problem alignment
features/cron-event.featureto exclude the new core privacy cleanup hook alongside existing excluded core hooks.Targeted test fixture change
--exclude=list in the affected scenario (Run cron events with --network flag on multisite).Patch excerpt
When I run `wp cron event run --due-now --network --exclude=wp_privacy_delete_old_export_files,wp_privacy_personal_data_cleanup_requests,wp_version_check,wp_update_plugins,wp_update_themes,wp_site_health_scheduled_check,wp_update_user_counts,wp_scheduled_delete,wp_scheduled_auto_draft_delete`