Changeset 53473 for trunk/tests/phpunit/tests/pluggable.php

Timestamp:

06/06/2022 10:48:21 PM (2 years ago)

Author:

audrasjb

Message:

General: Ensure wp_rand() returns 0 when $min and $max values are equal to 0.

This changeset ensures wp_rand() returns zero instead of a random number when both $min and $max values are equal to zero.

Fixes #55194.

File:

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

trunk/tests/phpunit/tests/pluggable.php

@@ -225 +225 @@
             ),
             'wp_rand'                         => array(
-                'min' => 0,
-                'max' => 0,
+                'min' => null,
+                'max' => null,
             ),
             'wp_set_password'                 => array( 'password', 'user_id' ),
@@ -355 +355 @@
         $this->assertSame( $current_user, $from_get_user_by );
     }
+
+    /**
+     * Tests that wp_rand() returns zero.
+     *
+     * @ticket 55194
+     * @dataProvider data_wp_rand_should_return_zero
+     * @covers ::wp_rand
+     *
+     * @param mixed $min Lower limit for the generated number.
+     * @param mixed $max Upper limit for the generated number.
+     */
+    public function test_wp_rand_should_return_zero( $min, $max ) {
+        $this->assertSame( 0, wp_rand( $min, $max ) );
+    }
+
+    /**
+     * Data provider.
+     *
+     * @return array
+     */
+    public function data_wp_rand_should_return_zero() {
+        return array(
+            'min and max as 0'      => array(
+                'min' => 0,
+                'max' => 0,
+            ),
+            'min and max as 0.0'    => array(
+                'min' => 0.0,
+                'max' => 0.0,
+            ),
+            'min as null, max as 0' => array(
+                'min' => null,
+                'max' => 0,
+            ),
+        );
+    }
+
+    /**
+     * Tests that wp_rand() returns a value between 1 and 99.
+     *
+     * @ticket 55194
+     * @dataProvider data_wp_rand_should_return_between_1_and_99
+     * @covers ::wp_rand
+     *
+     * @param int $min Lower limit for the generated number.
+     * @param int $max Upper limit for the generated number.
+     */
+    public function test_wp_rand_should_return_between_1_and_99( $min, $max ) {
+        $this->assertGreaterThan(
+            0,
+            wp_rand( $min, $max ),
+            'The value was not greater than 0'
+        );
+
+        $this->assertLessThan(
+            100,
+            wp_rand( $min, $max ),
+            'The value was not less than 100'
+        );
+    }
+
+    /**
+     * Data provider.
+     *
+     * @return array
+     */
+    public function data_wp_rand_should_return_between_1_and_99() {
+        return array(
+            '1 and 99'       => array(
+                'min' => 1,
+                'max' => 99,
+            ),
+            '-1 and 99'      => array(
+                'min' => -1,
+                'max' => 99,
+            ),
+            '1 and -99'      => array(
+                'min' => 1,
+                'max' => -99,
+            ),
+            '-1 and -99'     => array(
+                'min' => -1,
+                'max' => -99,
+            ),
+            '1.0 and 99.0'   => array(
+                'min' => 1.0,
+                'max' => 99.0,
+            ),
+            '-1.0 and -99.0' => array(
+                'min' => -1.0,
+                'max' => -99.0,
+            ),
+        );
+    }
 }