[shail-mehta]

Trac ticket: #65819

What

Adds a dedicated PHPUnit class for get_comment_ID() and comment_ID(), covering ID retrieval from the global comment, the empty fallback, the filter, and display output.

Test-only; no production code changes.

Why

These helpers had no direct @covers coverage. Small helper coverage fits the #65819 test cleanup track.

How

Add tests/phpunit/tests/comment/getCommentID.php with @covers for both functions.

Testing Instructions

1. npm run test:php -- --filter Tests_Comment_GetCommentID
2. npm run test:php -- --filter Tests_Comment_GetCommentID -c tests/phpunit/multisite.xml
3. npm run test:php -- --filter Tests_Comment_GetCommentID --order-by=random
4. npm run test:php -- --group comment

Use of AI Tools

• Cursor

[Copilot]

🟡 Changes recommended

The empty-fallback assertion remains order-dependent because it does not clear $GLOBALS['comment'] beforehand, allowing a stale comment ID from earlier tests to cause a failure.

1 open issue

• Clear the global comment before asserting the empty fallback · New

What changed in this PR

Adds direct PHPUnit coverage for the comment ID helpers as part of the #65819 test cleanup effort. No production code changes are included.

Changes:

• Tests ID retrieval from the global comment and the empty fallback.
• Verifies filter customization and echoed output.
• Adds explicit @covers annotations for both helpers.

File	Description
tests/​phpunit/​tests/​comment/​getCommentID.php	Adds fixtures and four tests for get_comment_ID() and comment_ID().

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

There are no active review issues for this test-only change adding PHPUnit coverage of the comment ID helpers.

No open issues 🎉

1 resolved since last review

• Clear the global comment before asserting the empty fallback

_{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.

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]

Tests string ID and empty fallback. Current unset before the no-comment assertion fixes the reported order dependency.

Local randomized single-site and multisite checks passed with related existing tests: 51 tests, 319 assertions across this five-PR batch (seed 15273). PHPCS passed.

Optional: Echo and constant-filter exercises are lower value than the string-ID and empty-fallback cases; no further edge cases needed.

_{AI review · gpt-6}

[lancewillett]

PR #15276 landed in https://core.trac.wordpress.org/changeset/65137