Changeset 59312 for trunk/tests/phpunit/tests/user/retrievePassword.php

Timestamp:

10/28/2024 08:03:13 PM (22 months ago)

Author:

SergeyBiryukov

Message:

Users: Set correct default value for $user_login in retrieve_password().

This resolves a "passing null to non-nullable" deprecation notice on PHP 8.1+:

Deprecated: trim(): Passing null to parameter #1 ($string) of type string is deprecated

Follow-up to [50129], [54477].

Props afragen, peterwilsoncc, SergeyBiryukov.
Fixes #62298.

File:

• trunk/tests/phpunit/tests/user/retrievePassword.php (modified) (1 diff)

trunk/tests/phpunit/tests/user/retrievePassword.php

@@ -75 +75 @@
                 $this->assertTrue( retrieve_password( 'bar@example.com' ), 'Fetching user by email failed.' );
         }
+
+        /**
+         * Tests that PHP 8.1 "passing null to non-nullable" deprecation notice
+         * is not thrown when the `$user_login` parameter is empty.
+         *
+         * The notice that we should not see:
+         * `Deprecated: trim(): Passing null to parameter #1 ($string) of type string is deprecated`.
+         *
+         * @ticket 62298
+         */
+        public function test_retrieve_password_does_not_throw_deprecation_notice_with_default_parameters() {
+                $this->assertWPError( retrieve_password() );
+        }
 }