Changeset 47064

Timestamp:

01/12/2020 02:16:57 PM (5 years ago)

Author:

SergeyBiryukov

Message:

Tests: In Tests_Site_Health, create a WP_Site_Health instance before clearing the cron array, as the constructor schedules its own task now.

See #47606.

File:

• trunk/tests/phpunit/tests/site-health.php (modified) (3 diffs)

trunk/tests/phpunit/tests/site-health.php

@@ -16 +16 @@
      */
     function test_cron_health_checks_critical() {
+        $wp_site_health = new WP_Site_Health();
+
         // Clear the cron array.
         _set_cron_array( array() );
-        $wp_site_health = new WP_Site_Health();
-        $cron_health    = $wp_site_health->get_test_scheduled_events();
+
+        $cron_health = $wp_site_health->get_test_scheduled_events();
 
         $this->assertSame( 'critical', $cron_health['status'] );
@@ -34 +36 @@
      */
     function test_cron_health_checks( $times, $expected_status, $expected_label, $expected_late, $expected_missed ) {
+        $wp_site_health = new WP_Site_Health();
+
         /*
          * Clear the cron array.
@@ -48 +52 @@
         }
 
-        $wp_site_health = new WP_Site_Health();
-        $cron_health    = $wp_site_health->get_test_scheduled_events();
+        $cron_health = $wp_site_health->get_test_scheduled_events();
 
         $this->assertSame( $expected_status, $cron_health['status'] );