Skip to content

[Tests] Fix legacy WP plugin activation selector - #3825

Merged
bgrgicak merged 3 commits into
trunkfrom
codex/legacy-wp-hello-dolly-activation
Jul 1, 2026
Merged

[Tests] Fix legacy WP plugin activation selector#3825
bgrgicak merged 3 commits into
trunkfrom
codex/legacy-wp-hello-dolly-activation

Conversation

@bgrgicak

@bgrgicak bgrgicak commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes the legacy WordPress boot test plugin activation phase so it waits for and clicks the Hello Dolly activation link specifically.

Motivation for the change, related issues

The test-legacy-wp-version-boot CI job can fail when the plugins page renders Akismet's activation link before Hello Dolly. The old fallback clicked the first Activate link, which can land on Akismet's setup screen and then time out while waiting for the Hello Dolly-style activation indicators.

Related failing job: https://github.com/WordPress/wordpress-playground/actions/runs/28081251039/job/83136494931?pr=3824

Implementation details

  • Removes the generic first-Activate fallback.
  • Waits for a[href*="action=activate"][href*="hello.php"] before clicking.
  • Keeps the existing SKIP / no activate link found status when no activation link is available.

Testing Instructions

  • CI

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Fixes flakiness in the legacy WordPress boot test by ensuring the plugin activation step targets the Hello Dolly activation link (instead of clicking the first “Activate” link, which can be Akismet).

Changes:

  • Replaces the “first Activate link” fallback with a Hello Dolly–specific activation selector.
  • Waits for the Hello Dolly activation link to render before attempting activation.
  • Preserves the existing skip behavior when no suitable activation link is found.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/playground/wordpress/tests/test-legacy-wp-version-boot.mjs Outdated
@bgrgicak
bgrgicak marked this pull request as draft June 24, 2026 09:20
@bgrgicak
bgrgicak marked this pull request as ready for review June 24, 2026 12:19
@bgrgicak bgrgicak self-assigned this Jun 24, 2026

@ashfame ashfame left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing! <3

@bgrgicak
bgrgicak merged commit 566f97f into trunk Jul 1, 2026
53 checks passed
@bgrgicak
bgrgicak deleted the codex/legacy-wp-hello-dolly-activation branch July 1, 2026 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment