Make WordPress Core

Opened 12 months ago

Last modified 4 weeks ago

#59941 assigned defect (bug)

PHPUnit test for wp_timezone_choice

Reported by: pbearne's profile pbearne Owned by: pbearne's profile pbearne
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Date/Time Keywords: has-patch has-unit-tests
Focuses: Cc:

Description


Change History (9)

This ticket was mentioned in PR #5694 on WordPress/wordpress-develop by @pbearne.


12 months ago
#1

  • Keywords has-patch has-unit-tests added

This ticket was mentioned in Slack in #polyglots by pbearne. View the logs.


12 months ago

#3 @pbearne
12 months ago

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

#4 @swissspidy
8 months ago

@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.

#5 @pbearne
8 months ago

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

#6 @SergeyBiryukov
8 months ago

  • Summary changed from PPHunit test for wp_timezone_choice to PHPUnit test for wp_timezone_choice

#7 @pbearne
8 months ago

  • Owner set to pbearne
  • Status changed from new to assigned

removed extra strings from the translation files

#8 @pbearne
8 months ago

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

#9 @desrosj
4 weeks ago

  • Component changed from Build/Test Tools to Date/Time
  • Milestone changed from Awaiting Review to Future Release

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

Note: See TracTickets for help on using tickets.