Skip to content

Fix AuditLogCest for WP7 iframed block editor#460

Closed
jerico wants to merge 1 commit into
masterfrom
fix/wp7-audit-log-cest-iframed-editor
Closed

Fix AuditLogCest for WP7 iframed block editor#460
jerico wants to merge 1 commit into
masterfrom
fix/wp7-audit-log-cest-iframed-editor

Conversation

@jerico

@jerico jerico commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • WP 7.0 renders the block editor inside iframe[name="editor-canvas"]. The title field no longer exists in the main document, so the previous .editor-post-title__input selector did nothing silently, leaving the post untitled and causing the audit log assertion to fail.
  • Fix: switch into the editor iframe before typing the title, then switch back to the main frame for the publish button.

Test plan

  • Run composer dev-tools codecept run -p packages/security/tests -- acceptance AuditLogCest against a WP 7.0 environment
  • Confirm OK (2 tests, 4 assertions)

Discovered during WP 7.0 QA (humanmade/product-dev#2127).

WP 7.0 renders the block editor inside an iframe (name="editor-canvas").
The title field no longer exists in the main document, so the previous
`.editor-post-title__input` selector silently did nothing, leaving the
post title empty and causing the audit log assertion to fail.

Switch into the editor iframe before typing the title, then switch back
to the main frame for the publish button interaction.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mikelittle

Copy link
Copy Markdown
Contributor

@jerico This is a duplicate of #452, which is already approved and merged. You should have picked up this change in your checked out testing repo?!?

@jerico

jerico commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

Closing — duplicate of #452 which was already merged. Thanks for flagging.

@jerico jerico closed this Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants