Changeset 8676 for trunk/tests/phpunit/testcases/core/functions.php

Timestamp:

07/23/2014 06:07:39 PM (12 years ago)

Author:

r-a-y

Message:

Fix issues with bp_core_time_since().

Previously, when the $newer_date parameter isn't passed, we generated a
UNIX timestamp using strtotime() and bp_core_current_time(). strtotime()
uses the default timezone, which can cause timezone discrepencies if the
server time is not UTC.

This commit removes the strtotime() usage and refactors bp_core_current_time()
to accept current_time()'s 'type' parameter. This allows us to correctly
grab the UTC timestamp when setting bp_core_current_time()'s second
argument to 'timestamp'.

Commit also adds some unit tests.

Fixes #4310.

File:

• trunk/tests/phpunit/testcases/core/functions.php (modified) (1 diff)

trunk/tests/phpunit/testcases/core/functions.php

@@ -346 +346 @@
         $this->assertNotEmpty( buddypress()->loaded_components['foo'] );
     }
+
+    /**
+     * @group bp_core_time_since
+     * @group bp_core_current_time
+     */
+    public function test_bp_core_time_since_timezone_right_now() {
+        // backup timezone
+        $tz_backup = date_default_timezone_get();
+
+        // set timezone to something other than UTC
+        date_default_timezone_set( 'Europe/Paris' );
+
+        $this->assertSame( 'right now', bp_core_time_since( time() ) );
+
+        // revert timezone back to normal
+        if ( $tz_backup ) {
+            date_default_timezone_set( $tz_backup );
+        }
+    }
+
+    /**
+     * @group bp_core_time_since
+     * @group bp_core_current_time
+     */
+    public function test_bp_core_time_since_timezone() {
+        // backup timezone
+        $tz_backup = date_default_timezone_get();
+
+        // set timezone to something other than UTC
+        date_default_timezone_set( 'Europe/Paris' );
+
+        $this->assertSame( '1 hour ago', bp_core_time_since( time() - 60*60 ) );
+
+        // revert timezone back to normal
+        if ( $tz_backup ) {
+            date_default_timezone_set( $tz_backup );
+        }
+    }
 }