Changeset 58109

Timestamp:

05/06/2024 06:47:42 PM (9 months ago)

Author:

johnbillion

Message:

Cron API: Add tests for associative and indexed array arguments when rescheduling a cron event.

Props johnbillion, peterwilsoncc

See #57271

File:

• trunk/tests/phpunit/tests/cron.php (modified) (2 diffs)

trunk/tests/phpunit/tests/cron.php

@@ -489 +489 @@
 
         // Reschedule event with preflight filter in place.
-        wp_reschedule_event( $ts1, 'daily', $hook );
+        $rescheduled = wp_reschedule_event( $ts1, 'daily', $hook );
+
+        // Check return value.
+        $this->assertTrue( $rescheduled );
 
         // Check cron option is unchanged.
@@ -1024 +1027 @@
 
     /**
+     * @ticket 57271
+     *
+     * @dataProvider data_wp_reschedule_event_works_with_args
+     *
+     * @covers ::wp_reschedule_event
+     */
+    public function test_wp_reschedule_event_works_with_args( array $args ) {
+        $time = time();
+
+        // Schedule events with the `$wp_error` parameter:
+        $event             = wp_schedule_event( $time, 'daily', 'hook', $args, true );
+        $rescheduled_event = wp_reschedule_event( $time, 'daily', 'hook', $args, true );
+        $unscheduled_event = wp_unschedule_event( $time, 'hook', $args, true );
+        $next_timestamp    = wp_next_scheduled( 'hook', $args );
+
+        // Ensure the events were added and updated correctly:
+        $this->assertNotWPError( $event );
+        $this->assertNotWPError( $rescheduled_event );
+        $this->assertNotWPError( $unscheduled_event );
+        $this->assertSame( $time + DAY_IN_SECONDS, $next_timestamp );
+    }
+
+    /**
+     * Data provider for test_wp_reschedule_event_works_with_args().
+     *
+     * @return array[]
+     */
+    public function data_wp_reschedule_event_works_with_args() {
+        return array(
+            'indexed'     => array(
+                array(
+                    1,
+                    2,
+                    3,
+                ),
+            ),
+            'associative' => array(
+                array(
+                    'one'   => 1,
+                    'two'   => 2,
+                    'three' => 3,
+                ),
+            ),
+        );
+    }
+
+    /**
      * @ticket 49961
      * @expectedDeprecated wp_clear_scheduled_hook