Changeset 58051

Timestamp:

04/26/2024 03:49:19 PM (11 days ago)

Author:

SergeyBiryukov

Message:

Tests: Add more specific messages for a custom assertion in WP_Test_REST_TestCase.

All assertions in PHPUnit have a $message parameter. Setting this parameter allows to distinguish which assertion is failing when a test runs multiple assertions, making debugging of the tests easier.

Follow-up to [34928], [51478], [58039].

See #60426.

File:

• trunk/tests/phpunit/includes/testcase-rest-api.php (modified) (1 diff)

trunk/tests/phpunit/includes/testcase-rest-api.php

@@ -20 +20 @@
         }
 
-        $this->assertWPError( $response, $message );
-        $this->assertSame( $code, $response->get_error_code(), $message );
+        $this->assertWPError( $response, $message . ' Passed $response is not a WP_Error object.' );
+        $this->assertSame( $code, $response->get_error_code(), $message . ' The expected error code does not match.' );
 
         if ( null !== $status ) {
             $data = $response->get_error_data();
-            $this->assertArrayHasKey( 'status', $data, $message );
-            $this->assertSame( $status, $data['status'], $message );
+            $this->assertArrayHasKey( 'status', $data, $message . ' Passed $response does not include a status code.' );
+            $this->assertSame( $status, $data['status'], $message . ' The expected status code does not match.' );
         }
     }