Skip to content

Tests: Fix Heartbeat Warning in Siteground Speed Optimizer#1006

Merged
n7studios merged 37 commits intomainfrom
fix-tests-siteground-optimiser
Feb 7, 2026
Merged

Tests: Fix Heartbeat Warning in Siteground Speed Optimizer#1006
n7studios merged 37 commits intomainfrom
fix-tests-siteground-optimiser

Conversation

@n7studios
Copy link
Contributor

@n7studios n7studios commented Feb 4, 2026

Summary

Ignore the PHP notice when running tests relating to the Siteground Speed Optimizer WordPress Plugin in WordPress 6.9.1, which they seem to think is okay, and configure the Siteground Speed Optimizer to not deregister the heartbeat script.

Tests EndToEnd PageBlockFormCest testFormBlockWithSitegroundSpeedOptimizerPlugin fail

Testing

Existing tests pass.

Checklist

@n7studios n7studios self-assigned this Feb 4, 2026
@Kit Kit deleted a comment from github-actions bot Feb 5, 2026
@Kit Kit deleted a comment from github-actions bot Feb 5, 2026
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@n7studios n7studios requested review from a team, ciccio-kit and noelherrick and removed request for a team February 5, 2026 07:34
@n7studios n7studios marked this pull request as ready for review February 5, 2026 07:35
Copy link

@ciccio-kit ciccio-kit left a comment

Choose a reason for hiding this comment

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

LGTM ✅

@n7studios n7studios merged commit 472ea04 into main Feb 7, 2026
84 of 86 checks passed
@github-actions
Copy link

github-actions bot commented Feb 7, 2026

WordPress Playground

🚀 Your PR has been built and is ready for testing in WordPress Playground!

Click here to test your changes in WordPress Playground

@n7studios n7studios deleted the fix-tests-siteground-optimiser branch February 10, 2026 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants