Tests: Add unit tests for the Bookmark Administration API - #13362
Tests: Add unit tests for the Bookmark Administration API#13362itzmekhokan wants to merge 1 commit into
Conversation
Adds coverage for the functions in `src/wp-admin/includes/bookmark.php`, none of which were previously exercised directly: `add_link()`, `edit_link()`, `get_default_link_to_edit()`, `wp_delete_link()`, `wp_get_link_cats()`, `get_link_to_edit()`, `wp_insert_link()`, `wp_set_link_cats()`, `wp_update_link()`, and `wp_link_manager_disabled_message()`. Fixes #66019.
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
1 participant
None of the ten functions in
src/wp-admin/includes/bookmark.phphad direct test coverage;wp_insert_link()andwp_delete_link()appeared only as fixtures for theget_bookmark()tests.This adds a test file per function under
tests/phpunit/tests/admin/includes/bookmark/, mirroring the source path and the structure used forwp-admin/includes/misc.phpin [62369]. The cases pin the documented contracts: the empty-URL and name-fallback returns inwp_insert_link(), its$wp_errorbranch, themanage_linkscheck and$_POSTescaping inedit_link(), category replacement and integer casting inwp_set_link_cats(), the partial-merge behaviour ofwp_update_link(), and the branches ofwp_link_manager_disabled_message().No production code is changed.
Trac ticket: https://core.trac.wordpress.org/ticket/66019
Use of AI Tools
AI assistance: Yes
Tool(s): Claude Code
Model(s): Claude Opus 5
Used for: writing the test cases and checking that each one fails when the function under test is broken. All changes were reviewed and validated by me.
This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.