[shail-mehta]

Trac ticket: #66480

What

Adds a dedicated PHPUnit class for get_self_link(), covering scheme-aware feed self URL construction from REQUEST_URI.

Test-only; no production code changes.

Why

get_self_link() had no direct @covers coverage. It builds the Atom/RSS self link used by feed templates. Small helper coverage fits the #65819 test cleanup track.

How

Add tests/phpunit/tests/feed/getSelfLink.php with @covers ::get_self_link, setting $_SERVER['REQUEST_URI'] and asserting the scheme-aware URL.

Testing Instructions

docker compose up -d mysql
docker compose run --rm php ./vendor/bin/phpunit --filter Tests_Feed_GetSelfLink

## Use of AI Tools
- Cursor

[Copilot]

🟡 Changes recommended

The test exercises only HTTP and would still pass without set_url_scheme(), leaving scheme-aware behavior unprotected without explicit HTTP and HTTPS assertions.

1 open issue

• Add HTTPS coverage for self-link scheme handling · New

What changed in this PR

Adds dedicated PHPUnit coverage for get_self_link(), the helper that constructs feed self URLs, as part of the #65819 test cleanup effort. No production code changes are included.

Changes:

• Adds a feed test class with @covers ::get_self_link.
• Checks URL construction from REQUEST_URI, including a query string and the home URL’s host and optional port.

File	Description
tests/​phpunit/​tests/​feed/​getSelfLink.php	Adds a direct test of feed self URL construction.

_{Review effort: Balanced}

[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 shailu25, 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.

[lancewillett]

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 has-patch and has-unit-tests keywords, and update the Trac link in this PR's description. I will review it under the new ticket.

_{AI review · claude-opus-5-5}

[shail-mehta]

Created Ticket: #66480