Changeset 52800

Timestamp:

02/25/2022 02:08:53 PM (3 years ago)

Author:

SergeyBiryukov

Message:

Tests: Rename the $success variable to $result in the wp_mail() test for an empty home URL.

The previously name could be a bit confusing if the function is expected to fail.

Follow-up to [52799].

See #54730.

File:

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

trunk/tests/phpunit/tests/mail.php

@@ -242 +242 @@
         add_filter( 'home_url', '__return_empty_string' );
 
-        $success = wp_mail( $to, $subject, $message );
-
-        $this->assertFalse( $success, 'wp_mail() should have returned false' );
+        $result = wp_mail( $to, $subject, $message );
+
+        $this->assertFalse( $result, 'wp_mail() should have returned false' );
         $this->assertGreaterThan( 0, did_action( 'wp_mail_failed' ), 'wp_mail_failed action was not called' );
     }