Skip to content

Use WP_HTML_Tag_Processor::set_modifiable_text in Optimization_Detective_Test_Helpers#2511

Merged
westonruter merged 5 commits into
WordPress:trunkfrom
Hug0-Drelon:fix-2354
Jun 4, 2026
Merged

Use WP_HTML_Tag_Processor::set_modifiable_text in Optimization_Detective_Test_Helpers#2511
westonruter merged 5 commits into
WordPress:trunkfrom
Hug0-Drelon:fix-2354

Conversation

@Hug0-Drelon

@Hug0-Drelon Hug0-Drelon commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Summary

See #2354

Relevant technical choices

Use WP_HTML_Tag_Processor::set_modifiable_text in Optimization_Detective_Test_Helpers as requested in the issue.
Note that I didn't used assertEqualHTML since WP 6.6 seems to be the lowest tested version in GitHub workflow.
Drop WP 6.6 from tests matrix in favor of WP 6.7.

Use of AI Tools

I helped myself with Cursor.

@Hug0-Drelon Hug0-Drelon requested a review from westonruter as a code owner June 4, 2026 09:55
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: Hug0-Drelon <hugod@git.wordpress.org>
Co-authored-by: westonruter <westonruter@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@codecov

codecov Bot commented Jun 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.29%. Comparing base (c9e07d5) to head (fc2e2ec).
⚠️ Report is 5 commits behind head on trunk.

Additional details and impacted files
@@           Coverage Diff           @@
##            trunk    #2511   +/-   ##
=======================================
  Coverage   69.29%   69.29%           
=======================================
  Files          90       90           
  Lines        7723     7723           
=======================================
  Hits         5352     5352           
  Misses       2371     2371           
Flag Coverage Δ
multisite 69.29% <ø> (ø)
single 35.76% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@westonruter westonruter 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.

Thank you!

@westonruter westonruter added [Type] Enhancement A suggestion for improvement of an existing feature [Plugin] Optimization Detective Issues for the Optimization Detective plugin no milestone PRs that do not have a defined milestone for release skip changelog PRs that should not be mentioned in changelogs labels Jun 4, 2026
@Hug0-Drelon

Copy link
Copy Markdown
Contributor Author

Thank you!

Sorry, I forced pushed a similar commit as your previous one...

@westonruter westonruter merged commit ac18b75 into WordPress:trunk Jun 4, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no milestone PRs that do not have a defined milestone for release [Plugin] Optimization Detective Issues for the Optimization Detective plugin skip changelog PRs that should not be mentioned in changelogs [Type] Enhancement A suggestion for improvement of an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants