Changeset 54513 for branches/5.9/tests/phpunit/tests/date/wpTimezone.php

Timestamp:

10/14/2022 06:49:30 AM (2 years ago)

Author:

peterwilsoncc

Message:

Tests: Replace the timezone used in date/time tests.

The Europe/Kiev timezone has been deprecated in PHP 8.2 and replaced with Europe/Kyiv.

The tests updated in this commit are testing the WordPress date/time functionality. They are not testing whether WP or PHP can handle deprecated timezone names correctly.

To ensure the tests follow the original purpose, the use of Europe/Kiev within these tests is now replaced with the Europe/Helsinki timezone, which is within the same timezone as Europe/Kyiv. This should ensure that these tests run without issue and test what they are supposed to be testing on every supported PHP version (unless at some point in the future Europe/Helsinki would be renamed, but that's a bridge to cross if and when).

Note: Separate tests should/will be added to ensure that relevant date/time related functions handle a deprecated timezone correctly, but that is not something these tests are supposed to be testing.

Follow-up to [45853], [45856], [45876], [45882], [45887], [45908], [45914], [46577], [46154], [46580], [46864], [46974], [54207].

Props jrf, costdev, SergeyBiryukov.
Merges [54217] to the 5.9 branch.
See #56468.

Location:

branches/5.9

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/date/wpTimezone.php (modified) (1 diff)

branches/5.9/tests/phpunit/tests/date/wpTimezone.php

@@ -32 +32 @@
      */
     public function test_should_return_timezone_string() {
-        update_option( 'timezone_string', 'Europe/Kiev' );
+        update_option( 'timezone_string', 'Europe/Helsinki' );
 
-        $this->assertSame( 'Europe/Kiev', wp_timezone_string() );
+        $this->assertSame( 'Europe/Helsinki', wp_timezone_string() );
 
         $timezone = wp_timezone();
 
-        $this->assertSame( 'Europe/Kiev', $timezone->getName() );
+        $this->assertSame( 'Europe/Helsinki', $timezone->getName() );
     }