PHPUnit test for wp_timezone_choice

[pbearne]

started the test but found and issue with setting the locale as the city files is missing the labels
asking on slack for help ​https://wordpress.slack.com/team/U02S95N2X

[swissspidy]

@pbearne Is this question about the missing translations still open?

Strings like "Select a city" are not in the continents-cities-es_ES.mo file but the regular es_ES.mo file.

Note that translation files within tests/phpunit/data/languages/ usually only contain a tiny amount of strings just for testing to keep things simpler.

[pbearne]

@swissspidy yes
I can add the needed strings but need make sure that is OK

[pbearne]

removed extra strings from the translation files

[pbearne]

Had to add the text strings to the new continents-cities-es_ES.l10n.php and ES.l10n.php files

add the string in the new fields to the.po and created .mo for them

[desrosj]

Since this is about adding tests and not a general build or test tooling change, I'm reassigning it to the relevant component.

[audrasjb]

I added some small change requests in the PR ;)

[audrasjb]

Self assigning for final review and commit

[audrasjb]

Moving for 6.9 consideration

[rollybueno]

The PR's latest trunk merged was 6 months ago and the Docker does not initiate. Not until I merge latest trunk after fetching. Command used:

git fetch upstream pull/5694/head:pr-5694
git checkout pr-5694

[rollybueno]

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: ​https://github.com/WordPress/wordpress-develop/pull/5694

Environment

• WordPress: 6.9-alpha-60093-src
• PHP: 8.2.28
• Server: nginx/1.29.0
• Database: mysqli (Server: 8.4.5 / Client: mysqlnd 8.2.28)
• Browser: Chrome 137.0.0.0
• OS: Linux
• Theme: Twenty Twenty-One 2.6
• MU Plugins: None activated
• Plugins:
  • Query Monitor 3.18.0
  • Test Reports 1.2.0

Actual Results

1. ✅ Issue resolved with patch.

Additional Notes

• Merge latest trunk manually when fetching the PR

Supplemental Artifacts

Spanish translations on Timezone:

Unit tests:

[audrasjb]

Removing myself from some tickets as I won't be super available for 6.9.