Skip to content

GH#2060: test: cover Gutenberg support screen gate (GH#2060) - #2061

Merged
superdav42 merged 1 commit into
mainfrom
feature/auto-20261028-014233-gh2060
Oct 28, 2026
Merged

superdav42 merged 1 commit into
mainfrom
feature/auto-20261028-014233-gh2060

Conversation

@superdav42

@superdav42 superdav42 commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Adds data-provider test asserting wu-gutenberg-support is enqueued on post, site and widget editors and not on the dashboard.

Files Changed

tests/WP_Ultimo/General_Compat_Test.php

Runtime Testing

  • Risk level: Medium
  • Verification: self-assessed — phpcs clean, php -l ok; PHPUnit WP test suite not installed on runner, covered by CI

Resolves #2060


aidevops.sh v3.39.49 plugin for OpenCode v1.19.0 with claude-sonnet-5-5 spent 57s and 2,102 tokens on this as a headless worker.

Summary by CodeRabbit

  • Tests
    • Added coverage for Gutenberg support script loading across dashboard, post, site editor, and widget editor screens. The tests verify that the script is enqueued for block editor screens and not for the dashboard.

@superdav42 superdav42 added the origin:worker Auto-created by pulse labelless backfill (t2112) label Oct 28, 2026
@superdav42

superdav42 commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator Author

Completion Summary


aidevops.sh v3.39.49 plugin for OpenCode v1.19.0 with claude-sonnet-5-5 spent 1m and 2,102 tokens on this as a headless worker.


Merged via PR #2061 to main.
Merged by deterministic merge pass (pulse-wrapper.sh).

Routing feedback

  • Route: standard; 1 attempt, 14 LLM requests, 0 capability escalations, 0 same-tier fallbacks.
  • Population: headless; 0 objective outcomes.
  • Usage: 877,153 tokens; $0.2810 estimated cost.
  • Recommendation: No routing change is recommended from this sample.

@ultimate-multisite ultimate-multisite Bot added the status:in-review Non-draft PR ready for review/merge label Oct 28, 2026
@coderabbitai

coderabbitai Bot commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: cbc6624c-87b6-4f59-a887-edb994ad2069
Commits

Reviewing files that changed from the base of the PR and between bc32c5a and cc70a6a.

Files selected for processing: 1
  • tests/WP_Ultimo/General_Compat_Test.php

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

Walkthrough

Adds parameterized tests for Gutenberg support script enqueueing on the dashboard, post editor, site editor, and widget editor. The tests reset the script registration and screen before and after each case.

Changes: Gutenberg screen coverage

Layer / File(s) Summary
Screen-gate test
tests/WP_Ultimo/General_Compat_Test.php
Adds four screen cases and checks that the support script is not enqueued on the dashboard and is enqueued on the three editor screens. The test resets the script registration and screen around each case.

Priority: Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Possibly related PRs

Merge Risk: Minimal · up to cc70a

No actionable merge-blocking risk was identified in this test-only change.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title names the change: a test covering the Gutenberg support screen gate, with the issue reference GH#2060. The changeset adds a data-provider test for the wu-gutenberg-support script across dash…
Linked Issues check Passed Issue #2060 requests focused coverage for Gutenberg_Support::add_scripts() on the dashboard, post editor, site editor, and block-based widget editor. The added data provider defines all four screen …
Out of Scope Changes check Passed The pull request changes only tests/WP_Ultimo/General_Compat_Test.php. The added provider and test directly implement issue #2060 coverage. The setup and cleanup support isolated screen-gate asserti…
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Full details: Title check

Explanation

The title names the change: a test covering the Gutenberg support screen gate, with the issue reference GH#2060. The changeset adds a data-provider test for the wu-gutenberg-support script across dashboard, post, site, and widget editor screens, so the title matches the main change. It repeats the issue reference, which adds noise but does not make the title misleading.

Full details: Linked Issues check

Explanation

Issue #2060 requests focused coverage for Gutenberg_Support::add_scripts() on the dashboard, post editor, site editor, and block-based widget editor. The added data provider defines all four screen contexts and expected results. The test calls add_scripts() and checks the wu-gutenberg-support enqueue state. It also sets the block-editor state explicitly for each case.

Full details: Out of Scope Changes check

Explanation

The pull request changes only tests/WP_Ultimo/General_Compat_Test.php. The added provider and test directly implement issue #2060 coverage. The setup and cleanup support isolated screen-gate assertions.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 1 files.

Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

🔨 Build Complete - Ready for Testing!

📦 Download Build Artifact (Recommended)

Download the zip build, upload to WordPress and test:

🌐 Test in WordPress Playground (Very Experimental)

Click the link below to instantly test this PR in your browser - no installation needed!
Playground support for multisite is very limitied, hopefully it will get better in the future.

🚀 Launch in Playground

Login credentials: admin / password

@superdav42
superdav42 merged commit 7f9e60e into main Oct 28, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

origin:worker Auto-created by pulse labelless backfill (t2112) status:in-review Non-draft PR ready for review/merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Review followup: PR #2057 — GH#2054: perf: load wu-gutenberg-support only on block editor screens (GH#2054)

1 participant