Repository navigation
Tests: Cover allowed_tags() - #15316
devmuhib009 wants to merge 1 commit into
Conversation
|
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: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. Props Bot commandsYou can trigger Props Bot actions by commenting on this PR:
|
There was a problem hiding this comment.
🟢 Approval recommended
No active review issues are reported for the added PHPUnit coverage of WordPress’s allowed_tags() function.
No open issues 🎉
What changed in this PR
Adds PHPUnit coverage for WordPress’s allowed_tags() function, supporting Trac ticket #65819 without changing production code.
Changes:
- Verifies entity-encoded output for default allowed tags.
- Tests custom tag and attribute formatting, plus empty allowed tags.
- Saves and restores
$allowedtagsaround each test to preserve global state.
| File | Description |
|---|---|
tests/phpunit/tests/general/allowedTags.php |
Adds three tests for default, custom, and empty allowed tags, with global-state cleanup. |
Review effort: Balanced
huzaifaalmesbah
left a comment
There was a problem hiding this comment.
Thanks! Nice coverage for allowed_tags(), including the custom and empty cases. LGTM.
AI Review: Opus 5.5
Trac ticket: https://core.trac.wordpress.org/ticket/65819
This PR adds unit tests for
allowed_tags()covering:$allowedtags$allowedtagshandling