[HttpFoundation] Tweak previous merged PR

Author: fabpot

Test/Constraint/ResponseFormatSame.php

@@ -25,8 +25,11 @@ final class ResponseFormatSame extends Constraint
     private Request $request;
     private ?string $format;
 
-    public function __construct(Request $request, ?string $format)
-    {
+    public function __construct(
+        Request $request,
+        ?string $format,
+        private readonly bool $verbose = true,
+    ) {
         $this->request = $request;
         $this->format = $format;
     }
@@ -57,6 +60,6 @@ protected function failureDescription($response): string
      */
     protected function additionalFailureDescription($response): string
     {
-        return (string) $response;
+        return $this->verbose ? (string) $response : explode("\r\n\r\n", (string) $response)[0];
     }
 }

Test/Constraint/ResponseIsRedirected.php

@@ -44,12 +44,11 @@ protected function failureDescription($response): string
         return 'the Response '.$this->toString();
     }
 
-    protected function additionalFailureDescription($other): string
+    /**
+     * @param Response $response
+     */
+    protected function additionalFailureDescription($response): string
     {
-        if ($this->verbose || !($other instanceof Response)) {
-            return (string) $other;
-        } else {
-            return explode("\r\n\r\n", (string) $other)[0];
-        }
+        return $this->verbose ? (string) $response : explode("\r\n\r\n", (string) $response)[0];
     }
 }

Test/Constraint/ResponseIsSuccessful.php

@@ -44,12 +44,11 @@ protected function failureDescription($response): string
         return 'the Response '.$this->toString();
     }
 
-    protected function additionalFailureDescription($other): string
+    /**
+     * @param Response $response
+     */
+    protected function additionalFailureDescription($response): string
     {
-        if ($this->verbose || !($other instanceof Response)) {
-            return (string) $other;
-        } else {
-            return explode("\r\n\r\n", (string) $other)[0];
-        }
+        return $this->verbose ? (string) $response : explode("\r\n\r\n", (string) $response)[0];
     }
 }

Test/Constraint/ResponseIsUnprocessable.php

@@ -44,12 +44,11 @@ protected function failureDescription($other): string
         return 'the Response '.$this->toString();
     }
 
-    protected function additionalFailureDescription($other): string
+    /**
+     * @param Response $response
+     */
+    protected function additionalFailureDescription($response): string
     {
-        if ($this->verbose || !($other instanceof Response)) {
-            return (string) $other;
-        } else {
-            return explode("\r\n\r\n", (string) $other)[0];
-        }
+        return $this->verbose ? (string) $response : explode("\r\n\r\n", (string) $response)[0];
     }
 }

Test/Constraint/ResponseStatusCodeSame.php

@@ -44,12 +44,11 @@ protected function failureDescription($response): string
         return 'the Response '.$this->toString();
     }
 
-    protected function additionalFailureDescription($other): string
+    /**
+     * @param Response $response
+     */
+    protected function additionalFailureDescription($response): string
     {
-        if ($this->verbose || !($other instanceof Response)) {
-            return (string) $other;
-        } else {
-            return explode("\r\n\r\n", (string) $other)[0];
-        }
+        return $this->verbose ? (string) $response : explode("\r\n\r\n", (string) $response)[0];
     }
 }