Repository navigation
Tests: Add unit coverage for bloginfo_rss() - #15425
shail-mehta wants to merge 2 commits into
Conversation
Cover the bloginfo_rss filter arguments and echoed output. See #65819.
There was a problem hiding this comment.
🟢 Approval recommended
There are no active review issues for this test-only addition covering the filter and echo behavior of bloginfo_rss().
No open issues 🎉
What changed in this PR
Adds direct PHPUnit coverage for bloginfo_rss() to address Trac #65819, focusing on its display-filter and echo contract. No production code changes.
Changes:
- Adds a dedicated test class with
@covers ::bloginfo_rss. - Verifies filtered output, a single filter invocation, and the value and selector passed to the filter.
| File | Description |
|---|---|
tests/phpunit/tests/feed/bloginfoRss.php |
Adds one test with three assertions covering filter arguments, call count, and echoed output. |
Review effort: Balanced
|
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:
|
|
Thanks, @shail-mehta. Per #65819 comment 1521, test-only changes now get their own Trac ticket in the component of the code under test, instead of #65819. That now includes open PRs like this one. Could you open a new ticket for this change (likely Feeds)? Please add the AI review · |
|
Created Ticket: #66478 |
Trac ticket: #66478
What
Adds a dedicated PHPUnit class for
bloginfo_rss(), covering thebloginfo_rssfilter arguments and echoed output.Test-only; no production code changes.
Why
bloginfo_rss()had no direct@coverscoverage.get_bloginfo_rss()already covers value conversion; this keeps the display filter + echo contract.How
Add
tests/phpunit/tests/feed/bloginfoRss.phpwith@covers ::bloginfo_rss, asserting filter call count, arguments, and output.Testing Instructions
Expected: OK (1 test, 3 assertions).
Use of AI Tools