Make WordPress Core


Ignore:
Timestamp:
03/22/2020 02:43:22 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.9 branch.
See #30336, #49485.

Location:
branches/3.9
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • branches/3.9

  • branches/3.9/tests/phpunit/includes/testcase.php

    r47329 r47490  
    166166        }
    167167    }
    168    
     168
    169169    function flush_cache() {
    170170        global $wp_object_cache;
     
    506506        return $files;
    507507    }
     508
     509    /**
     510     * Helper to Convert a microtime string into a float
     511     */
     512    protected function _microtime_to_float($microtime ){
     513        $time_array = explode( ' ', $microtime );
     514        return array_sum( $time_array );
     515    }
    508516}
Note: See TracChangeset for help on using the changeset viewer.