[shail-mehta]

Trac ticket: #65819

What

Strengthens the existing Tests_Functions::test_wp_ext2type() coverage with literal extension-to-type mappings that stay independent of the registry.

Test-only; no production code changes.

Why

The existing test already walks all registered extensions and the unknown result. A few literal rows add independence from the registry without a second class, and fit the #65819 cleanup track.

How

Update tests/phpunit/tests/functions.php:

• Assert literal mappings for jpg/PNG → image, mp3 → audio, mp4 → video, and pdf → document
• Add @covers ::wp_ext2type

Testing Instructions

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

Use of AI Tools

• Cursor

[Copilot]

🟢 Approval recommended

The change adds dedicated wp_ext2type() test coverage, with no active review issues identified.

No open issues 🎉

What changed in this PR

Adds direct PHPUnit coverage for wp_ext2type() as part of the test cleanup work in Trac #65819. No production code changes.

Changes:

• Adds a dedicated test class with @covers ::wp_ext2type.
• Checks known extension mappings and uppercase normalization.
• Verifies that an unknown extension returns null.

File	Description
tests/​phpunit/​tests/​functions/​wpExt2type.php	Adds data-driven mapping tests and an unknown-extension test.

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

Please consolidate this with Tests_Functions::test_wp_ext2type() in tests/phpunit/tests/functions.php:1151–1164. That test already checks all registered extensions in both cases and the unknown result. The literal mappings add independence from the registry, but can strengthen the existing test without repeating the same lookup scenarios in another class.

Existing coverage.

This is a request to keep the coverage lean, not a reported runtime regression.

_{AI review · gpt-6}

[Copilot]

🟢 Approval recommended

No active review issues remain for the added extension-to-type test coverage.

No open issues 🎉

_{Review effort: Balanced}