Repository navigation
GH#2060: test: cover Gutenberg support screen gate (GH#2060) - #2061
Conversation
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. Routing feedback
|
|
No actionable comments were generated in the recent review.
|
| 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:
Estimated code review effort: 2 (Simple) | ~8 minutes
Change: Other
Possibly related PRs
- Ultimate-Multisite/ultimate-multisite#2057: Adds the screen gate that these tests cover.
Merge Risk: 
Minimal · up to cc70a
-
No actionable merge-blocking risk was identified in this test-only change.
Pre-merge checks: 
5
-
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
#2060requests focused coverage forGutenberg_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 callsadd_scripts()and checks thewu-gutenberg-supportenqueue 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#2060coverage. 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.
Comment @coderabbitai help to get the list of available commands.
🔨 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! Login credentials: |
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
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