[shail-mehta]

Part of: #64895

What

Replaces the remaining assertNotEquals() call in tests/phpunit/tests/meta.php with assertNotSame() for the user login comparison.

Updated method:

• test_user_metadata_not_exists(): assertNotEquals → assertNotSame for user_login

Why

#64895 asks for strict assertions when both sides share a type. These are user_login strings, so assertNotSame() is appropriate.

How

• Convert the author vs found-user login inequality check from assertNotEquals() to assertNotSame()
• Leave other assertions in the file unchanged

Testing instructions

1. Start the local WordPress develop environment.
2. Run: npm run test:php -- --filter test_user_metadata_not_exists

Use of AI tools

Cursor

[Copilot]

🟢 Approval recommended

Both operands are login strings, so the strict assertion preserves the intended check without introducing a correctness issue.

No open issues 🎉

What changed in this PR

Updates the user metadata query test to use a strict comparison for login strings, aligning with WordPress ticket #64895.

Changes:

• Replaces assertNotEquals() with assertNotSame() in test_user_metadata_not_exists().
• Leaves all other assertions unchanged.

File	Description
tests/​phpunit/​tests/​meta.php	Uses strict inequality for the author and found-user login comparison.

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

[Copilot]

🟢 Approval recommended

The narrow assertion change is appropriate for string login values and preserves the test’s intended check.

No open issues 🎉

_{Review effort: Balanced}

[lancewillett]

Both operands are user-login strings. The strict comparison preserves the check that the meta query excludes the fixture author.

Unit-tests audit completed. Combined randomized runs passed: single-site 497 tests / 1,609 assertions; multisite 521 tests / 1,694 assertions, with one skip in each run. PHPCS and CI passed. Copilot has no open findings.

_{AI review · gpt-6}

[lancewillett]

PR #14855 landed in https://core.trac.wordpress.org/changeset/64782