Changeset 47491 for branches/3.8/tests/phpunit/includes/testcase.php

Timestamp:

03/22/2020 02:43:45 PM (5 years ago)

Author:

SergeyBiryukov

Message:

When asserting microtime output as a number, make it a number.

microtime is by default a string. Doing a greater then or less than check of that string is a bad idea since it uses the first part (the micro part of microtime) rather then the actual time. This adds a helper to convert microtime output into a float which we can then use to properly compare the output of microtime.

This fixes an intermittent test failure.

Props jorbin.
Merges [30337] to the 3.8 branch.
See #30336, #49485.

Location:

branches/3.8

Files:

• . (modified) (1 prop)
• tests/phpunit/includes/testcase.php (modified) (2 diffs)

branches/3.8/tests/phpunit/includes/testcase.php

@@ -163 +163 @@
         }
     }
-    
+
     function flush_cache() {
         global $wp_object_cache;
@@ -503 +503 @@
         return $files;
     }
+
+    /**
+     * Helper to Convert a microtime string into a float
+     */
+    protected function _microtime_to_float($microtime ){
+        $time_array = explode( ' ', $microtime );
+        return array_sum( $time_array );
+    }
 }