[pbearne]

Trac ticket: https://core.trac.wordpress.org/ticket/57224

[mukeshpanchal27]

Thanks @pbearne Left some nit-pick.

[pbearne]

@mukeshpanchal27 Thanks

[Copilot]

🟡 Changes recommended

The tests remain order-dependent because they do not clear existing submenu entries, and one test fails to restore the original widget-support state.

2 open issues

• Reset $submenu before order-dependent assertions · New
• Use _remove_theme_support() to restore widget support · New

What changed in this PR

Adds PHPUnit coverage for WordPress’s wp_widgets_add_menu() function, associated with Trac ticket #57224. No production code changes are included.

Changes:

• Tests that no submenu is added without widget support.
• Tests Widgets submenu placement with widget support and with block template parts enabled.

File	Description
tests/​phpunit/​tests/​functions/​wpWidgetsAddMenu.php	Adds three tests for Widgets submenu creation and placement.

_{Review effort: Balanced}

[lancewillett]

This needs a behavior/fixture refresh for current core: classic widgets now occupy index 9, and block-template-parts support no longer selects the block-theme branch. Please control the theme and submenu inputs before asserting the Widgets entry. This also overlaps #5830; consolidate the alternatives rather than landing both.

_{AI review · gpt-6}

[github-actions]

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs.

Core Committers: Use this line as a base for the props when committing in SVN:

Props pbearne, mukesh27, lancewillett.

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

---

Props Bot commands

You can trigger Props Bot actions by commenting on this PR:

• @propsbot add <username> to give props to a GitHub user. Requires write access.
• @propsbot remove <username> to leave a user out of the props. Requires write access.
• @propsbot refresh to refresh this list. Adding the props-bot label works too.

[Copilot]

🟡 Changes recommended

The new test expects the Widgets submenu at index 7, but the existing implementation uses index 9, causing the test to fail.

1 open issue

• Correct expected Widgets submenu index to 9 · New

2 resolved since last review

• Use _remove_theme_support() to restore widget support
• Reset $submenu before order-dependent assertions

_{Review effort: Balanced}

[Copilot]

🟡 Changes recommended

The test’s empty submenu allows it to pass even if existing Appearance entries are overwritten; it should seed an entry and verify that Widgets is appended at the next numeric index.

1 open issue

• Test that block theme submenu entries are appended · New

1 resolved since last review

• Correct expected Widgets submenu index to 9

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

No active review issues are identified for the added widget submenu tests.

No open issues 🎉

1 resolved since last review

• Test that block theme submenu entries are appended

_{Review effort: Balanced}